Closed
Bug 792201
Opened 13 years ago
Closed 13 years ago
move pushed snippets to a different directory
Categories
(Release Engineering :: Release Automation, defect)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: mozilla)
References
Details
Attachments
(3 files)
|
1.21 KB,
patch
|
nthomas
:
review+
|
Details | Diff | Splinter Review |
|
2.13 KB,
patch
|
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
|
841 bytes,
patch
|
bhearsum
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
When pushsnip finishes running successfully it should move the pushed directory somewhere else (a 'pushed' subdirectory? /opt/aus2/snippets/pushed?). This would do two things:
1) Help unclutter /opt/aus2/snippets/staging
2) Help avoid accidentally repushing snippets that were already pushed
|
Assignee | |
Comment 1•13 years ago
|
||
Should we put the .time files there too, or leave them?
Attachment #662303 -
Flags: review?(bhearsum)
|
|
Assignee | |
Updated•13 years ago
|
Attachment #662303 -
Flags: review?(bhearsum) → review?(nthomas)
|
||
Comment 2•13 years ago
|
||
Comment on attachment 662303 [details] [diff] [review]
move pushed dir
Looks fine. I'd say it makes sense to put the .time files in PUSHED_DIR too, feel free to make that change on checkin.
Attachment #662303 -
Flags: review?(nthomas) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
Checking in pushsnip;
/cvsroot/mozilla/tools/release/bin/pushsnip,v <-- pushsnip
new revision: 1.9; previous revision: 1.8
done
Attachment #662345 -
Flags: checked-in+
|
|
Assignee | |
Comment 4•13 years ago
|
||
Updated aus3-staging:~ffxbld/bin .
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
Reporter | |
Comment 5•13 years ago
|
||
Unfortunately, this broke pushsnip:
446 100% 5.97kB/s 0:00:00 (xfer#40060, to-check=0/97678)
sent 22957641 bytes received 822880 bytes 26003.85 bytes/sec
total size is 18841404 speedup is 0.79
mv: cannot stat `Firefox-16.0b4-build1-test': No such file or directory
Wrong cwd / should be using full path, maybe?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
|
Reporter | |
Updated•13 years ago
|
Assignee: nobody → aki
|
|
Assignee | |
Comment 6•13 years ago
|
||
Attachment #663053 -
Flags: review?(bhearsum)
|
|
Reporter | |
Updated•13 years ago
|
Attachment #663053 -
Flags: review?(bhearsum) → review+
|
|
Assignee | |
Comment 7•13 years ago
|
||
Comment on attachment 663053 [details] [diff] [review]
fix pushsnip
Checking in pushsnip;
/cvsroot/mozilla/tools/release/bin/pushsnip,v <-- pushsnip
new revision: 1.10; previous revision: 1.9
done
Attachment #663053 -
Flags: checked-in+
|
|
Assignee | |
Updated•13 years ago
|
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 8•13 years ago
|
||
Updated aus3-staging:~ffxbld/bin after cvs-mirror updated.
|
|
Reporter | |
Comment 9•13 years ago
|
||
I moved all of the already pushed directory from staging/ into pushed/. Hooray for having a clean staging/ directory!
|
|
Reporter | |
Comment 10•13 years ago
|
||
Worked fine for the beta channel push. Thanks!
|
||
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•