Auto Package Testing

Merge lp://staging/~psivaa/auto-package-testing/console-cleaning into lp://staging/~canonical-ci-engineering/auto-package-testing/adt-nova-support

  1. console-cleaning
  2. Merge into adt-nova-support
Proposed by Para Siva
Status: Needs review
Proposed branch: lp://staging/~psivaa/auto-package-testing/console-cleaning
Merge into: lp://staging/~canonical-ci-engineering/auto-package-testing/adt-nova-support
Diff against target: 28 lines (+3/-3)
1 file modified
jenkins/run-nova-autopkgtest (+3/-3)
To merge this branch: bzr merge lp://staging/~psivaa/auto-package-testing/console-cleaning
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Paul Larson Needs Information
Review via email: mp+253803@code.staging.launchpad.net

Commit message

Console log cleaning. Suppresses curl progress output.

Description of the change

Console log cleaning. Suppresses curl progress output.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

Have you also explored using curl -s instead? I worry that we might unintentionally hide an error by doing this, and -s should eliminate the transfer stats we currently see in the log.

review: Needs Information
Reply
lp://staging/~psivaa/auto-package-testing/console-cleaning updated
411. By Para Siva

Use -sS to suppress the progress bar and show error

Revision history for this message
Para Siva (psivaa) wrote :

Thanks for the valid point Paul. I have now used -sS since I saw some posts complaining about -s alone not working in some versions of curl.

Could you check again please?

Reply
Revision history for this message
Francis Ginther (fginther) wrote :

Approve

review: Approve
Reply

Unmerged revisions

411. By Para Siva

Use -sS to suppress the progress bar and show error

410. By Para Siva

Less uglier console output for nova adt runs

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