releasetasks_graph_gen.py should use release-runner.yml, not ini

RESOLVED FIXED

Status

Release Engineering
Release Automation: Other
P1
normal
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: rail, Assigned: rail)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [releaseduty])

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Assignee)

Description

11 months ago
We stopped updating the ini file and switched to yml in release runner, but not in releasetasks_graph_gen.py
Comment hidden (mozreview-request)
(Assignee)

Comment 2

11 months ago
Created attachment 8903109 [details] [review]
PR for releasewarrior
Attachment #8903109 - Flags: review?(mtabara)
(Assignee)

Comment 3

11 months ago
It worked fine on bm83
(Assignee)

Updated

11 months ago
See Also: → bug 1347635

Comment 4

11 months ago
mozreview-review
Comment on attachment 8903107 [details]
Bug 1395516 - releasetasks_graph_gen.py should use release-runner.yml, not ini

https://reviewboard.mozilla.org/r/174902/#review179958

We also need to remove references from puppet.

* to delete this file https://hg.mozilla.org/build/puppet/file/tip/modules/releaserunner/templates/release-runner.ini.erb
* to remove puppetization from https://hg.mozilla.org/build/puppet/file/tip/modules/releaserunner/manifests/init.pp#l67
Attachment #8903107 - Flags: review?(mtabara) → review+
Attachment #8903109 - Flags: review?(mtabara) → review+
Whiteboard: [releaseduty]
(Assignee)

Comment 5

11 months ago
(In reply to Mihai Tabara [:mtabara]⌚️GMT from comment #4)
> Comment on attachment 8903107 [details]
> Bug 1395516 - releasetasks_graph_gen.py should use release-runner.yml, not
> ini
> 
> https://reviewboard.mozilla.org/r/174902/#review179958
> 
> We also need to remove references from puppet.
> 
> * to delete this file
> https://hg.mozilla.org/build/puppet/file/tip/modules/releaserunner/templates/
> release-runner.ini.erb
> * to remove puppetization from
> https://hg.mozilla.org/build/puppet/file/tip/modules/releaserunner/manifests/
> init.pp#l67

We still use those for "prod-old": https://hg.mozilla.org/build/puppet/file/tip/manifests/moco-config.pp#l291. I'll leave it there as is for now.
(Assignee)

Comment 6

11 months ago
Comment on attachment 8903107 [details]
Bug 1395516 - releasetasks_graph_gen.py should use release-runner.yml, not ini

https://hg.mozilla.org/build/tools/rev/8d9a5ba92c39f5eeaff96ea0acd2c5b240f6cbf8
Attachment #8903107 - Flags: checked-in+
(Assignee)

Updated

11 months ago
Attachment #8903109 - Flags: checked-in+
(Assignee)

Comment 7

11 months ago
Deployed
Status: NEW → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED
(In reply to Rail Aliiev [:rail] ⌚️UTC+3 from comment #5)
> We still use those for "prod-old":
> https://hg.mozilla.org/build/puppet/file/tip/manifests/moco-config.pp#l291.
> I'll leave it there as is for now.

You're absolutely right, mea culpa - I rushed into conclusions!
You need to log in before you can comment on or make changes to this bug.