*** tosky has quit IRC | 00:05 | |
*** vishalmanchanda has joined #openstack-requirements | 03:32 | |
*** udesale has joined #openstack-requirements | 03:55 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-requirements | 05:33 | |
ricolin | We need https://review.opendev.org/#/c/701105/ back again to fix gate job for multiple projects | 06:00 |
---|---|---|
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/701104 | 06:17 |
*** brinzhang has joined #openstack-requirements | 07:04 | |
*** brinzhang_ has joined #openstack-requirements | 07:07 | |
*** brinzhang has quit IRC | 07:10 | |
*** brinzhang_ has quit IRC | 07:38 | |
*** hberaud has joined #openstack-requirements | 08:35 | |
*** e0ne has joined #openstack-requirements | 08:41 | |
*** ralonsoh has joined #openstack-requirements | 08:52 | |
*** e0ne has quit IRC | 09:26 | |
*** e0ne has joined #openstack-requirements | 10:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.config to new release 7.0.0 https://review.opendev.org/701188 | 10:29 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/stein: update constraint for os-brick to new release 2.8.3 https://review.opendev.org/701189 | 10:31 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/train: update constraint for os-apply-config to new release 10.5.1 https://review.opendev.org/701190 | 10:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.i18n to new release 3.25.1 https://review.opendev.org/701191 | 10:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/train: update constraint for os-brick to new release 2.10.1 https://review.opendev.org/701192 | 10:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.log to new release 3.45.2 https://review.opendev.org/701193 | 10:34 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/train: update constraint for paunch to new release 5.3.1 https://review.opendev.org/701194 | 10:37 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/train: update constraint for os-net-config to new release 11.3.1 https://review.opendev.org/701195 | 10:38 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslotest to new release 3.9.0 https://review.opendev.org/701196 | 10:39 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.db to new release 6.0.0 https://review.opendev.org/701197 | 10:39 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for keystonemiddleware to new release 8.0.0 https://review.opendev.org/701198 | 10:42 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/train: update constraint for os-collect-config to new release 10.5.1 https://review.opendev.org/701199 | 10:43 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/train: update constraint for tripleo-common to new release 11.3.2 https://review.opendev.org/701200 | 10:43 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.messaging to new release 10.5.0 https://review.opendev.org/701201 | 10:43 |
*** hberaud is now known as hberaud|lunch | 10:44 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements stable/rocky: update constraint for os-brick to new release 2.5.9 https://review.opendev.org/701202 | 10:47 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for oslo.versionedobjects to new release 1.37.0 https://review.opendev.org/701203 | 10:56 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for tooz to new release 1.67.2 https://review.opendev.org/701205 | 11:08 |
*** udesale has quit IRC | 11:16 | |
*** e0ne has quit IRC | 11:18 | |
*** e0ne has joined #openstack-requirements | 11:57 | |
*** hberaud|lunch is now known as hberaud | 12:01 | |
openstackgerrit | Akihiro Motoki proposed openstack/requirements master: Cap Django again https://review.opendev.org/701213 | 12:39 |
*** tosky has joined #openstack-requirements | 14:01 | |
*** bnemec has joined #openstack-requirements | 14:28 | |
KeithMnemonic | prometheanfire: i think i found another constraint/requirements issue. can you help decipher this https://zuul.opendev.org/t/openstack/build/e3e57f71fea4489ab614e6b174ea5c26/log/job-output.txt#1959 | 15:19 |
prometheanfire | KeithMnemonic: fun thing, my terminal likes to break when I paste that line :D | 15:31 |
prometheanfire | github firefox plugin too, huh | 15:32 |
prometheanfire | https://gist.github.com/prometheanfire/a9bde5e05e62cdee7efe7a06a07fec7d | 15:32 |
prometheanfire | heh, coredump'd firefox | 15:34 |
KeithMnemonic | yeah it is a pretty big line, i used a json viewer | 15:35 |
prometheanfire | https://gist.githubusercontent.com/prometheanfire/a9bde5e05e62cdee7efe7a06a07fec7d/raw/0615f912601c76d9bdae7f80fd1be0758af36e15/gistfile1.txt doesn't look right, too much? | 15:39 |
prometheanfire | https://gist.github.com/6a0ab66ae0d894ece2367884bb0e9efb is more readable | 15:43 |
prometheanfire | ContextualVersionConflict: (osc-lib 1.11.1 (/home/zuul/src/opendev.org/openstack/openstackclient/osc_plugins/lib/python2.7/site-packages), Requirement.parse(\'osc-lib>=1.14.0\'), set([\'python-openstackclient\'])) | 15:43 |
prometheanfire | VersionConflict: (osc-lib 1.11.1 (/home/zuul/src/opendev.org/openstack/openstackclient/osc_plugins/lib/python2.7/site-packages), Requirement.parse(\'osc-lib>=1.14.0\')) | 15:43 |
prometheanfire | KeithMnemonic: master ir looks like? | 15:45 |
KeithMnemonic | so is this the same thing we need a contstraint on osc-lib for stable/rocky? | 15:46 |
prometheanfire | is this stable/rocky? | 16:01 |
KeithMnemonic | yes that patch is for rocky | 16:08 |
*** bnemec has quit IRC | 16:10 | |
ricolin | Any one around to approve https://review.opendev.org/#/c/701105/ again:) | 16:19 |
ricolin | smcginnis, ^^^ | 16:25 |
smcginnis | ricolin: Got it | 16:27 |
*** e0ne has quit IRC | 16:32 | |
KeithMnemonic | prometheanfire: efried did a quick analysis in openstack-sdks if you want to read the thread | 16:38 |
KeithMnemonic | also any help on this patch from amotoki is appreciated. it is blocking a designate-dashboard patch currently https://review.opendev.org/#/c/700377/ | 16:44 |
*** tosky has quit IRC | 16:46 | |
prometheanfire | KeithMnemonic: it looks like something is not sync'd with requirements (osc_plugins it looks like is not in upper-constraints.txt or in global-requirements.txt | 16:46 |
KeithMnemonic | so does it need a series of patches from master down to add it? | 16:49 |
prometheanfire | probably, it looks like the project never added requirements syncing | 16:57 |
KeithMnemonic | ok so what is the best way to drive this? i can submit the patches if i know what versions to lock the release to | 17:04 |
*** nicolasbock has joined #openstack-requirements | 17:12 | |
*** evrardjp has quit IRC | 17:33 | |
*** evrardjp has joined #openstack-requirements | 17:33 | |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.i18n to new release 3.25.1 https://review.opendev.org/701191 | 18:05 |
*** nicolasbock has quit IRC | 18:15 | |
*** ralonsoh has quit IRC | 18:29 | |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.log to new release 3.45.2 https://review.opendev.org/701193 | 18:31 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslotest to new release 3.9.0 https://review.opendev.org/701196 | 18:31 |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.db to new release 6.0.0 https://review.opendev.org/701197 | 18:31 |
*** KeithMnemonic_ has joined #openstack-requirements | 18:50 | |
*** hberaud is now known as hberaud|gone | 19:05 | |
*** adriant has quit IRC | 19:24 | |
*** e0ne has joined #openstack-requirements | 19:28 | |
*** adriant has joined #openstack-requirements | 19:29 | |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.messaging to new release 10.5.0 https://review.opendev.org/701201 | 19:32 |
*** e0ne has quit IRC | 19:43 | |
*** bnemec has joined #openstack-requirements | 19:47 | |
openstackgerrit | Merged openstack/requirements master: update constraint for oslo.versionedobjects to new release 1.37.0 https://review.opendev.org/701203 | 19:50 |
openstackgerrit | Merged openstack/requirements master: update constraint for tooz to new release 1.67.2 https://review.opendev.org/701205 | 19:50 |
openstackgerrit | Merged openstack/requirements master: Cap Django again https://review.opendev.org/701213 | 19:50 |
openstackgerrit | Merged openstack/requirements master: confluent-kafka 1.3.0 fails to install https://review.opendev.org/701105 | 19:50 |
*** KeithMnemonic_ has quit IRC | 20:50 | |
*** tosky has joined #openstack-requirements | 20:55 | |
*** zbr has quit IRC | 21:05 | |
*** zbr has joined #openstack-requirements | 21:07 | |
*** zbr has quit IRC | 21:14 | |
*** zbr has joined #openstack-requirements | 21:22 | |
*** zbr has quit IRC | 21:26 | |
*** zbr has joined #openstack-requirements | 21:30 | |
*** zbr_ has joined #openstack-requirements | 21:37 | |
openstackgerrit | Keith Berger proposed openstack/requirements stable/train: Constraint openstackclient for stable branch https://review.opendev.org/701285 | 21:39 |
*** zbr has quit IRC | 21:39 | |
*** zbr_ has quit IRC | 22:02 | |
KeithMnemonic | smcginnis, i can do a master branch patch for the openstack requirements first if that is what is best. i was not sure if this was similar to the patches amotoki did where it was not correct to do on master | 22:02 |
KeithMnemonic | https://review.opendev.org/700377 this one for example | 22:02 |
*** zbr has joined #openstack-requirements | 22:02 | |
smcginnis | KeithMnemonic: Who is using openstackclient? | 22:20 |
KeithMnemonic | smcginnis: the failure is coming here https://review.opendev.org/701066. something is pulling in a newer python-openstackclient. | 22:21 |
smcginnis | If it's actually needed and used, it should be added as a new requirement that we track in global-requirements and upper-constraints just like if we were to add any other dependency. | 22:21 |
smcginnis | What about openstackclient though? Or are you accidentally mixing the two up? | 22:21 |
smcginnis | Because the naming for them is so clear and obvious. :eye roll: | 22:21 |
KeithMnemonic | no, gyee helped me look into this and so did efried. something is out of whack. i do not think anything will pass on stable/rocky ATM | 22:22 |
KeithMnemonic | my patch did not touch any of that | 22:22 |
smcginnis | That looks like osc-lib is the issue on that patch you referenced. | 22:22 |
KeithMnemonic | correct, but whati s pulling in 1.14? | 22:22 |
KeithMnemonic | it should not be pulled in | 22:23 |
smcginnis | Definitely not openstackclient. | 22:23 |
KeithMnemonic | but openstack client pulls python-openstackclient>=4.0.0 # Apache-2.0 | 22:24 |
KeithMnemonic | https://github.com/openstack/openstackclient/blob/master/requirements.txt#L19 | 22:24 |
KeithMnemonic | vs https://github.com/openstack/openstackclient/blob/0.1.0/requirements.txt#L17 | 22:25 |
*** zbr has quit IRC | 22:25 | |
smcginnis | Yeah, but nothing there is pulling in openstackclient, so it's kind of a moot point. | 22:25 |
KeithMnemonic | something is pulling in python-openstackclient=4.0.0 | 22:25 |
KeithMnemonic | i see it in the logs | 22:26 |
smcginnis | Which also has nothing to do with openstackclient, so... | 22:26 |
KeithMnemonic | openstackclient has a requirement of in python-openstackclient=4.0.0 | 22:27 |
KeithMnemonic | we see in the zuul log it getting openstackclient from master | 22:28 |
smcginnis | That sounds backwards. Openstackclient is a metapackage to pull in python-openstackclient and common plugins. So some kind of circular reference? | 22:28 |
KeithMnemonic | whhttps://zuul.opendev.org/t/openstack/build/e3e57f71fea4489ab614e6b174ea5c26/log/job-output.txt#174-185 | 22:29 |
KeithMnemonic | correct, but since it uses openstackclient from master (4.0). that pulls in https://opendev.org/openstack/python-openstackclient/src/branch/master/requirements.txt | 22:30 |
KeithMnemonic | which has osc-lib>=1.14.0 # Apache-2.0 | 22:30 |
smcginnis | We need to track down which of its direct dependencies is incorrectly pulling in openstackclient, because no project should be using that directly. | 22:31 |
smcginnis | The issue is not capping an upper-constraint, it's finding out where things are being mishandled. | 22:31 |
smcginnis | It's not in python-openstackclient's requirements.txt, test-requirements.txt, or steup.cfg. | 22:31 |
KeithMnemonic | ok, but for know merging to openstackclient is broken, at least on rocky and probably stein/train | 22:32 |
smcginnis | So still looking to find who/where it's getting grabged from. | 22:32 |
smcginnis | openstackclient is broken too? | 22:32 |
KeithMnemonic | sorry that time i confused them ;-) python-openstackclient | 22:32 |
smcginnis | OK, good, one problem at a time. :) | 22:32 |
smcginnis | KeithMnemonic: The job is invalid. | 22:36 |
KeithMnemonic | the check plugin job? | 22:37 |
smcginnis | Looks like this probably needs to be constrained by branch - https://opendev.org/openstack/openstackclient/src/branch/master/.zuul.yaml#L3 | 22:37 |
smcginnis | Still looking though. | 22:37 |
KeithMnemonic | this was the last patch "Merge "Revert "check-plugins job: convert the job to tox""" | 22:41 |
KeithMnemonic | seems very suspicious | 22:41 |
smcginnis | So the check-plugins job is part of the openstackclient-plugin-jobs template. | 22:42 |
smcginnis | And that template is in the python-openstackclient .zuul.yaml https://github.com/openstack/python-openstackclient/blob/stable/rocky/.zuul.yaml#L193 | 22:42 |
smcginnis | So either the job needs to be constrained to not run on older branches, or the job definition needs to account for correct handling of that. | 22:43 |
smcginnis | I don't know enough about defining zuul jobs to know the right answer there. | 22:43 |
KeithMnemonic | is that an infra question then? | 22:43 |
smcginnis | But the issue is definitely the job and the fact that it pulls in master to do it's thing, not that we need an upper-constraint for something that is not tracked as a requirement. | 22:43 |
smcginnis | Yeah, probably an infra question, unfortunately. | 22:44 |
KeithMnemonic | would it be too much to ask you to post a nice summary in openstack-infra for this? | 22:44 |
KeithMnemonic | otherwise i will just be the middle man | 22:45 |
KeithMnemonic | thanks i see it now | 22:46 |
smcginnis | Yeah, let's see what they say. | 22:49 |
smcginnis | I'm not even the middle man, I'm just the innocent bystander. :D | 22:49 |
smcginnis | KeithMnemonic: Pretty sure https://review.opendev.org/#/c/701293/ is the right solution. | 22:53 |
smcginnis | If I get the syntax right... | 22:53 |
KeithMnemonic | thanks, i can abandon my patch now | 22:56 |
*** zbr has joined #openstack-requirements | 23:04 | |
*** zbr has quit IRC | 23:09 | |
*** zbr_ has joined #openstack-requirements | 23:09 | |
*** zbr_ has quit IRC | 23:14 | |
KeithMnemonic | it passed now so just need some reviews. is it worth looking at the nv failure in my free time? i will have some more python-openstackclient patches coming to some non master brances | 23:16 |
*** zbr has joined #openstack-requirements | 23:21 | |
*** tosky has quit IRC | 23:23 | |
openstackgerrit | Merged openstack/requirements stable/train: update constraint for tripleo-common to new release 11.3.2 https://review.opendev.org/701200 | 23:51 |
openstackgerrit | Merged openstack/requirements stable/rocky: update constraint for os-brick to new release 2.5.9 https://review.opendev.org/701202 | 23:51 |
openstackgerrit | Merged openstack/requirements stable/stein: update constraint for os-brick to new release 2.8.3 https://review.opendev.org/701189 | 23:51 |
openstackgerrit | Merged openstack/requirements stable/train: update constraint for os-apply-config to new release 10.5.1 https://review.opendev.org/701190 | 23:51 |
openstackgerrit | Merged openstack/requirements stable/train: update constraint for os-net-config to new release 11.3.1 https://review.opendev.org/701195 | 23:51 |
openstackgerrit | Merged openstack/requirements stable/train: update constraint for os-collect-config to new release 10.5.1 https://review.opendev.org/701199 | 23:51 |
openstackgerrit | Merged openstack/requirements master: update constraint for keystonemiddleware to new release 8.0.0 https://review.opendev.org/701198 | 23:51 |
*** KeithMnemonic has quit IRC | 23:56 | |
*** KeithMnemonic has joined #openstack-requirements | 23:57 | |
*** KeithMnemonic has quit IRC | 23:58 | |
*** KeithMnemonic has joined #openstack-requirements | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!