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

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, enhancement, P1)

Description

[Rail Aliiev [:rail]]

We stopped updating the ini file and switched to yml in release runner, but not in releasetasks_graph_gen.py

Comment 1

[Rail Aliiev [:rail]]

Attachment: Bug 1395516 - releasetasks_graph_gen.py should use release-runner.yml, not ini

Review commit: https://reviewboard.mozilla.org/r/174902/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/174902/

Comment 2

[Rail Aliiev [:rail]]

Attachment: PR for releasewarrior

Comment 3

[Rail Aliiev [:rail]]

It worked fine on bm83

Comment 4

[Mihai Tabara [:mtabara]⌚️GMT]

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

Comment 5

[Rail Aliiev [:rail]]

(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.

Comment 6

[Rail Aliiev [:rail]]

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

Comment 7

[Rail Aliiev [:rail]]

Deployed

Comment 8

[Mihai Tabara [:mtabara]⌚️GMT]

(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!