Closed Bug 863258 Opened 11 years ago Closed 11 years ago

give cltbld write permission to /dev/video0, and add to audio group on ec2 ubuntu linux slaves

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

Product:
Infrastructure & Operations Graveyard
Component:
CIDuty
Platform:
x86_64
Windows 7
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: ted, Assigned: jhopkins)

References

Details

Attachments

(2 files)

99-video4linux.rules
38 bytes, text/plain
Details
[puppet] add cltbld to ['video','audio'] groups on Ubuntu
1.59 KB, patch
dustin
: review+
ted
: feedback+
jhopkins
: checked-in+
Details | Diff | Splinter Review
Attached file 99-video4linux.rules — 
Bug 826265 added the v4l2loopback driver to the Linux test slaves. Unfortunately /dev/video0 is chmod 0600, with group "video", which cltbld is not currently in. There are two easy ways to fix this:
a) Add cltbld to the video group
b) Use a udev rule to chmod it 0666

Attached is a udev rule you can drop in /etc/udev/rules.d/ to do the latter. I don't care which way you fix it, they should both work fine.
This is for the ubuntu ec2 linux slaves, right?
Assignee: nobody → jhopkins
Correct, sorry.
Summary: give cltbld write permission to /dev/video0 → give cltbld write permission to /dev/video0 on ec2 linux slaves
I also need cltbld to be in the audio group, as it turns out. (Could probably fudge this with udev magic as well, but I haven't tested.)
Summary: give cltbld write permission to /dev/video0 on ec2 linux slaves → give cltbld write permission to /dev/video0, and add to audio group on ec2 ubuntu linux slaves
If I had to guess, probably just:
SUBSYSTEM=="sound" MODE="0666"

        Attachment #740391 -
        Flags: review?(rail)

        Attachment #740391 -
        Flags: feedback?(ted)

    
    

    
  
Comment on attachment 740391 [details] [diff] [review]
[puppet] add cltbld to ['video','audio'] groups on Ubuntu

302 dustin, since he is the initial author of that module.

        Attachment #740391 -
        Flags: review?(rail) → review?(dustin)

    
    

    
  
Comment on attachment 740391 [details] [diff] [review]
[puppet] add cltbld to ['video','audio'] groups on Ubuntu

This seems plausible, although I don't speak puppet specifically.

        Attachment #740391 -
        Flags: feedback?(ted) → feedback+

        Attachment #740391 -
        Flags: review?(dustin) → review+

    
    

    
  
Test results...

Prior to the patch:

cltbld@tst-linux64-ec2-003:~$ groups
cltbld

With the patch applied:

cltbld@tst-linux64-ec2-001:~$ groups
cltbld audio video

Also tested on Linux and Mac build slaves to confirm no changes will be made (we only want to update Ubuntu build slaves).

    
    

    
  
Comment on attachment 740391 [details] [diff] [review]
[puppet] add cltbld to ['video','audio'] groups on Ubuntu

Landed in http://hg.mozilla.org/build/puppet/rev/1148b6746afa

        Attachment #740391 -
        Flags: checked-in+

    
  
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering

    
  
Product: Release Engineering → Infrastructure & Operations

    
  
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: