| *** slaweq has quit IRC | 00:02 | |
| sean-k-mooney | ... i think it took ~90 seconds so the 60 second timeout expired but by the time it printed the list the compute service was there so this is just a timeing issue :( | 00:02 |
|---|---|---|
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert "Temporarily skip TestNovaMigrationsMySQL" https://review.opendev.org/683011 | 00:06 |
| *** mriedem is now known as mriedem_afk | 00:09 | |
| openstackgerrit | Merged openstack/nova-specs master: Track user_id/project_id for migrations https://review.opendev.org/682511 | 00:12 |
| *** mmethot has quit IRC | 00:18 | |
| *** mmethot has joined #openstack-nova | 00:18 | |
| *** slaweq has joined #openstack-nova | 00:18 | |
| *** mmethot has quit IRC | 00:19 | |
| *** mmethot has joined #openstack-nova | 00:19 | |
| *** gyee has quit IRC | 00:22 | |
| *** slaweq has quit IRC | 00:23 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793 | 00:23 |
| *** markvoelker has joined #openstack-nova | 00:25 | |
| *** markvoelker has quit IRC | 00:29 | |
| *** ociuhandu has joined #openstack-nova | 00:40 | |
| *** brinzhang has joined #openstack-nova | 00:41 | |
| *** ociuhandu has quit IRC | 00:45 | |
| KeithMnemonic1 | mriedem thanks! so how do i proceed, create a qorking branch, apply you the conflicting patches and then try your cherry pick? | 00:49 |
| KeithMnemonic1 | s/qorking/working | 00:49 |
| *** macz has quit IRC | 00:52 | |
| *** dave-mccowan has joined #openstack-nova | 00:52 | |
| melwitt | KeithMnemonic1: you can pull down his patch with 'git review -d 683008' and then you just need to resolve the merge conflicts he left in the tests | 00:53 |
| KeithMnemonic1 | and then do a git review once i think they are resolved? | 00:54 |
| melwitt | then --amend the commit and then push a new patch set | 00:54 |
| melwitt | yes | 00:54 |
| KeithMnemonic1 | perfect let me try that, thank you both! | 00:54 |
| melwitt | cool good luck | 00:55 |
| *** markvoelker has joined #openstack-nova | 01:00 | |
| *** jmlowe has quit IRC | 01:09 | |
| *** mmethot has quit IRC | 01:10 | |
| *** mmethot has joined #openstack-nova | 01:10 | |
| *** mriedem_afk has quit IRC | 01:21 | |
| *** yaawang has joined #openstack-nova | 01:43 | |
| openstackgerrit | Fan Zhang proposed openstack/nova master: Fix exception translation when creating volume https://review.opendev.org/678991 | 01:44 |
| *** Tianhao_Hu has joined #openstack-nova | 01:49 | |
| *** Tianhao_Hu has left #openstack-nova | 01:50 | |
| KeithMnemonic1 | melwitt quick question, if some of those conflicts are from other patches that cherry pick clean to pike, is the the proper process? | 01:51 |
| KeithMnemonic1 | isnt it better to pull in cleanly any dependencies instead of me trying to shove some stuff in his review? | 01:52 |
| KeithMnemonic1 | and then rebase that patch once they are merged? | 01:53 |
| melwitt | KeithMnemonic1: we don't pull in dependencies that are unrelated to the backport, it can get pretty unwieldy that way. by resolving the conflict you're just making the patch fit within the env of the older release, which shouldn't result in adding a bunch of stuff | 01:54 |
| melwitt | from what I saw when I glanced, the conflicts didn't look too big and mriedem documented all of them in the commit message already. just need to be smoothed out in the .py files | 01:54 |
| *** gbarros has joined #openstack-nova | 01:56 | |
| *** mmethot has quit IRC | 02:18 | |
| *** mmethot has joined #openstack-nova | 02:18 | |
| *** ricolin has joined #openstack-nova | 02:22 | |
| *** mmethot has quit IRC | 02:25 | |
| *** mmethot has joined #openstack-nova | 02:25 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Move Destination object tests to their own test class https://review.opendev.org/683017 | 02:29 |
| *** mmethot has quit IRC | 02:30 | |
| *** mmethot has joined #openstack-nova | 02:31 | |
| *** TxGirlGeek has quit IRC | 02:38 | |
| *** markvoelker has quit IRC | 02:47 | |
| *** gbarros has quit IRC | 02:48 | |
| *** tbachman has quit IRC | 02:50 | |
| *** BjoernT has joined #openstack-nova | 02:50 | |
| *** BjoernT_ has joined #openstack-nova | 02:54 | |
| *** BjoernT has quit IRC | 02:56 | |
| *** jmlowe has joined #openstack-nova | 02:58 | |
| *** jmlowe has quit IRC | 03:05 | |
| openstackgerrit | Merged openstack/nova master: Temporarily skip TestNovaMigrationsMySQL https://review.opendev.org/683009 | 03:21 |
| openstackgerrit | Merged openstack/nova stable/stein: Fix race in _test_live_migration_force_complete https://review.opendev.org/681744 | 03:21 |
| openstackgerrit | Merged openstack/nova stable/stein: Fix a misuse of assertGreaterEqual https://review.opendev.org/682168 | 03:21 |
| *** mkrai_ has joined #openstack-nova | 03:27 | |
| *** psachin has joined #openstack-nova | 03:34 | |
| *** udesale has joined #openstack-nova | 04:04 | |
| *** mkrai_ has quit IRC | 04:12 | |
| *** mkrai has joined #openstack-nova | 04:13 | |
| *** pcaruana has joined #openstack-nova | 04:42 | |
| *** ratailor has joined #openstack-nova | 04:49 | |
| *** Luzi has joined #openstack-nova | 04:50 | |
| *** rcernin_ has joined #openstack-nova | 04:56 | |
| *** rcernin has quit IRC | 04:59 | |
| *** boxiang has quit IRC | 05:11 | |
| *** boxiang has joined #openstack-nova | 05:12 | |
| *** janki has joined #openstack-nova | 05:21 | |
| openstackgerrit | garyk proposed openstack/nova master: Deconstruct the mother of all locks https://review.opendev.org/682242 | 05:45 |
| *** BjoernT_ has quit IRC | 05:45 | |
| *** alex_xu has quit IRC | 05:51 | |
| *** tbachman has joined #openstack-nova | 05:53 | |
| *** tbachman has quit IRC | 06:16 | |
| *** belmoreira has joined #openstack-nova | 06:17 | |
| *** markvoelker has joined #openstack-nova | 06:27 | |
| *** ralonsoh has joined #openstack-nova | 06:28 | |
| *** markvoelker has quit IRC | 06:31 | |
| *** jaosorior has joined #openstack-nova | 06:43 | |
| openstackgerrit | hulina proposed openstack/nova master: Nova raise exceptions when extending volume fails https://review.opendev.org/680648 | 06:45 |
| *** ociuhandu has joined #openstack-nova | 06:46 | |
| *** rcernin_ has quit IRC | 06:48 | |
| *** takamatsu has quit IRC | 06:50 | |
| *** ociuhandu has quit IRC | 06:51 | |
| *** cshen has joined #openstack-nova | 06:52 | |
| *** psachin has quit IRC | 06:53 | |
| *** udesale has quit IRC | 06:57 | |
| *** udesale has joined #openstack-nova | 06:58 | |
| *** mkrai has quit IRC | 07:03 | |
| *** mkrai_ has joined #openstack-nova | 07:03 | |
| *** rpittau|afk is now known as rpittau | 07:05 | |
| *** damien_r has joined #openstack-nova | 07:05 | |
| *** xek has joined #openstack-nova | 07:05 | |
| *** slaweq has joined #openstack-nova | 07:07 | |
| *** awalende has joined #openstack-nova | 07:08 | |
| *** udesale has quit IRC | 07:18 | |
| *** udesale has joined #openstack-nova | 07:19 | |
| *** slaweq has quit IRC | 07:20 | |
| *** belmoreira has quit IRC | 07:25 | |
| *** tesseract has joined #openstack-nova | 07:26 | |
| *** slaweq has joined #openstack-nova | 07:33 | |
| *** bbowen_ has quit IRC | 07:35 | |
| *** bbowen_ has joined #openstack-nova | 07:35 | |
| *** luksky has joined #openstack-nova | 07:37 | |
| *** xek has quit IRC | 07:38 | |
| *** xek has joined #openstack-nova | 07:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Update keypairs in saving an instance object https://review.opendev.org/683043 | 07:41 |
| gibi | sean-k-mooney: thanks for the work on https://review.opendev.org/#/c/669738/ I +Ad it | 07:42 |
| *** takamatsu has joined #openstack-nova | 07:44 | |
| *** mjozefcz has joined #openstack-nova | 07:45 | |
| *** ivve has joined #openstack-nova | 07:45 | |
| *** ttsiouts has joined #openstack-nova | 07:46 | |
| bauzas | gibi: sean-k-mooney: I just left some comments (and I concurrently +Wd, my bad) that honestly I feel we could have defined this test module as a plain top-level function testsuite for configdrivers | 07:47 |
| bauzas | configdrives* | 07:47 |
| gibi | bauzas: ack. I think the reorg for a top level test module can be done separately | 07:49 |
| bauzas | gibi: oh yeah totally | 07:49 |
| gibi | bauzas: and I'm supportive :) | 07:49 |
| bauzas | gibi: just saying this regression testclass is way more than a single check :) | 07:49 |
| gibi | yeah | 07:49 |
| *** jaosorior has quit IRC | 07:51 | |
| gibi | bauzas: here is a nova-manage heal allocation test addition if you have some review time https://review.opendev.org/#/c/669879/11 | 07:51 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857 | 07:56 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pull up compute node queries to init_host https://review.opendev.org/682680 | 07:56 |
| *** zbr is now known as zbr|ruck | 07:57 | |
| openstackgerrit | hulina proposed openstack/nova master: Nova raise exceptions when extending volume fails https://review.opendev.org/680648 | 07:57 |
| *** ttsiouts has quit IRC | 08:00 | |
| *** ttsiouts has joined #openstack-nova | 08:01 | |
| *** ttsiouts has quit IRC | 08:05 | |
| kashyap | mnaser: Hi, reading the scroll. | 08:05 |
| * gibi got visa for China | 08:15 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix unit of hw_rng:rate_period https://review.opendev.org/683051 | 08:16 |
| kashyap | mnaser: There's a lot scrollback; let's discuss today when you're around. :-) | 08:16 |
| *** elod has quit IRC | 08:19 | |
| *** tkajinam has quit IRC | 08:19 | |
| *** arxcruz|ruck is now known as arxcruz|rover | 08:20 | |
| *** adrianc has quit IRC | 08:20 | |
| *** adrianc has joined #openstack-nova | 08:21 | |
| *** lennyb has quit IRC | 08:21 | |
| bauzas | gibi: congrats for your visa, I got mine too yesterday by postal service | 08:22 |
| gibi | bauzas: congrats for you too then | 08:22 |
| bauzas | (and I understand why they asked me my passport) | 08:23 |
| * mordred is thankful he already had a china visa | 08:23 | |
| *** elod has joined #openstack-nova | 08:24 | |
| bauzas | I have a powerful passport, it was the first time I asked for this but I was a bit afraid my visa application could be not supported because I had some coffee stains | 08:24 |
| bauzas | (on my passport) | 08:25 |
| kashyap | mnaser: But a couple of things to get out of the way: | 08:25 |
| *** ttsiouts has joined #openstack-nova | 08:26 | |
| kashyap | mnaser: * When you use 'host-model', you won't actually see 'host-model' in the *live* guest XML; libvirt expands it (see next point) | 08:26 |
| kashyap | mnaser: * To refresh your memory, 'host-model' means: (a) get maximum possible CPU features from the host; (b) make the guest live-migratable; and (c) auto-add critical CPU flags -- provided you have the right underlying bits for microcode, kernel, etc. | 08:26 |
| kashyap | mnaser: * The "IvyBridge-IBRS" model does *not* contain 'vmx'; you have to explicitly add it. (You can `grep` the 'cpu_map.xml'; or if you're running new enough libvirt `grep vmx /usr/share/libvirt/cpu_map/x86_IvyBridge-IBRS.xml`) | 08:26 |
| *** markvoelker has joined #openstack-nova | 08:28 | |
| *** derekh has joined #openstack-nova | 08:30 | |
| *** zzzeek has quit IRC | 08:30 | |
| kashyap | mnaser: And lastly, since you're in the nested setup: make sure that your level-1 guest explicitly has 'vmx' flag set, even if you're using 'host-model'. | 08:30 |
| * kashyap stops the monologue now | 08:30 | |
| *** zzzeek has joined #openstack-nova | 08:31 | |
| *** takashin has left #openstack-nova | 08:32 | |
| *** dtantsur|afk is now known as dtantsur | 08:32 | |
| *** markvoelker has quit IRC | 08:33 | |
| *** lpetrut has joined #openstack-nova | 08:33 | |
| *** lennyb has joined #openstack-nova | 08:34 | |
| *** lpetrut has quit IRC | 08:39 | |
| *** ociuhandu has joined #openstack-nova | 08:45 | |
| *** ociuhandu has quit IRC | 08:46 | |
| *** takamatsu_ has joined #openstack-nova | 08:46 | |
| *** takamatsu has quit IRC | 08:46 | |
| *** ociuhandu has joined #openstack-nova | 08:47 | |
| *** ociuhandu has quit IRC | 08:53 | |
| *** ociuhandu has joined #openstack-nova | 08:53 | |
| *** ociuhandu has quit IRC | 08:54 | |
| *** ociuhandu has joined #openstack-nova | 08:54 | |
| *** mkrai_ has quit IRC | 08:57 | |
| bauzas | grraaah, got swamped by internal escalation | 09:01 |
| openstackgerrit | Arthur Dayne proposed openstack/nova master: Fix block disk attachment failure https://review.opendev.org/682772 | 09:04 |
| *** lpetrut has joined #openstack-nova | 09:05 | |
| *** jangutter has joined #openstack-nova | 09:09 | |
| *** takamatsu_ has quit IRC | 09:19 | |
| *** tetsuro has joined #openstack-nova | 09:20 | |
| *** slaweq has quit IRC | 09:25 | |
| *** slaweq has joined #openstack-nova | 09:27 | |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.opendev.org/621646 | 09:31 |
| *** takamatsu has joined #openstack-nova | 09:36 | |
| *** jawad_axd has joined #openstack-nova | 09:44 | |
| *** ociuhandu has quit IRC | 09:46 | |
| *** ociuhandu has joined #openstack-nova | 09:47 | |
| *** ociuhandu has quit IRC | 09:52 | |
| *** janki has quit IRC | 09:57 | |
| *** takamatsu has quit IRC | 10:00 | |
| *** udesale has quit IRC | 10:01 | |
| *** gbarros has joined #openstack-nova | 10:03 | |
| *** takamatsu has joined #openstack-nova | 10:06 | |
| *** cshen has quit IRC | 10:09 | |
| *** ociuhandu has joined #openstack-nova | 10:15 | |
| *** rcernin_ has joined #openstack-nova | 10:15 | |
| *** gbarros has quit IRC | 10:21 | |
| *** ttsiouts has quit IRC | 10:21 | |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: doc: attaching virtual persistent memory to guests https://review.opendev.org/680300 | 10:22 |
| *** ttsiouts has joined #openstack-nova | 10:22 | |
| *** cshen has joined #openstack-nova | 10:25 | |
| *** ttsiouts has quit IRC | 10:27 | |
| *** markvoelker has joined #openstack-nova | 10:29 | |
| *** cshen has quit IRC | 10:31 | |
| *** udesale has joined #openstack-nova | 10:32 | |
| *** cshen has joined #openstack-nova | 10:33 | |
| *** tetsuro has quit IRC | 10:34 | |
| *** markvoelker has quit IRC | 10:34 | |
| *** trident has quit IRC | 10:35 | |
| *** rcernin_ has quit IRC | 10:36 | |
| *** udesale has quit IRC | 10:37 | |
| *** udesale has joined #openstack-nova | 10:37 | |
| *** sapd1_x has joined #openstack-nova | 10:37 | |
| *** cshen has quit IRC | 10:37 | |
| *** cshen has joined #openstack-nova | 10:38 | |
| *** jangutter has quit IRC | 10:44 | |
| *** trident has joined #openstack-nova | 10:48 | |
| *** pcaruana has quit IRC | 10:54 | |
| *** rpittau is now known as rpittau|afk | 10:56 | |
| *** jangutter has joined #openstack-nova | 11:00 | |
| *** ttsiouts has joined #openstack-nova | 11:00 | |
| *** takamatsu has quit IRC | 11:00 | |
| *** udesale has quit IRC | 11:01 | |
| *** jangutter has quit IRC | 11:05 | |
| *** takamatsu has joined #openstack-nova | 11:06 | |
| sean-k-mooney | bauzas: ya i coudl do that. i am only testing the regression cases so i would have to add more test that are not based on the conf option to make it a propper config-drive test suite | 11:07 |
| sean-k-mooney | bauzas: i originaly had the tests in the test_server.py module and mriedem suggested making it a regression so i coudl avoid the old legacy code and use teh integrated helpers but since then i have added alot more tests | 11:09 |
| sean-k-mooney | bauzas: so ya it definetrly worth considering factoring it into its own module and addign the remiaing tests to validate other config drive behavior e.g. passing the flage to server create and i think there is an image property you can set. so if i added 2 or 3 more tests it would fully cover the behavior | 11:11 |
| *** jmlowe has joined #openstack-nova | 11:14 | |
| *** pcaruana has joined #openstack-nova | 11:17 | |
| mnaser | kashyap: yeah it seems we've pretty much came to the same conclusion with sean-k-mooney regarding vmx yesterday. I appreciate you actually validating our theories | 11:21 |
| mnaser | Knowing libvirt expands it is good to knwo | 11:21 |
| kashyap | mnaser: Yes, that's something the casual user might miss: | 11:21 |
| kashyap | mnaser: "I specified 'host-model' on guest boot, why don't I see it in live?" | 11:21 |
| sean-k-mooney | might miss. consider a bug... | 11:21 |
| kashyap | mnaser: ... not realizing that libvirt expands it at run-time | 11:21 |
| sean-k-mooney | that said i know why it does it | 11:22 |
| kashyap | sean-k-mooney: Guess I broke your long sentence, sorry | 11:22 |
| *** takamatsu has quit IRC | 11:22 | |
| sean-k-mooney | not really | 11:22 |
| kashyap | Ah! | 11:22 |
| kashyap | You actually were commemting at my remark. (I already forgot I said "might miss") | 11:23 |
| sean-k-mooney | i was just making the point that while i know why it expands it some might consier some of the expantion a bug | 11:23 |
| kashyap | sean-k-mooney: Why bug? | 11:23 |
| kashyap | sean-k-mooney: Ah, yeah, I see what you mean, though | 11:23 |
| sean-k-mooney | because in this case its not additive | 11:23 |
| kashyap | sean-k-mooney: It needs to documented clearly. | 11:23 |
| kashyap | But I agree with what you imply | 11:23 |
| sean-k-mooney | it replace the requested model with what was equvalent on that host | 11:23 |
| kashyap | Yes | 11:24 |
| sean-k-mooney | it would be require to do the current expantion if it move the vm to another host | 11:24 |
| kashyap | I can't parse your second q. | 11:24 |
| kashyap | s/q./comment/ | 11:24 |
| sean-k-mooney | if you were to live migrate it could not rely on the models being the same | 11:25 |
| sean-k-mooney | it can check but it simpler to expand the flags for a migration | 11:25 |
| sean-k-mooney | so rather then specail case that they made it the default behavior | 11:25 |
| sean-k-mooney | fortunetly however you added the extra_flags config | 11:26 |
| kashyap | sean-k-mooney: Ah, yes; thanks for expanding your comment | 11:26 |
| sean-k-mooney | so you can just add vmx or whatever you thing is missing | 11:26 |
| kashyap | Yeah. sean-k-mooney -- But I haven't had the chance to thoroughly test the multiple CPU models thing we merged | 11:26 |
| kashyap | Yeah | 11:27 |
| sean-k-mooney | well mnaser is still on stien | 11:27 |
| sean-k-mooney | at least until train is actully released | 11:27 |
| sean-k-mooney | mnaser: im expecting to hear vexxhost is fully in production with train in about a week or two right :P | 11:27 |
| *** macz has joined #openstack-nova | 11:28 | |
| *** takamatsu has joined #openstack-nova | 11:28 | |
| kashyap | sean-k-mooney: If you addressed to me something after "at least until train ..." please re-post; I got dropped briefly | 11:32 |
| kashyap | I just said that I realize mnaser is on Stein; but I was just talking from the POV of functional testing in Train | 11:32 |
| *** macz has quit IRC | 11:32 | |
| *** mkrai has joined #openstack-nova | 11:32 | |
| sean-k-mooney | kashyap: no i was just admireing/makeing fun of the fact that vexxhost/osa has good ci/cd and unlike most operators actully runs a recent openstack release | 11:33 |
| kashyap | Ah, nod :-) | 11:33 |
| openstackgerrit | Merged openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693 | 11:33 |
| mnaser | Train on release day would be cool, we're seeing.. | 11:34 |
| sean-k-mooney | osa at least in the passed did not aim to have upgrade support day 1 corret. so that will be one of the requirements pre upgrading | 11:35 |
| sean-k-mooney | although you mentioned in teh project in denver that ye were tyring to get upgrade testing in the gate to run on each patch | 11:36 |
| sean-k-mooney | so maybe that has changed? | 11:36 |
| mnaser | We have upgrade jobs now that upgrade from Stein to master | 11:37 |
| mnaser | Which afaik included placement reshuffle | 11:37 |
| *** sapd1_x has quit IRC | 11:38 | |
| openstackgerrit | Merged openstack/nova master: objects: Add 'InstanceNUMATopology.cpu_pinning' property https://review.opendev.org/680106 | 11:39 |
| sean-k-mooney | mnaser: cool i hope the placement work is finished in tripleo by now | 11:39 |
| *** nweinber has joined #openstack-nova | 11:51 | |
| *** jangutter has joined #openstack-nova | 11:53 | |
| *** tetsuro has joined #openstack-nova | 11:58 | |
| bauzas | mmm, super weirdo : http://paste.openstack.org/show/777697/ | 12:05 |
| bauzas | but https://docs.openstack.org/api-ref/compute/?expanded=evacuate-server-evacuate-action-detail,update-forced-down-detail#update-forced-down tells me it's right | 12:05 |
| bauzas | gibi: any ideas ? ^ | 12:05 |
| bauzas | or sean-k-mooney: ^ | 12:05 |
| *** ricolin has quit IRC | 12:06 | |
| bauzas | 'host' and 'binary' are correct arguments for force-down (at least for 2.77 microversion) | 12:06 |
| bauzas | I don't see why I'm getting a HTTP400 | 12:07 |
| gibi | looking | 12:08 |
| bauzas | thanks | 12:08 |
| gibi | Starting with microversion 2.53 this API is superseded by PUT /os-services/{service_id}. | 12:08 |
| bauzas | ah | 12:09 |
| bauzas | I remember that but the API docs don't tell it AFAICS | 12:09 |
| gibi | I copied this note from the API doc | 12:10 |
| bauzas | holy shit, it does | 12:10 |
| gibi | s | 12:10 |
| bauzas | yeah, me facepalms | 12:10 |
| bauzas | FWIW, I just used some existing helper method, but I guess it sticks with an old microversion call then | 12:10 |
| gibi | yeah, could be | 12:11 |
| bauzas | it does indeed, 2.14 | 12:11 |
| bauzas | we should deprecate this caller | 12:11 |
| bauzas | and tell to only use with microversion less than 2.53 (or force an older microversion) | 12:11 |
| *** markvoelker has joined #openstack-nova | 12:13 | |
| sean-k-mooney | huh so that is what that looks like when you uses the wrong microversion | 12:15 |
| bauzas | indeed, but I had to RTFM | 12:17 |
| bauzas | (which I did, but my pink eyes got me wrong apparently) | 12:17 |
| bauzas | so, nothing to change from an API perspective IHM | 12:18 |
| bauzas | IMHO | 12:18 |
| bauzas | it's just the helper method which would ideally add some docstring | 12:18 |
| *** takamatsu has quit IRC | 12:21 | |
| *** takamatsu has joined #openstack-nova | 12:27 | |
| *** ociuhandu has quit IRC | 12:27 | |
| *** lbragstad has quit IRC | 12:29 | |
| *** lbragstad has joined #openstack-nova | 12:30 | |
| *** eandersson has quit IRC | 12:31 | |
| *** ociuhandu has joined #openstack-nova | 12:36 | |
| *** priteau has joined #openstack-nova | 12:36 | |
| *** mgariepy has quit IRC | 12:37 | |
| *** brault has joined #openstack-nova | 12:37 | |
| *** takamatsu has quit IRC | 12:38 | |
| *** jmlowe has quit IRC | 12:39 | |
| *** ociuhandu has quit IRC | 12:40 | |
| *** ociuhandu has joined #openstack-nova | 12:41 | |
| *** markvoelker has quit IRC | 12:46 | |
| *** lpetrut has quit IRC | 12:48 | |
| *** takashin has joined #openstack-nova | 12:48 | |
| *** brault has quit IRC | 12:50 | |
| *** brault has joined #openstack-nova | 12:51 | |
| *** ociuhandu has quit IRC | 12:55 | |
| *** brault has quit IRC | 12:55 | |
| *** ociuhandu has joined #openstack-nova | 12:55 | |
| *** zzzeek has quit IRC | 12:56 | |
| *** jmlowe has joined #openstack-nova | 12:57 | |
| *** zzzeek has joined #openstack-nova | 12:58 | |
| *** ociuhandu has quit IRC | 13:00 | |
| *** ociuhandu has joined #openstack-nova | 13:01 | |
| *** mkrai has quit IRC | 13:04 | |
| *** mkrai has joined #openstack-nova | 13:04 | |
| *** slaweq has quit IRC | 13:07 | |
| *** ociuhandu has quit IRC | 13:12 | |
| openstackgerrit | Merged openstack/nova master: make config drives sticky bug 1835822 https://review.opendev.org/669738 | 13:12 |
| openstack | bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,In progress] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney) | 13:12 |
| *** eharney has joined #openstack-nova | 13:12 | |
| sean-k-mooney | TIL things can merge on there 1st time in the gate queue | 13:13 |
| sean-k-mooney | is suspcious | 13:13 |
| *** dtantsur is now known as dtantsur|bbl | 13:14 | |
| *** takamatsu has joined #openstack-nova | 13:16 | |
| *** jaosorior has joined #openstack-nova | 13:18 | |
| *** awalende has quit IRC | 13:18 | |
| *** brault has joined #openstack-nova | 13:18 | |
| *** tbachman has joined #openstack-nova | 13:19 | |
| *** brault has quit IRC | 13:20 | |
| *** brault has joined #openstack-nova | 13:20 | |
| *** macz has joined #openstack-nova | 13:21 | |
| mordred | sean-k-mooney: yeah. my first thought when something is green the first time is that I did something wrong | 13:22 |
| sean-k-mooney | i think we are finally getting past the gate instablity and have nailed down some of the intermitly failing tests | 13:23 |
| *** udesale has joined #openstack-nova | 13:23 | |
| *** udesale has quit IRC | 13:24 | |
| sean-k-mooney | there are still more work to be done but its looking a lot healtier then it was on monday | 13:24 |
| *** udesale has joined #openstack-nova | 13:25 | |
| *** macz has quit IRC | 13:26 | |
| *** pcaruana has quit IRC | 13:28 | |
| mordred | \o. | 13:29 |
| mordred | gah | 13:29 |
| *** derekh has quit IRC | 13:29 | |
| mordred | I meant | 13:29 |
| mordred | \o/ | 13:29 |
| mordred | stupid keyboards and fingers | 13:29 |
| bauzas | mordred: just use a French keyboard for a minute and you'll love the current keyboard you have | 13:30 |
| *** mriedem has joined #openstack-nova | 13:30 | |
| bauzas | Alt Gr, my love | 13:30 |
| *** mkrai has quit IRC | 13:32 | |
| sean-k-mooney | i use one of these with cherry mx brown switches. a mechanical keyboard make typing better regardless of the layout | 13:33 |
| sean-k-mooney | https://www.corsair.com/eu/en/k70-rgb-gaming-keyboard | 13:33 |
| *** boxiang has quit IRC | 13:34 | |
| *** boxiang has joined #openstack-nova | 13:35 | |
| *** boxiang has quit IRC | 13:36 | |
| *** boxiang has joined #openstack-nova | 13:37 | |
| *** ociuhandu has joined #openstack-nova | 13:38 | |
| *** ociuhandu has quit IRC | 13:40 | |
| *** ociuhandu has joined #openstack-nova | 13:41 | |
| bauzas | mriedem: around ? | 13:41 |
| bauzas | mriedem: I saw your comment on https://review.opendev.org/#/c/670112/ | 13:42 |
| bauzas | mriedem: thanks for it | 13:42 |
| bauzas | that said, I have one concern | 13:42 |
| bauzas | if we are adding some check that depends on an unfix bug, then we would need to fix this test when the bugfix arrives | 13:43 |
| bauzas | I mean, I'm totally ok with doing an evacuation | 13:43 |
| bauzas | but I just want to make sure that we would still have orphaned allocations even if someone fixes the "undeleted allocations" stuff when a host is deleted | 13:44 |
| mriedem | there are other ways to orphan migration-based allocations, like resizing a server and then before confirming delete the source compute service | 13:44 |
| *** tbachman has quit IRC | 13:44 | |
| mriedem | there might be other ways | 13:44 |
| mriedem | i do'nt know when those bugs will be fixed | 13:44 |
| mriedem | my brain is also not really functioning atm | 13:44 |
| bauzas | no worries | 13:44 |
| bauzas | you know what ? I'll just stick with your plan and leave a big fat comment, others could voice | 13:45 |
| *** ociuhandu has quit IRC | 13:47 | |
| *** jawad_axd has quit IRC | 13:51 | |
| *** ratailor has quit IRC | 13:51 | |
| *** jawad_axd has joined #openstack-nova | 13:53 | |
| *** Luzi has quit IRC | 13:53 | |
| *** pcaruana has joined #openstack-nova | 13:54 | |
| *** JamesBenson has joined #openstack-nova | 13:55 | |
| *** efried_pto is now known as efried | 13:56 | |
| efried | Nova meeting in 3 minutens | 13:57 |
| efried | ^ pseudo-german, apparently | 13:57 |
| *** jawad_axd has quit IRC | 13:57 | |
| *** derekh has joined #openstack-nova | 13:58 | |
| mriedem | efried: so you're running it? | 13:59 |
| mriedem | i just updated the agenda fwiw | 13:59 |
| *** BjoernT has joined #openstack-nova | 13:59 | |
| efried | oh, thank you, I guess I'll get a merge conflict then | 13:59 |
| *** tetsuro has quit IRC | 14:00 | |
| *** tesseract has quit IRC | 14:00 | |
| *** BjoernT_ has joined #openstack-nova | 14:02 | |
| *** nweinber has quit IRC | 14:02 | |
| *** cfriesen has joined #openstack-nova | 14:03 | |
| *** macz has joined #openstack-nova | 14:03 | |
| *** tbachman has joined #openstack-nova | 14:03 | |
| *** BjoernT has quit IRC | 14:04 | |
| *** tetsuro has joined #openstack-nova | 14:06 | |
| *** ociuhandu has joined #openstack-nova | 14:06 | |
| *** ociuhandu has quit IRC | 14:06 | |
| *** belmoreira has joined #openstack-nova | 14:07 | |
| *** ociuhandu has joined #openstack-nova | 14:07 | |
| *** macz has quit IRC | 14:07 | |
| *** ivve has quit IRC | 14:09 | |
| bauzas | oh crap, discovered a very interesting case | 14:12 |
| bauzas | https://docs.openstack.org/api-ref/placement/?expanded=delete-allocations-detail#delete-allocations deletes all allocations from this consumer | 14:12 |
| bauzas | but that supposes there are no duplicate allocations having the same consumer UUID between two RPs | 14:12 |
| bauzas | which isn't the case when you evacuate an instance and delete the source host | 14:12 |
| *** mrch_ has joined #openstack-nova | 14:12 | |
| *** ociuhandu has quit IRC | 14:12 | |
| bauzas | we're missing some API syntax to say 'I just want *this* allocation attached to *this* RP | 14:13 |
| *** takashin has quit IRC | 14:15 | |
| bauzas | that's sad because if you run my new audit command, then you'll loose your allocation on an existing instance until you heal it | 14:16 |
| *** takamatsu has quit IRC | 14:16 | |
| *** boxiang has quit IRC | 14:16 | |
| *** takashin has joined #openstack-nova | 14:17 | |
| gibi | bauzas: In my mind there is not two allocation with the same consumer UUID but there is a singel consumer that allocates from two RPs. | 14:18 |
| bauzas | gibi: sure, but at the end, we delete both records | 14:18 |
| *** macz has joined #openstack-nova | 14:19 | |
| bauzas | anyway, I'll amend my audit command to be preventive and claim there are duplicate records | 14:19 |
| bauzas | instead of deleting | 14:19 |
| stephenfin | mriedem: fwiw, that live-migrate job failed because placement-api didn't start in time. I'm assuming that couldn't be related to the change, which adds checks on compute-manager startup | 14:19 |
| mriedem | stephenfin: yeah i just left a comment about that | 14:19 |
| mriedem | likely a slow node | 14:19 |
| mriedem | devstack times out at 60 seconds | 14:19 |
| stephenfin | Ah, cool. Thanks for checking | 14:20 |
| *** macz has quit IRC | 14:23 | |
| mriedem | stephenfin: fwiw it looks rare: | 14:28 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22%5BERROR%5D%5C%22%20AND%20message%3A%5C%22placement-api%20did%20not%20start%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | 14:28 |
| *** pcaruana has quit IRC | 14:33 | |
| donnyd | FWIW all three of those ran on FN, so I am curious if its slowness or something else | 14:33 |
| *** ociuhandu has joined #openstack-nova | 14:34 | |
| *** mmedvede has quit IRC | 14:36 | |
| kashyap | mriedem: Dear cupid, just to bounce back on the ping from yesterday, I've responded earlier in the day to mnaser -- http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-09-19.log.html#t2019-09-19T08:25:57 | 14:36 |
| *** ociuhandu has quit IRC | 14:36 | |
| kashyap | mriedem: And we ended up being on the same sentence, same word: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-09-19.log.html#t2019-09-19T11:21:12 | 14:37 |
| kashyap | :D | 14:37 |
| *** ociuhandu has joined #openstack-nova | 14:37 | |
| *** mmedvede has joined #openstack-nova | 14:37 | |
| mriedem | donnyd: i'm assuming slowness, placement-api workers took 70+ seconds to start, | 14:37 |
| mriedem | devstack times out at 60 | 14:37 |
| mriedem | kashyap: ack | 14:38 |
| *** ociuhandu has quit IRC | 14:39 | |
| kashyap | I'll also send a patch to libvirt docs to explicitly note that 'host-model' will be "expanded" for the live guest. | 14:39 |
| *** ociuhandu has joined #openstack-nova | 14:39 | |
| *** jawad_axd has joined #openstack-nova | 14:40 | |
| donnyd | https://grafana.fortnebula.com/d/9MMqh8HWk/openstack-utilization?orgId=2&from=1568848907005&to=1568854899362 | 14:41 |
| donnyd | Here is the load snapshot from one of the failures.. doesn't look like anything is overloaded | 14:41 |
| *** jawad_axd has quit IRC | 14:44 | |
| *** ociuhandu has quit IRC | 14:45 | |
| *** macz has joined #openstack-nova | 14:45 | |
| *** brault has quit IRC | 14:46 | |
| lyarwood | does anyone recall where we actually call initialize_connection during a LM for the destination when using cinderv2? | 14:47 |
| lyarwood | I was sure it was hidden in pre_live_migration but I can't seem to find it. | 14:47 |
| *** macz has quit IRC | 14:50 | |
| *** tetsuro has quit IRC | 14:52 | |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Add spec for vm scoped sriov numa affinity https://review.opendev.org/683174 | 14:53 |
| sean-k-mooney | efried: so you are saying i should start writing more specs now :) | 14:53 |
| *** dtantsur|bbl is now known as dtantsur | 14:54 | |
| efried | sean-k-mooney: no, I'm saying there's no point writing a spec for something brand new. | 14:54 |
| sean-k-mooney | https://review.opendev.org/#/c/674072/3 is the code for ^ by the way | 14:54 |
| sean-k-mooney | because we have enough of a backlog already to finish | 14:54 |
| mriedem | lyarwood: it is in pre_live_migration from the compute manager | 14:54 |
| mriedem | eventually calls down into nova.virt.block_device - refresh_connection_info method | 14:55 |
| mriedem | lyarwood: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6719 | 14:55 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L677 | 14:55 |
| *** tetsuro has joined #openstack-nova | 14:55 | |
| efried | sean-k-mooney: that, and because it's absolute hell for the "customer" relying on an upstream feature to meet a business commitment to have no better than a 60% chance of the feature actually landing, despite being "approved" (or even *prioritized*) for the release. I'm hoping by reducing scope we can increase that percentage. | 14:55 |
| *** damien_r has quit IRC | 14:55 | |
| lyarwood | mriedem: thanks! | 14:56 |
| *** gyee has joined #openstack-nova | 14:56 | |
| mriedem | lyarwood: it's probably worth putting a comment by that call in the compute manager saying for legacy attached BDMs that will update the bdm connection_info using the dest host connector | 14:56 |
| lyarwood | mriedem: ack, I'll do that shortly this isn't the first time I've missed this. | 14:57 |
| *** efried is now known as efried_pto | 14:57 | |
| *** damien_r has joined #openstack-nova | 14:58 | |
| *** takashin has left #openstack-nova | 15:01 | |
| *** ociuhandu has joined #openstack-nova | 15:02 | |
| *** ociuhandu has quit IRC | 15:02 | |
| *** ociuhandu has joined #openstack-nova | 15:03 | |
| *** slaweq has joined #openstack-nova | 15:03 | |
| *** munimeha1 has joined #openstack-nova | 15:06 | |
| *** cshen has quit IRC | 15:11 | |
| *** damien_r has quit IRC | 15:11 | |
| *** mlavalle has joined #openstack-nova | 15:11 | |
| *** nweinber has joined #openstack-nova | 15:12 | |
| *** tetsuro has quit IRC | 15:13 | |
| *** tetsuro has joined #openstack-nova | 15:14 | |
| *** tetsuro has quit IRC | 15:15 | |
| *** derekh has quit IRC | 15:16 | |
| *** derekh has joined #openstack-nova | 15:18 | |
| *** derekh has quit IRC | 15:18 | |
| mriedem | lyarwood: remember, anything involving block devices has to be 3 layers of abstraction removed from where you'd expect it to be happening | 15:19 |
| *** TxGirlGeek has joined #openstack-nova | 15:20 | |
| *** ricolin has joined #openstack-nova | 15:21 | |
| *** jangutter has quit IRC | 15:29 | |
| *** mkrai has joined #openstack-nova | 15:33 | |
| *** mkrai has quit IRC | 15:35 | |
| donnyd | mriedem: 683166 in neutron is so the def route in ipv6 providers isn't purged when the public side of the network is created in the job | 15:38 |
| mriedem | ok. do we have job failures as a result of that? if so, do you have a link to one so we can try to categorize it in elastic-recheck? | 15:40 |
| openstackgerrit | Takashi Kajinami proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.opendev.org/456571 | 15:42 |
| *** jawad_axd has joined #openstack-nova | 15:42 | |
| mriedem | melwitt: on https://review.opendev.org/#/c/541420/ i'll try removing the weird dict/attribute stuff since that might be fixed now since i originally posted that patch, | 15:42 |
| mriedem | melwitt: but i wasn't sure what you were saying with "But, I was wondering, should we cover the detach case in the DNM patch above this one too? (I left a comment there)." | 15:42 |
| mriedem | oh nvm let me look at your comment first | 15:42 |
| openstackgerrit | Takashi Kajinami proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.opendev.org/456571 | 15:42 |
| donnyd | mriedem: 2019-09-19 09:50:54.608322 | controller | +++ lib/neutron_plugins/services/l3:_neutron_create_public_subnet_v6:330 : oscwrap --os-cloud devstack-admin --os-region RegionOne subnet create --ip-version 6 --gateway 2001:db8::2 --network 0dd00e84-7a3a-41a5-bcf0-79097e5c1d1f --subnet-range 2001:db8::/64 --no-dhcp ipv6-public-subnet | 15:43 |
| donnyd | https://storage.gra1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_650/682647/3/check/ironic-standalone-ipa-src/65091a8/job-output.txt | 15:43 |
| donnyd | there are a couple more examples | 15:43 |
| donnyd | https://www.irccloud.com/pastebin/7aez1E6y/ | 15:44 |
| melwitt | mriedem: ok cool. I never knew there was any issue with attr going to the object | 15:46 |
| mriedem | donnyd: sorry i'm not sure what i'm looking for here that indicates the job fails because of that | 15:46 |
| donnyd | The gist is on v6 providers the def route is dropped at this step and it may or may not come back up in time depending on RA / fw connections | 15:47 |
| *** jawad_axd has quit IRC | 15:47 | |
| mriedem | donnyd: as indicated by "[Zuul] Log Stream did not terminate" ? | 15:48 |
| donnyd | frickler: may have more insights | 15:48 |
| *** ttsiouts has quit IRC | 15:48 | |
| mriedem | "[Zuul] Log Stream did not terminate" shows up in passing jobs so that's not related | 15:49 |
| mriedem | anyway, i don't have +2 on devstack anyway, i was just doing a drive by review | 15:49 |
| *** ttsiouts has joined #openstack-nova | 15:49 | |
| donnyd | mriedem: well I was hoping for at least 6 more +1's | 15:50 |
| *** mjozefcz has quit IRC | 15:52 | |
| melwitt | 6 specifically? lol | 15:52 |
| donnyd | yes | 15:52 |
| melwitt | :) | 15:53 |
| donnyd | but no more than 8... that is too many | 15:53 |
| mriedem | back in the icehouse days i think 6 +1s from a single vendor equaled one +2 | 15:53 |
| *** ttsiouts has quit IRC | 15:54 | |
| mriedem | especially all within 1 hour of each other after their scrum call ended | 15:54 |
| melwitt | lol | 15:54 |
| melwitt | too real | 15:54 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Stop using volume_api.initialize/terminate_connection https://review.opendev.org/682508 | 15:55 |
| donnyd | mriedem: I am really just working on upping my level of irc sarcasm - ircasm | 15:55 |
| mriedem | donnyd: i should probably use o-) or :P or something until you're used to it | 15:55 |
| donnyd | :( | 15:55 |
| *** mkrai has joined #openstack-nova | 15:56 | |
| donnyd | ok | 15:56 |
| * donnyd slinks to the back of the room | 15:57 | |
| mriedem | somehow i've offended you and i'm not sure how i did | 15:57 |
| mriedem | i meant, i'm 80% sarcasm | 15:58 |
| mriedem | so don't take me literally | 15:58 |
| *** mgoddard has quit IRC | 15:58 | |
| *** mgoddard has joined #openstack-nova | 15:59 | |
| *** openstackgerrit has quit IRC | 16:06 | |
| *** takamatsu has joined #openstack-nova | 16:06 | |
| mriedem | https://review.opendev.org/#/c/679640/ failed with something i haven't seen in the gate before | 16:09 |
| mriedem | https://zuul.opendev.org/t/openstack/build/998bcd66b810467fae1664aaf280d53d/log/controller/logs/screen-n-cpu.txt.gz?severity=4#25797 | 16:09 |
| mriedem | os_brick.exception.VolumeDeviceNotFound: Volume device not found at /dev/disk/by-id. | 16:09 |
| *** macz has joined #openstack-nova | 16:10 | |
| mriedem | lyarwood: you might care about that ^ since it's a luks encrypted volume test | 16:10 |
| lyarwood | looking | 16:14 |
| mriedem | this is the multi-cell job, so i wonder if the encrypted volume is created on one host and then the server is created on the other host - but i'd expect that to be a problem in any multi-host job | 16:15 |
| *** macz has quit IRC | 16:15 | |
| *** markvoelker has joined #openstack-nova | 16:15 | |
| mriedem | maybe we don't run that test in other multihost jobs | 16:15 |
| *** mriedem is now known as mriedem_afk | 16:15 | |
| *** yaawang has quit IRC | 16:16 | |
| *** yaawang has joined #openstack-nova | 16:17 | |
| bauzas | argh, I'm stuck with trying to find a way to delete a single allocation record that I found orphaned | 16:18 |
| *** mkrai has quit IRC | 16:19 | |
| *** mkrai_ has joined #openstack-nova | 16:19 | |
| bauzas | because from the API perspective, I can only delete all allocs from a consumer UUID | 16:19 |
| sean-k-mooney | really you can do it by RP? | 16:20 |
| bauzas | mriedem_afk: in the case of an orphaned allocation from an evacuation, the audit will find it but we can't delete it or we gonna delete the whole instance resources (and then the user would have to heal) | 16:20 |
| bauzas | sean-k-mooney: https://docs.openstack.org/api-ref/placement/?expanded=delete-allocations-detail#delete-allocations | 16:20 |
| bauzas | I can list the allocations per RP, but I can't delete it | 16:20 |
| bauzas | I was thinking of just checking the allocations and bail out, but in the case of a duplicate, I need to check every allocation per RP | 16:21 |
| sean-k-mooney | ya i was wondering if there was a delete on /resource_providers/{uuid}/allocations | 16:21 |
| sean-k-mooney | bauzas: you could do an update of the consumer allocation | 16:22 |
| sean-k-mooney | and just remove teh allocation on the rp you want to delete | 16:22 |
| bauzas | ah | 16:22 |
| *** dtantsur is now known as dtantsur|afk | 16:22 | |
| sean-k-mooney | so get the allcotion and do a put to /allocations/{consumer_uuid} - the allcoation you want to remove | 16:23 |
| bauzas | that makes the code a bit less easy to read, but I can try | 16:23 |
| bauzas | sean-k-mooney: I guess placement doesn't delete the allocation when I update by removing the last RP ? | 16:24 |
| sean-k-mooney | there is one way to find out | 16:24 |
| sean-k-mooney | but i dont know | 16:24 |
| bauzas | writing code now | 16:24 |
| bauzas | hopefully I should be able to push something tomorrow morning | 16:25 |
| *** udesale has quit IRC | 16:27 | |
| *** udesale has joined #openstack-nova | 16:28 | |
| bauzas | sean-k-mooney: any idea on how I could remove a RP usage by https://docs.openstack.org/api-ref/placement/?&expanded=update-allocations-detail#update-allocations ? | 16:29 |
| bauzas | sean-k-mooney: I guess I need to update with the existing RPs but the one I wanna remove | 16:29 |
| lyarwood | mriedem_afk: this isn't specific to encrypted volumes, the call to connect_volume is failing because once we've connected to the target we can't find a block device locally that matches the WWN provided by c-api/c-vol. | 16:30 |
| lyarwood | mriedem_afk: at this point the fact the volume is encrypted doesn't come into play | 16:30 |
| sean-k-mooney | bauzas: isnt the usage caulated by subtracting the allcoation form the avilabe inventory | 16:30 |
| bauzas | sean-k-mooney: sorry, nevermind the "usage" wording, it's confusing | 16:30 |
| sean-k-mooney | i.e. once you remvoe teh allocation againt the RP the usage should be caulated correctly when next quired | 16:30 |
| bauzas | sean-k-mooney: I meant, say https://docs.openstack.org/api-ref/placement/?&expanded=update-allocations-detail#id72 | 16:31 |
| bauzas | sean-k-mooney: if I'd like to remove allocations against 89873422-1373-46e5-b467-f0c5e6acf08f, then I would update the endpoint with a payload having 4e061c03-611e-4caa-bf26-999dcff4284e only ? | 16:31 |
| bauzas | sean-k-mooney: and in this case, would it remove the DB records ? | 16:31 |
| sean-k-mooney | yes | 16:31 |
| sean-k-mooney | that is my understanding at least | 16:32 |
| bauzas | ok, we can test with my functional test at least | 16:32 |
| sean-k-mooney | or with osc-placment but sure | 16:33 |
| bauzas | we miss cdent, tetsuro, efried_pto | 16:33 |
| sean-k-mooney | functional test are good too | 16:33 |
| sean-k-mooney | https://github.com/openstack/placement/blob/master/placement/tests/functional/gabbits/allocations-1.28.yaml#L174-L181 | 16:34 |
| sean-k-mooney | bauzas: ^ | 16:34 |
| sean-k-mooney | so based on there functionla test if you use 1.28+ it should work | 16:35 |
| *** itlinux has joined #openstack-nova | 16:35 | |
| sean-k-mooney | well i should include the next test too so | 16:36 |
| sean-k-mooney | https://github.com/openstack/placement/blob/master/placement/tests/functional/gabbits/allocations-1.28.yaml#L174-L187 | 16:36 |
| sean-k-mooney | bauzas: so a put of empty allcoation is accpeted implying a put of a partill set of allcoation shoudl also work | 16:37 |
| *** openstackgerrit has joined #openstack-nova | 16:38 | |
| openstackgerrit | Merged openstack/nova-specs master: Re-propose cross-cell-resize spec for Ussuri https://review.opendev.org/683002 | 16:38 |
| *** ociuhandu has quit IRC | 16:40 | |
| *** igordc has joined #openstack-nova | 16:44 | |
| lyarwood | mriedem_afk: actually os-brick is doing some random lookups when encrypted volumes are used | 16:52 |
| *** udesale has quit IRC | 16:52 | |
| *** jmlowe has quit IRC | 16:59 | |
| bauzas | sean-k-mooney: it works seamlessly | 17:01 |
| *** pcaruana has joined #openstack-nova | 17:01 | |
| *** markvoelker has quit IRC | 17:01 | |
| * bauzas uploading a new rev now | 17:01 | |
| * bauzas can't wait December to get optic fiber at home | 17:01 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Add a placement audit command https://review.opendev.org/670112 | 17:02 |
| *** jmlowe has joined #openstack-nova | 17:02 | |
| lyarwood | mriedem_afk: so it looks like /dev/disk/by-id/ is just slow to populate on these CI nodes and we aren't retrying the lookup https://github.com/openstack/os-brick/blob/1c02d525bb77487d5de89c4bd58116b916d44207/os_brick/initiator/connectors/iscsi.py#L521-L540 | 17:03 |
| *** markvoelker has joined #openstack-nova | 17:04 | |
| lyarwood | mriedem_afk: the fact we are using /dev/disk/by-id/ is due to the volume being encrypted https://github.com/openstack/os-brick/blob/1c02d525bb77487d5de89c4bd58116b916d44207/os_brick/initiator/connectors/iscsi.py#L545-L553 | 17:04 |
| lyarwood | mriedem_afk: and is due to the horrid design of the original encryptors that symlink things in a horrific way such that we can't use the actual block device path on the host otherwise we'd nuke it https://bugs.launchpad.net/os-brick/+bug/1703954 | 17:05 |
| openstack | Launchpad bug 1703954 in OpenStack Compute (nova) "Attach/Detach encrypted volume problems with real paths" [Undecided,Incomplete] | 17:05 |
| *** cfriesen has quit IRC | 17:06 | |
| lyarwood | mriedem_afk: I'll post something in the morning to retry _get_device_link but in the longer term I think I need to sort out the symlinking in the encryptors such that it's safe to provide them with a real block device path and they don't do anything stupid. | 17:07 |
| *** zbr|ruck is now known as zbr | 17:07 | |
| lyarwood | mriedem_afk: anyway both tasks for the morning \o_ | 17:07 |
| *** cfriesen has joined #openstack-nova | 17:08 | |
| *** cfriesen has quit IRC | 17:11 | |
| *** pcaruana has quit IRC | 17:14 | |
| *** ociuhandu has joined #openstack-nova | 17:16 | |
| *** ociuhandu has quit IRC | 17:16 | |
| *** mkrai_ has quit IRC | 17:20 | |
| *** ralonsoh has quit IRC | 17:21 | |
| *** gmann_afk is now known as gmann | 17:21 | |
| *** ociuhandu has joined #openstack-nova | 17:24 | |
| *** pcaruana has joined #openstack-nova | 17:25 | |
| *** ociuhandu has quit IRC | 17:28 | |
| *** ricolin has quit IRC | 17:31 | |
| *** mriedem_afk is now known as mriedem | 17:33 | |
| mriedem | lyarwood: ack thanks for investigating, i'll open a bug (i found https://bugs.launchpad.net/os-brick/+bug/1820007 but looks old/different) and start tracking in e-r | 17:36 |
| openstack | Launchpad bug 1820007 in os-brick "Failed to attach encrypted volumes after detach: volume device not found at /dev/disk/by-id" [Undecided,New] | 17:36 |
| mriedem | it also only failed on ovh nodes so maybe those are slower here, idk | 17:36 |
| *** jawad_axd has joined #openstack-nova | 17:37 | |
| mriedem | actually a lot of the details in that bug look the same so i'll just re-use that | 17:37 |
| *** pcaruana has quit IRC | 17:55 | |
| *** efried_pto has quit IRC | 18:01 | |
| *** efried has joined #openstack-nova | 18:03 | |
| *** efried is now known as efried_pto | 18:03 | |
| *** priteau has quit IRC | 18:07 | |
| *** brault has joined #openstack-nova | 18:09 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova stable/pike: WIP: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008 | 18:10 |
| *** brault has quit IRC | 18:10 | |
| *** igordc has quit IRC | 18:20 | |
| *** jawad_axd has quit IRC | 18:21 | |
| *** priteau has joined #openstack-nova | 18:28 | |
| *** priteau has quit IRC | 18:29 | |
| openstackgerrit | Merged openstack/nova master: Trigger real BuildAbortException during migrate with bandwidth https://review.opendev.org/682876 | 18:35 |
| *** trident has quit IRC | 18:45 | |
| *** mvkr has quit IRC | 18:48 | |
| *** gbarros has joined #openstack-nova | 18:49 | |
| *** brault has joined #openstack-nova | 18:55 | |
| *** trident has joined #openstack-nova | 18:55 | |
| *** brault has quit IRC | 18:56 | |
| *** brault has joined #openstack-nova | 18:56 | |
| *** trident has quit IRC | 19:01 | |
| *** gbarros has quit IRC | 19:01 | |
| *** cshen has joined #openstack-nova | 19:05 | |
| *** trident has joined #openstack-nova | 19:10 | |
| *** brault has quit IRC | 19:16 | |
| *** cshen has quit IRC | 19:19 | |
| *** tbachman has quit IRC | 19:21 | |
| *** cshen has joined #openstack-nova | 19:30 | |
| *** cshen has quit IRC | 19:36 | |
| *** jmlowe has quit IRC | 19:45 | |
| *** tbachman has joined #openstack-nova | 19:46 | |
| *** macz has joined #openstack-nova | 19:49 | |
| *** pcaruana has joined #openstack-nova | 19:51 | |
| *** nweinber has quit IRC | 20:01 | |
| *** mmethot_ has joined #openstack-nova | 20:10 | |
| *** mmethot has quit IRC | 20:11 | |
| openstackgerrit | Merged openstack/nova master: Validate CPU config options against running instances https://review.opendev.org/680107 | 20:15 |
| openstackgerrit | Merged openstack/nova master: objects: Add 'NUMACell.pcpuset' field https://review.opendev.org/680108 | 20:15 |
| openstackgerrit | Merged openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800 | 20:15 |
| *** larainema has joined #openstack-nova | 20:31 | |
| *** pcaruana has quit IRC | 20:31 | |
| *** igordc has joined #openstack-nova | 20:33 | |
| *** eharney has quit IRC | 20:42 | |
| *** brault has joined #openstack-nova | 20:51 | |
| *** brault has quit IRC | 20:55 | |
| *** TxGirlGeek has quit IRC | 21:00 | |
| *** TxGirlGeek has joined #openstack-nova | 21:00 | |
| *** slaweq has quit IRC | 21:01 | |
| *** brault has joined #openstack-nova | 21:02 | |
| *** brault has quit IRC | 21:08 | |
| *** brault has joined #openstack-nova | 21:17 | |
| *** markvoelker has quit IRC | 21:20 | |
| *** JamesBenson has quit IRC | 21:29 | |
| *** benj_ has quit IRC | 21:31 | |
| *** benj has joined #openstack-nova | 21:31 | |
| *** JamesBenson has joined #openstack-nova | 21:31 | |
| *** benj is now known as Guest29634 | 21:32 | |
| *** JamesBenson has quit IRC | 21:36 | |
| *** lbragstad has quit IRC | 21:37 | |
| *** lbragstad has joined #openstack-nova | 21:37 | |
| *** eharney has joined #openstack-nova | 21:39 | |
| *** luksky has quit IRC | 21:41 | |
| *** markvoelker has joined #openstack-nova | 21:48 | |
| *** xek has quit IRC | 21:51 | |
| *** eharney has quit IRC | 21:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add evacuate vs rebuild contributor doc https://review.opendev.org/683245 | 21:56 |
| mriedem | efried_pto: you asked for it ^ dansmith your review on that would also be helpful due to your blog post | 21:57 |
| mriedem | *the* blog post | 21:57 |
| *** BjoernT_ has quit IRC | 22:00 | |
| dansmith | mriedem: https://www.youtube.com/watch?v=gVpzrxjiAtc | 22:00 |
| mriedem | coincidentally i'm wearing that same outfit | 22:01 |
| mriedem | just went to the scarf store last night | 22:01 |
| *** munimeha1 has quit IRC | 22:01 | |
| dansmith | I don't doubt it | 22:01 |
| dansmith | coincidentally, I'm wearing nothing *but* that scarf | 22:01 |
| *** mmethot_ has quit IRC | 22:02 | |
| mriedem | man, i can't imagine the amount of coke flowing through these guys in this video | 22:02 |
| *** mmethot_ has joined #openstack-nova | 22:02 | |
| dansmith | yeah keith looks like he hasn't slept in two weeks | 22:03 |
| *** cfriesen has joined #openstack-nova | 22:14 | |
| *** brault has quit IRC | 22:22 | |
| *** macz has quit IRC | 22:25 | |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341 | 22:28 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | 22:28 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: WIP: Provider Config File: Merge provider configs to provider tree https://review.opendev.org/676522 | 22:28 |
| *** markvoelker has quit IRC | 22:32 | |
| *** mriedem is now known as mriedem_away | 22:41 | |
| *** igordc has quit IRC | 22:44 | |
| *** macz has joined #openstack-nova | 22:52 | |
| *** macz has quit IRC | 22:54 | |
| *** macz has joined #openstack-nova | 22:54 | |
| *** igordc has joined #openstack-nova | 23:01 | |
| *** tkajinam has joined #openstack-nova | 23:02 | |
| *** TxGirlGeek has quit IRC | 23:08 | |
| *** TxGirlGeek has joined #openstack-nova | 23:08 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | 23:15 |
| *** alex_xu has joined #openstack-nova | 23:23 | |
| *** rcernin has joined #openstack-nova | 23:25 | |
| *** takashin has joined #openstack-nova | 23:37 | |
| *** JamesBenson has joined #openstack-nova | 23:47 | |
| *** artom has joined #openstack-nova | 23:48 | |
| *** JamesBenson has quit IRC | 23:51 | |
| *** macz has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!