Added bodhi reporting to upgradepath.yml Removed trailing comma from .arcconfig
Details
- Reviewers
kparal tflink - Maniphest Tasks
- T172: enable bodhi reporting in rpmlint and upgradepath
- Commits
- rUPGR89fc6a5626d7: added bodhi reporting to upgradepath.yml
Run task in staging
Diff Detail
- Repository
- rUPGR task-upgradepath
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Thanks, I fixed the comma and pushed right away, because it broke the arc command.
Regarding Bodhi comments, @tflink says we're going to be using fake_fedorainfra until we turn off autoqa, so that sounds good. But why does it have doreport: 'all'? Do I understand it correctly that it will add a comment for every time that update is checked, i.e. easily 10 times per day? Is that for debugging purposes? Maybe we should rather use the same setting we will use for the production system and real Bodhi?
I had it set to 'all' while working on it. I wasn't sure what the production usage would be so I just left it alone. It should be switched to whatever we would use in the real instance.
I wasn't sure what the production usage would be
It should be 'onchange' for production
upgradepath.yml | ||
---|---|---|
31 | This needs to be 'onchange' |
Looks good, please push.
@tflink, can we have a link to the staging fake_fedorainfra on https://taskotron-stg.fedoraproject.org/ ? It would be great to have a simple way to access the fake results and make sure everything works correctly.
@tflink, can we have a link to the staging fake_fedorainfra on https://taskotron-stg.fedoraproject.org/
I had the playbook written but hadn't actually deployed it yet.
https://taskotron-stg.fedoraproject.org/fakefedorainfra/
bodhi url should be: http://taskotron-stg.fedoraproject.org/fakefedorainfra/bodhi
This looks good to me. I thought that the revision was already accepted but adding another +1
I actually meant adding a new hyperlink to https://taskotron-stg.fedoraproject.org page that would link to the fake_fedorainfra interface. But I assume we would have to remove the "reset db" button.
This needs to be 'onchange'