QA Jenkins Jobs

Merge lp://staging/~canonical-platform-qa/qa-jenkins-jobs/remove-sudo-when-cleaning-workspace into lp://staging/qa-jenkins-jobs

  1. remove-sudo-when-cleaning-workspace
  2. Merge into trunk
Proposed by Allan LeSage
Status: Work in progress
Proposed branch: lp://staging/~canonical-platform-qa/qa-jenkins-jobs/remove-sudo-when-cleaning-workspace
Merge into: lp://staging/qa-jenkins-jobs
Diff against target: 12 lines (+1/-1)
1 file modified
macros.yaml (+1/-1)
To merge this branch: bzr merge lp://staging/~canonical-platform-qa/qa-jenkins-jobs/remove-sudo-when-cleaning-workspace
Reviewer Review Type Date Requested Status
platform-qa-bot continuous-integration Approve
Max Brustkern Pending
Review via email: mp+295884@code.staging.launchpad.net

Commit message

Don't sudo when cleaning workspace.

Description of the change

Don't sudo when cleaning workspace; I get a tty request which fails. We really shouldn't be doing anything as sudo IMO!

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

The provisioning for upgrade testing needs to run with sudo, so presently, the upgrade testing command is run with sudo, so any artifacts it creates are owned by root, so we need sudo to remove it. We can restrict that to just the upgrade testing jobs for now while we work on a fix.

Reply
Revision history for this message
platform-qa-bot (platform-qa-bot) wrote :

PASSED: Continuous integration, rev:48
https://platform-qa-jenkins.ubuntu.com/job/qa-jenkins-jobs-ci/5/
Executed test runs:
    SUCCESS: https://platform-qa-jenkins.ubuntu.com/job/build-wily-amd64-package/372
        deb: https://platform-qa-jenkins.ubuntu.com/job/build-wily-amd64-package/372/artifact/work/output/*zip*/output.zip
    SUCCESS: https://platform-qa-jenkins.ubuntu.com/job/build-wily-i386-package/372
        deb: https://platform-qa-jenkins.ubuntu.com/job/build-wily-i386-package/372/artifact/work/output/*zip*/output.zip
    None: https://platform-qa-jenkins.ubuntu.com/job/generic-update-mp/327/console

Click here to trigger a rebuild:
https://platform-qa-jenkins.ubuntu.com/job/qa-jenkins-jobs-ci/5/rebuild

review: Approve (continuous-integration)
Reply

Unmerged revisions

48. By Allan LeSage

Remove sude when cleaning workspace.

47. By Max Brustkern

Added hwe jobs

46. By Allan LeSage

Merge use-ubuntu-system-tests-run .

45. By Max Brustkern

Manually installing dl-ubuntu-test-iso to get the latest version

44. By Max Brustkern

Merged trigger changes

43. By Max Brustkern

Merging trunk

42. By Max Brustkern

Removing spurious rules

41. By Max Brustkern

Added packaging

40. By Max Brustkern

Renamed write-config.sh to write-ust-config.sh to keep it less ambiguous when installed

39. By Allan LeSage

Merge restore-config-de-endlining.

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