Merge lp://qastaging/~javier.collado/utah/bug1039540 into lp://qastaging/utah

Proposed by Javier Collado
Status: Rejected
Rejected by: Javier Collado
Proposed branch: lp://qastaging/~javier.collado/utah/bug1039540
Merge into: lp://qastaging/utah
Diff against target: 12 lines (+1/-1)
1 file modified
utah/run.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~javier.collado/utah/bug1039540
Reviewer Review Type Date Requested Status
UTAH Dev Pending
Review via email: mp+121528@code.qastaging.launchpad.net

Description of the change

This one line change should fix the permissions problems when running test cases on a machine provisioned from scratch.

According to my experience, what works fine for me is:
- Avoid running the utah client as root
- Use always `utah` user to run test cases

I think the client has been executed as root for some time, so there must be some other thing that has changed. Anyway, I believe that running the test cases as `utah` user should be the default to avoid permissions problems. In particular, what I've seen is that when the client is executed as `root` and, after that, as `utah` there are problems to remove test files that were left by the previous run.

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

If I run test cases as the utah user, can they reboot if necessary? It seems like that wasn't implemented yet a while ago, but if it is now, I can move all my stuff to the user instead of root.

Revision history for this message
Javier Collado (javier.collado) wrote :

I haven't tested that. Do you have an example that I could use? Thanks.

Revision history for this message
Max Brustkern (nuclearbob) wrote :

This is a master runlist I use for bootspeed testing that includes reboots:
---
- name: prepare
  fetch_cmd: rm -rf prepare && bzr export prepare lp:~nuclearbob/+junk/utah_bootspeed_testsuite/prepare

Revision history for this message
Javier Collado (javier.collado) wrote :

I've tested that now and it doesn't work. In the logs I see multiple error messages regarding permissions problems. I guess that the test cases aren't being executed as root after all.

I'll reject this merge proposal since it isn't really solving the problem and think about alternative solutions.

Unmerged revisions

661. By Javier Collado

Updated utah command to be executed as regular user

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches