tox.ini settings for global constraints are out of date

Bug #1745471 reported by Doug Hellmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
placement-osc-plugin
Fix Released
Low
Ameed Ashour

Bug Description

The way the constraints list is managed via tox is out of date for osc-placement. The requirements team can help with an example of how it should be done now.

Ameed Ashour (ameeda)
Changed in placement-osc-plugin:
assignee: nobody → Ameed Ashour (ameeda)
Revision history for this message
Ameed Ashour (ameeda) wrote :

should I change tox.ini as this one https://github.com/openstack/nova/blob/master/tox.ini ?

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

Please talk to the requirements team about exactly how this needs to be set up for your project, either via the openstack-dev list or #openstack-requirements on IRC.

Revision history for this message
Ameed Ashour (ameeda) wrote :

Hi Doug,

Thanks for reply. I contact the mentioned channels.
I will upload patch when they respond.

Regards

Revision history for this message
Ameed Ashour (ameeda) wrote :

Hi Doug,

I contacted with prometheanfire on #openstack-requirements.

[22:52] <prometheanfire> ameeda: I'm not sure I see what doug sees, so I don't know what's wrong

please check this out to see the log.
http://paste.openstack.org/show/666964/

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

Compare the tox.ini settings in the orc-placement repo (http://git.openstack.org/cgit/openstack/osc-placement/tree/tox.ini) to the ones in oslo.config (http://git.openstack.org/cgit/openstack/oslo.config/tree/tox.ini).

Note that the install_command and deps entries are different.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

Sorry, osc-placement not orc.

Revision history for this message
Ameed Ashour (ameeda) wrote :

Hi Doug,

Thanks for reply, I will check that with him.

Regards
Ameed

Ameed Ashour (ameeda)
Changed in placement-osc-plugin:
status: New → In Progress
Revision history for this message
Ameed Ashour (ameeda) wrote :

Hi Doug,

based on IRC chat with prometheanfire, he told me to change the install_command and deps entries of orc-placement tox.ini to oslo.config tox.ini

IRC log: https://pasteboard.co/H7g7HuA.png

Fix proposed here
https://review.openstack.org/#/c/543348/

Changed in placement-osc-plugin:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/osc-placement 1.1.0

This issue was fixed in the openstack/osc-placement 1.1.0 release.

Matt Riedemann (mriedem)
Changed in placement-osc-plugin:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.