Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-01-25
20:02:08 openstackgerrit Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
20:02:08 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
20:02:09 openstackgerrit Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227
#openstack-nova - 2020-01-26
14:57:48 openstackgerrit Balazs Gibizer proposed openstack/nova stable/rocky: Mask the token used to allow access to consoles https://review.opendev.org/704255
15:02:13 openstackgerrit Balazs Gibizer proposed openstack/nova stable/rocky: Mask the token used to allow access to consoles https://review.opendev.org/704255
#openstack-nova - 2020-01-27
09:45:23 stephenfin bauzas: Today's a spec review day for me. Just reviewed https://review.opendev.org/#/c/702943/
09:49:00 openstackgerrit John Garbutt proposed openstack/nova-specs master: Small fixes to unified limits spec https://review.opendev.org/703773
09:59:44 bauzas stephenfin: ack, thanks
10:00:08 bauzas FWIW, I should do the same...
10:38:49 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.opendev.org/702943
10:50:47 stephenfin bauzas, efried: Think you folks could look at https://review.opendev.org/#/c/682655/ today? It's just a reproposal and I hope to have the mostly trivial code up later this week
11:17:20 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Additional upgrade clarifications for cpu-resources https://review.opendev.org/666032
11:30:10 openstackgerrit Merged openstack/nova-specs master: spec update: virtual persistent memory https://review.opendev.org/695868
11:53:03 stephenfin kashyap: Loads of comments left on https://review.opendev.org/#/c/693844/, when you have time
11:54:46 kashyap stephenfin: Will check, on the phone; thanks!
12:07:02 kashyap stephenfin: Thanks for taking time to review; reasonable questions, will respond there.
12:15:54 stephenfin huaqiang: Is it you that's working on bp/use-pcpu-and-vcpu-in-one-instance ? If so, just hit the spec at https://review.opendev.org/#/c/668656/
12:33:16 openstackgerrit Merged openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152
13:35:41 gibi stephenfin: replied in https://review.opendev.org/#/c/701763
14:00:57 gibi stephenfin: you were the second +2 on https://review.opendev.org/#/c/699669/ but did not +A it. Is it intentional?
14:11:44 openstackgerrit Merged openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.opendev.org/702943
14:18:16 efried stephenfin: Would you please make a blueprint for that?
14:35:30 openstackgerrit Vladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent https://review.opendev.org/704330
14:36:36 openstackgerrit Vladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent https://review.opendev.org/704330
14:50:03 efried lyarwood: I've been procrastinating on https://review.opendev.org/#/c/669121/ -- at some point would you mind directing me to the piece of the CI that shows it's doing what's expected?
14:51:41 efried oh, I guess it's not waiting on me, since there's an open dep. That makes me feel less slackerish.
14:55:33 lyarwood efried: yup I've been pinging away in openstack-qa about this for a while, I'll try to get it over the line today. The CI you're looking for is actually against that change as the dep is for an openstack/grenade change.
14:57:06 efried thanks
14:58:10 lyarwood gah .gz files, has zuul regressed recently in terms of being able to display these .gz log files as plain text via the web ui?
14:58:40 lyarwood or were these just plain text before and are now .gz?
14:59:16 lyarwood either way it's now impossible to share URLs to specific log lines etc
15:01:43 efried lyarwood: Yeah, infra changed a thing, something about swift, and now a few jobs, including legacy ones, are double-compressing log files. It's a serious PITA.
15:02:41 efried Apparently we can fix it somewhere in the job definition itself, but so far nobody has gone and done that. Not sure how much of that is because nobody knows how (that's certainly *my* excuse) or just haven't had the time.
15:03:16 efried Another hesitation with that is that we *should* just go convert those jobs to zv3, which would make the problem go away, and which we're supposed to be doing anyway.
15:03:43 efried that also has been not done, and for similar reasons I imagine.
15:03:55 efried smarter people than me have made attempts, but nothing has closed
15:04:31 lyarwood efried: ack thanks, I did start looking at switching the nova-lvm job over last week but didn't post anything.
15:04:54 lyarwood sean-k-mooney: have you started converting any of the grenade jobs to zuulv3?
15:05:04 lyarwood iirc you were looking at this right?
15:05:07 efried I will personally buy a cookie for whoever makes the .gz problem go away, I don't care how.
15:21:28 gmann lyarwood: grenade job zuulv3 work https://review.opendev.org/#/q/topic:grenade_zuulv3+status:open
15:22:32 lyarwood gmann: awesome thank you, I'll take a look shortly
15:50:10 lyarwood melwitt: https://review.opendev.org/#/c/703005/ - Did you have another openstack/nova change using https://review.opendev.org/#/c/548936/ btw?
16:02:32 melwitt lyarwood: yeah, I was dual purposing this https://review.opendev.org/701478
16:03:32 lyarwood melwitt: ah cool thanks
16:04:29 melwitt I got motivated to test out the grenade change while trying to get some info on that cell timeout gate failure in grenade jobs (bc the log files are all gzipped and annoying to have to download). zuul v3 conversion solves that problem
16:13:12 dansmith efried: just want to make sure it's clear that I'm waiting for you to ack this before I really do much with it: https://review.opendev.org/#/c/631243/52
16:14:38 efried dansmith: okay, thanks for the heads up.
16:15:03 dansmith efried: seems like you haven't been through that set lately, are you planning to go through it again soon?
16:15:18 efried I can do that, yes.
16:15:33 efried dansmith: I was actually mostly waiting for you to be happy with it.
16:15:46 dansmith efried: last time I did I got to like patch #4 and found yet many more sections of code where there wasn't any testing, and/or tests that asserted things were called, but not with any specific parameters and got annoyed
16:16:03 efried okay.
16:16:14 dansmith and so I'm feeling a little like I'm the only guy digging deep on those things, which is exhausting
16:24:38 efried I'll take a thorough swipe this week. You should take some time off to prevent burnout. Go do some sledding or something.
16:25:17 dansmith well, I'm just back from time off for general burn-out reasons,
16:25:18 dansmith but I also don
16:25:51 dansmith I also don't want to get too myopic on this set, so I think it would be good to have someone else take a pass through and see if they find things I'm not seeing
16:26:26 dansmith like, a little more than just my comments get changed each time and I don't want to settle into an itemized list of "yes, my comment was addressed" without looking close enough at the whole change
16:26:40 efried ack
16:37:58 lyarwood melwitt: sorry got slightly distracted, did you want to use that change to switch things over to the new zuulv3 job or should we create a fresh one?
16:38:41 melwitt lyarwood: I'd create a fresh one. I don't know why I didn't make a fresh one, sorry
16:40:22 lyarwood melwitt: ack np I can take a swing at that now
16:40:38 melwitt lyarwood: yeah sure, go ahead. thanks
16:41:17 melwitt just have to remove the -py3 from the name, I think that's all
16:41:53 melwitt (and depends-on tosky's grenade change)
16:42:45 lyarwood melwitt: yeah, doesn't our grenade job also run some live migration tests?
16:43:06 melwitt lyarwood: I ... don't know tbh
16:44:18 lyarwood https://github.com/openstack/nova/blob/80539a5e849bf8ec1239f0ed01433a7f8a54015b/playbooks/legacy/nova-grenade-multinode/run.yaml#L57-L59 - yeah it does, I don't think the new job includes this yet.
16:44:19 melwitt note that I was referencing only the single node grenade job earlier, we also have a multinode grenade job that I did _not_ try to convert to yet using tosky's change
16:44:27 lyarwood ah okay I see
16:44:34 melwitt so that might be the one
16:44:49 melwitt and you already linked it so yes
16:46:13 tosky and tosky wasn't able to find a proper workaround for the failures when using grenade plugins (shame on him)
16:54:58 melwitt tosky: do you have a change where you are working on getting plugins to work? is that here? https://review.opendev.org/639774 is there any way we could help?
16:56:46 tosky melwitt: I don't have a change; the problem is that I'm not able to figure out where the py2 openstackclient is installed and make sure its CLI used over its py3 counterpart
16:57:08 melwitt oh, interesting
16:57:26 tosky I would expect that starting from a py3 deployment, even if the py2 version is installed afterwards, the py3 version installed first would still be the one used for the CLI
17:00:10 melwitt I would expect that as well
17:01:28 stephenfin efried: create a blueprint for what?
17:02:43 lyarwood stephenfin: https://review.opendev.org/702943 - I think he was talking about that spec that had just merged.
17:03:01 stephenfin ah, makes sense
17:03:03 stephenfin will do
17:03:32 tosky talking about jobs, a few days ago the nova-live-migration job was failing in stable/pike; do you know if it has been solved?
17:04:38 melwitt tosky: it has not been solved yet. sean-k-mooney said they would try to dig into it more soon. it's not 100% failure rate but seems like it's close to that
17:06:44 efried stephenfin: I was talking about https://review.opendev.org/#/c/682655/
17:07:04 efried It looks like you're using a new bp name for this round, but there's no bp by that name.
17:09:31 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Switch to the Zuulv3 grenade job https://review.opendev.org/704364
17:10:50 stephenfin efried: https://blueprints.launchpad.net/nova/+spec/flavor-extra-spec-validators
17:11:00 efried thanks
17:15:19 sean-k-mooney tosky: its actully the other way around. which ever version is installed last get used but only if its the first installation
17:15:40 sean-k-mooney if you upgrade an existing install it does not replace the console script i think
17:16:11 sean-k-mooney so ify its py2 then py3 on n-1 1 and py3 only on n i thing the console scrript will point to py2
17:17:15 sean-k-mooney anyway its a bit of a pain. we likely should make the py2 install in py3 mode optional in train
17:17:26 sean-k-mooney so we can disable it in the greade job
17:18:15 sean-k-mooney but yes i will try to look at nova-live-migration this week. i think i have got a downstream backport working so i should have more time to look at it this week
17:18:35 sean-k-mooney i might try to set aside tomorow to look at it
17:26:32 stephenfin gibi: RE: https://review.opendev.org/#/c/701763/4/specs/ussuri/approved/non-admin-filter-instance-by-az.rst@83, are you saying it's not possible to use policy to allow any user access to the API?
17:28:45 gibi stephenfin: I think the policy is not per field
17:30:12 gibi stephenfin: so if we want to make some fields available for filtering to the normal user then we need to change code

Earlier   Later