Merge lp:~gesha/linaro-license-protection/remove-artifacts-from-upload into lp:~linaro-automation/linaro-license-protection/trunk
Proposed by
Georgy Redkozubov
Status: | Merged |
---|---|
Merged at revision: | 157 |
Proposed branch: | lp:~gesha/linaro-license-protection/remove-artifacts-from-upload |
Merge into: | lp:~linaro-automation/linaro-license-protection/trunk |
Diff against target: |
35 lines (+7/-3) 2 files modified
scripts/publish_to_snapshots.py (+5/-1) tests/test_publish_to_snapshots.py (+2/-2) |
To merge this branch: | bzr merge lp:~gesha/linaro-license-protection/remove-artifacts-from-upload |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paul Sokolovsky | Approve | ||
Review via email: mp+145580@code.launchpad.net |
Description of the change
If BUILD-INFO.txt is not present among artifacts being published, we don't publish them. To avoid possible artifacts leakage with this branch we remove unpublished artifacts from upload dir.
To post a comment you must log in.
<pfalcon> gesha: I wanted to ask before, what kind of error reporting this gets back to the user? Will jenkins job which does publishing for example will get something into its build log?
<pfalcon> gesha: because otherwise I may suggest for example instead of just removing all files and publishing nothing, publish a text file saying "nothing was published because of lack of BUILDINFO.txt, please add it"
<gesha> pfalcon, right now it shows only "BUILD-INFO.txt is not present in artifacts being published" but add info about nothing published is a good idea
<gesha> will add