added bodhi reporting to upgradepath.yml
ClosedPublic

Authored by roshi on May 28 2014, 9:28 PM.

Details

Summary
Added bodhi reporting to upgradepath.yml
Removed trailing comma from .arcconfig
Test Plan
Run task in staging

Diff Detail

Repository
rUPGR task-upgradepath
Lint
Lint Skipped
Unit
Unit Tests Skipped
roshi retitled this revision from to added bodhi reporting to upgradepath.yml.May 28 2014, 9:28 PM
roshi updated this object.
roshi edited the test plan for this revision. (Show Details)
roshi added a reviewer: tflink.
roshi updated this revision to Diff 314.May 28 2014, 9:30 PM
  • fixed commented out section

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?

roshi added a comment.May 29 2014, 2:37 PM

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.

tflink requested changes to this revision.May 29 2014, 2:44 PM
I wasn't sure what the production usage would be

It should be 'onchange' for production

upgradepath.yml
31

This needs to be 'onchange'

This revision now requires changes to proceed.May 29 2014, 2:44 PM
roshi updated this revision to Diff 325.May 29 2014, 3:45 PM
  • fixed 'doreport' to be 'onchange' for bodhi reporting.
kparal accepted this revision.EditedMay 30 2014, 12:26 PM

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

tflink accepted this revision.May 30 2014, 3:54 PM

This looks good to me. I thought that the revision was already accepted but adding another +1

This revision is now accepted and ready to land.May 30 2014, 3:54 PM
roshi closed this revision.May 30 2014, 4:40 PM
roshi updated this revision to Diff 331.

Closed by commit rUPGR89fc6a5626d7 (authored by @roshi).

In D111#16, @tflink wrote:

@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

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.