Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-05-23
14:13:23 sean-k-mooney they did not say what version of openstack they were using but im guessign it was quite an old version if they are seeing placment check the source allocations like they describe
14:14:58 sean-k-mooney i dont think any supported branch including the ones in EM still use a signel allocation for both hosts
14:15:15 dansmith yeah I'm sure not
14:15:41 sean-k-mooney we will see if they respond i guess
14:30:32 opendevreview ribaudr proposed openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506
15:14:46 kashyap melwitt: When you're about, can you review this, please: https://review.opendev.org/c/openstack/nova/+/838926 (It already has a +2 from gibi)
15:15:12 kashyap (And it is successfully tested by a Red Hat contributor downstream)
15:18:16 kashyap bauzas: --^
15:42:05 melwitt bauzas, elodilles, gmann: I started an etherpad for stable branch ci here. will fill it in with more info https://etherpad.opendev.org/p/nova-stable-branch-ci
15:43:08 melwitt kashyap: thanks for noticing :)
15:44:39 melwitt gibi, sean-k-mooney: yeah I thought stephenfin may want to look at the current spec bc he had a lot of feedback in previous PS
15:44:54 stephenfin oh, I'm looking at it at the moment
15:44:56 kashyap melwitt: I recall debugging it with you in the past; thanks for bringing it to its logical conclusion
15:45:17 kashyap Looks good to me to workaround it. (Also, the CI failure seems unrelated)
15:45:20 melwitt stephenfin: great :)
15:46:17 melwitt kashyap: will also review your patch
15:46:29 kashyap melwitt: It's also a workaround :D
15:46:36 melwitt haha nice
15:46:53 elodilles melwitt: thanks \o/ it was on my TODO as well o:)
15:47:51 kashyap melwitt: But it is functionally tested by a Red Hat support eng downstream
15:48:01 kashyap (And it works)
15:48:06 gmann melwitt: ack, thanks.
15:48:21 melwitt kashyap: ack
15:49:35 kashyap Thank you
15:54:18 elodilles melwitt: i proposed quite many bandit capping patches in the past, but did not notice that placement needs it :-o ( https://review.opendev.org/q/topic:cap-bandit-1.6.2 )
15:56:37 melwitt elodilles: yeah I remembered that being a thing and I think I found your patches for other projects. placement had been working ok for awhile, not actually sure when this cropped up. backports don't get proposed there often
15:58:10 bauzas Uggla: made a review with a French link on https://review.opendev.org/c/openstack/nova-specs/+/831506/
15:58:35 bauzas either way, +2d
15:58:52 bauzas cores, a spec seems very close to be accepted https://review.opendev.org/c/openstack/nova-specs/+/831506/4.55
15:58:58 bauzas cores, a spec seems very close to be accepted https://review.opendev.org/c/openstack/nova-specs/+/831506/4..5 even
16:00:36 elodilles melwitt: yeah, and we have periodic-stable-jobs in placement only since xena: https://review.opendev.org/c/openstack/placement/+/775384
16:01:44 stephenfin gibi: sean-k-mooney: melwitt: My only comment is on the config option. 'device_list' suggests it's a list but it's not - it's a MultiStrOpt (meaning you'd simply specify it multiple times in nova.conf)
16:02:13 melwitt elodilles: ah right
16:02:14 elodilles melwitt: we have the periodic-weekly for master, but maybe this worth a backport if we want to catch broken gates on older stable branches
16:02:14 gmann bauzas: FYI, i will be absent in nova meeting tomorrow as we have 'release naming meeting' at the same time - http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028618.html
16:02:19 stephenfin We can bikeshed on the correct name for that now or later, I don't mind. So long as we don't call it a plural thing when it's not actually plural
16:04:25 gibi stephenfin: hm
16:04:33 sean-k-mooney stephenfin: well eventully we shoudl make it a list instad of a multi opt
16:04:53 sean-k-mooney stephenfin: the alisa support json list syntax or mutlti opt repating
16:05:25 sean-k-mooney stephenfin: generally it enables multipel things by the way
16:05:47 sean-k-mooney it supprots regex and glob mathch or you can just use vender id and product id
16:05:48 gibi stephenfin: we can drop the _list postfix and simply call it [pci]devices
16:06:06 sean-k-mooney ya that would work
16:12:02 sean-k-mooney gibi: stephenfin if ye want to s/device_list/devices/ im fine with that
16:12:14 sean-k-mooney and or do it as a followup
16:12:42 gibi as I see stephenfin has things against the pluralness as well
16:13:04 gibi anyhow I will reply in the spec and we can discuss it tomorrow over IRC
16:13:32 sean-k-mooney well from my point of view it is plural as it generally refers to multiple devices via the vendor/product id or the regex support
16:13:39 sean-k-mooney sure
16:14:02 sean-k-mooney elodilles: by the way i propsoed https://review.opendev.org/c/openstack/requirements/+/842991 to test if that will help with stable/ussuri
16:14:39 sean-k-mooney elodilles: i think that woudl pull in most of the wait for sshabel changes but certenly not all
16:15:23 sean-k-mooney there are some other idea we had of how to impove the stabel branch situation but that was the lowest effort quick change we cam up with
16:16:38 sean-k-mooney ah actully i might need older
16:16:51 sean-k-mooney we might be hitting py36 issues witht hat
16:17:24 sean-k-mooney although we should supprot 36 on yoga so proably somethign else
16:18:08 sean-k-mooney ah oslo_log dependices
16:19:30 sean-k-mooney hum why is that using upper constratis form master
16:19:57 sean-k-mooney https://zuul.opendev.org/t/openstack/build/1f08c1134a1540bb8e9ae836c5ba60a9/log/job-output.txt#20845 ERROR: Could not find a version that satisfies the requirement oslo.log===5.0.0 (from -c https://releases.openstack.org/constraints/upper/master
16:20:21 bauzas gmann: ack no worries
16:21:06 sean-k-mooney ok i need to also change devstack to work aroudn that
16:26:04 gibi dansmith: could you check the unshelve to host spec https://review.opendev.org/c/openstack/nova-specs/+/831506/ I think now it is also aligned to you comments too
16:27:02 dansmith oh yeah I saw this ack
16:27:05 dansmith oops
16:27:33 dansmith that was intended to be two statements to two different people, but it worked out this time :)
16:28:19 sean-k-mooney oh i have debated if that was useful in the past
16:28:39 sean-k-mooney i sometimes start replyint in the wrong tab if misclick
16:28:49 sean-k-mooney but more often then not i assuem that causes issues
16:29:18 gibi :)
16:29:37 dansmith I'm not a fan.. end up with too many "that guy is an ass okay sure buddy" things where you don't remember to hit send in a /query and then switch
16:33:01 sean-k-mooney https://review.opendev.org/c/openstack/requirements/+/842991 is going to fail but even using an old upper-constriats i think wont work
16:33:13 sean-k-mooney its been resolving for ~5 mins locally
16:34:13 sean-k-mooney realistically if i want to bump the tempest version i think i would need to either do this differently via devstack or bump other constraits
16:41:22 sean-k-mooney actully the best apptoch might be to just not use an upper-constratits file for the venv
16:44:03 dansmith gibi: I have one typo gripe plus bauzas
16:44:12 dansmith can we fix and re-ack real quick?
16:44:23 dansmith I can edit inline and then +2 if you'll be around to +W
16:44:47 bauzas dansmith: sure
16:45:14 opendevreview Dan Smith proposed openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506
16:45:22 bauzas gibi: I'll have comments on the PCI spec, mostly on the config options
16:45:22 sean-k-mooney ok i think the best way to make this work is just to remove using upper-constraits from the venv in devstack ill try that and see if that work
16:45:30 dansmith bauzas: gibi done
16:45:35 bauzas dansmith: saw
16:46:30 bauzas Uggla: typey typey now
16:46:44 Uggla ?
16:53:01 bauzas Uggla: your spec got approved
16:53:08 bauzas which means "hands on deck now"
16:53:15 Uggla oh cool
16:59:18 gibi dansmith, bauzas: thanks
17:04:23 sean-k-mooney gmann: o/ hi looks like you are also tryinng to use a newer version of tempest on ussuri
17:04:25 sean-k-mooney https://review.opendev.org/c/openstack/devstack/+/838051
17:04:55 sean-k-mooney im trying to do the same with https://review.opendev.org/c/openstack/requirements/+/842991 andhttps://review.opendev.org/c/openstack/devstack/+/843006
17:07:06 sean-k-mooney ok im going to call it a day but ill take a look at those again tomorrow
17:14:03 opendevreview Merged openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506
17:16:15 melwitt sean-k-mooney: have you seen this re: upper-constraints? https://review.opendev.org/c/openstack/nova/+/842813
17:16:46 melwitt oh, nvm, I guess that's what you just talking about the last few messages
17:18:02 sean-k-mooney no i had not
17:18:16 sean-k-mooney but i dont think we want to do that
17:18:52 sean-k-mooney i dont really agree with pinnint tempest but if we do im not conviced we shoudl do it via devstack
17:19:37 sean-k-mooney we could but i think we need to contol that form the project side unless the QA team want to support all the brances we care about including the EM ones
17:20:32 melwitt it's a nova patch

Earlier   Later