Tuesday, 2019-03-26

« Monday, 2019-03-25 Index Wednesday, 2019-03-27 »
*** igordc has quit IRC00:02
*** gyee has quit IRC00:13
*** mdbooth_ has joined #openstack-nova00:16
*** mdbooth has quit IRC00:17
*** lbragstad has quit IRC00:18
*** ivve has quit IRC00:27
*** mdbooth_ has quit IRC00:27
*** tosky has quit IRC00:27
*** luksky has quit IRC00:36
*** dustinc|away has quit IRC00:56
*** hongbin has quit IRC01:00
*** tosky has joined #openstack-nova01:01
*** mdbooth has joined #openstack-nova01:03
*** awalende has joined #openstack-nova01:04
*** tosky has quit IRC01:06
*** awalende has quit IRC01:08
*** takashin has joined #openstack-nova01:18
*** tiendc has joined #openstack-nova01:19
*** wolverineav has quit IRC01:21
*** wolverineav has joined #openstack-nova01:22
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens  https://review.openstack.org/63771601:23
*** wolverineav has quit IRC01:23
*** wolverineav has joined #openstack-nova01:23
*** wolverineav has quit IRC01:27
*** wolverineav has joined #openstack-nova01:28
*** wolverineav has quit IRC01:33
*** mdbooth_ has joined #openstack-nova01:40
*** mdbooth has quit IRC01:42
*** whoami-rajat has joined #openstack-nova01:57
*** erlon has quit IRC02:11
*** BjoernT has joined #openstack-nova02:16
openstackgerritTakashi NATSUME proposed openstack/nova stable/stein: Override the 'get' method in DriverBlockDevice class  https://review.openstack.org/64764602:20
*** BjoernT has quit IRC02:21
*** lbragstad has joined #openstack-nova02:29
*** hongbin has joined #openstack-nova02:34
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a live migration regression test  https://review.openstack.org/64120002:50
*** cfriesen has quit IRC03:01
*** gbarros has joined #openstack-nova03:13
openstackgerritTakashi NATSUME proposed openstack/nova stable/rocky: Override the 'get' method in DriverBlockDevice class  https://review.openstack.org/64765103:14
*** igordc has joined #openstack-nova03:30
*** wolverineav has joined #openstack-nova03:42
*** gbarros has quit IRC03:43
*** igordc has quit IRC03:54
*** takashin has left #openstack-nova03:56
*** udesale has joined #openstack-nova03:58
*** janki has joined #openstack-nova03:59
openstackgerritMerged openstack/nova stable/pike: Handle tags in _bury_in_cell0  https://review.openstack.org/62482504:02
*** hongbin has quit IRC04:10
*** ratailor has joined #openstack-nova04:14
*** janki has quit IRC04:16
*** janki has joined #openstack-nova04:17
openstackgerritMerged openstack/nova stable/pike: Exclude build request marker from server listing  https://review.openstack.org/62895404:17
openstackgerritMerged openstack/nova stable/pike: Add functional regressions tests for server_group_members OverQuota  https://review.openstack.org/58186604:17
openstackgerritMerged openstack/nova stable/pike: Fix server_group_members quota check  https://review.openstack.org/58186704:17
openstackgerritMerged openstack/nova stable/pike: Correct examples in "Manage Compute services" documentation  https://review.openstack.org/63847504:17
openstackgerritMerged openstack/nova stable/pike: Update port device_owner when unshelving  https://review.openstack.org/62640904:17
openstackgerritMerged openstack/nova stable/pike: Fix disk size during live migration with disk over-commit  https://review.openstack.org/53074304:17
openstackgerritMerged openstack/nova stable/pike: Refix disk size during live migration with disk over-commit  https://review.openstack.org/63137204:18
*** sridharg has joined #openstack-nova04:27
*** lbragstad has quit IRC04:34
*** wolverineav has quit IRC04:45
*** tkajinam_ has joined #openstack-nova05:06
*** tkajinam has quit IRC05:08
*** markvoelker has joined #openstack-nova05:17
openstackgerritsean mooney proposed openstack/nova-specs master: re add numa aware live migration for train  https://review.openstack.org/64758105:41
openstackgerritsean mooney proposed openstack/nova-specs master: forward port sriov live migration spec to train  https://review.openstack.org/64758205:49
*** ileixe has joined #openstack-nova06:06
ileixeHi, guys.06:06
ileixeI have an another simple question operating Nova :)06:07
ileixeIs there any way to use flavor with diffrent scope (all / project scope) like 'community' scope in glance?06:07
ileixes/(all / project scope)/(not all / project project)06:08
ileixeGetting access per every projects for flavor  is not fun06:09
ileixeT_T06:09
openstackgerritMerged openstack/nova stable/pike: Null out instance.availability_zone on shelve offload  https://review.openstack.org/60616106:19
*** wolverineav has joined #openstack-nova06:25
*** wolverineav has quit IRC06:33
openstackgerritMerged openstack/nova master: Use Selection object to fill request group mapping  https://review.openstack.org/63871106:42
openstackgerritMerged openstack/nova stable/ocata: Handle binding_failed vif plug errors on compute restart  https://review.openstack.org/62636906:42
openstackgerritMerged openstack/nova stable/ocata: Handle unbound vif plug errors on compute restart  https://review.openstack.org/62655606:43
openstackgerritMerged openstack/nova stable/pike: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64668406:43
*** Luzi has joined #openstack-nova06:43
*** ivve has joined #openstack-nova06:58
*** tkajinam__ has joined #openstack-nova07:05
*** wolverineav has joined #openstack-nova07:06
*** tkajinam_ has quit IRC07:07
*** kaisers has joined #openstack-nova07:09
*** pcaruana has joined #openstack-nova07:10
*** slaweq has quit IRC07:15
*** slaweq has joined #openstack-nova07:17
openstackgerritMerged openstack/nova stable/ocata: Ensure rbd auth fallback uses matching credentials  https://review.openstack.org/62701207:37
*** wolverineav has quit IRC07:39
openstackgerritBoxiang Zhu proposed openstack/nova master: Remove unsafe convert of lvm imagebackend  https://review.openstack.org/64266707:40
openstackgerritBoxiang Zhu proposed openstack/python-novaclient master: [WIP] Add force_scheduler in boot instance  https://review.openstack.org/64767107:54
openstackgerritMerged openstack/nova stable/queens: Document unset/reset wrinkle for *_allocation_ratio options  https://review.openstack.org/64729107:55
*** luksky has joined #openstack-nova07:57
*** ccamacho has joined #openstack-nova08:03
*** rpittau|afk is now known as rpittau08:06
*** wolverineav has joined #openstack-nova08:06
openstackgerritEric Fried proposed openstack/nova-specs master: re add numa aware live migration for train  https://review.openstack.org/64758108:10
*** awalende has joined #openstack-nova08:11
*** tosky has joined #openstack-nova08:14
*** tesseract has joined #openstack-nova08:16
openstackgerritEric Fried proposed openstack/nova-specs master: forward port sriov live migration spec to train  https://review.openstack.org/64758208:21
openstackgerritEric Fried proposed openstack/nova-specs master: forward port sriov live migration spec to train  https://review.openstack.org/64758208:22
*** xek has joined #openstack-nova08:23
*** helenafm has joined #openstack-nova08:23
openstackgerritMerged openstack/nova master: Re-enable testing of console with TLS in nova-next job  https://review.openstack.org/64543208:24
openstackgerritMerged openstack/nova master: Add description about sort order in API ref guideline  https://review.openstack.org/62728208:24
*** tkajinam__ has quit IRC08:25
*** tssurya has joined #openstack-nova08:28
*** ttsiouts has joined #openstack-nova08:30
openstackgerritMerged openstack/nova-specs master: re add numa aware live migration for train  https://review.openstack.org/64758108:30
*** dtantsur|afk is now known as dtantsur08:33
openstackgerritMerged openstack/nova master: Override the 'get' method in DriverBlockDevice class  https://review.openstack.org/63882108:34
*** rcernin has quit IRC08:36
openstackgerritBoxiang Zhu proposed openstack/nova master: [WIP] Scheduler filters evaluated even forced host  https://review.openstack.org/64552008:37
*** wolverineav has quit IRC08:40
openstackgerritMerged openstack/nova-specs master: forward port sriov live migration spec to train  https://review.openstack.org/64758208:41
*** ttsiouts has quit IRC08:43
*** ttsiouts has joined #openstack-nova08:43
*** shilpasd has quit IRC08:45
*** ttsiouts has quit IRC08:47
*** ralonsoh has joined #openstack-nova08:54
*** tetsuro has joined #openstack-nova08:57
*** priteau has joined #openstack-nova08:58
*** priteau has quit IRC09:01
*** ttsiouts has joined #openstack-nova09:02
*** priteau has joined #openstack-nova09:02
*** mdbooth has joined #openstack-nova09:19
*** mdbooth_ has quit IRC09:21
*** shilpasd has joined #openstack-nova09:28
*** IvensZambrano has joined #openstack-nova09:30
*** luksky has quit IRC09:35
*** jangutter has quit IRC09:37
*** wolverineav has joined #openstack-nova09:37
*** rcernin has joined #openstack-nova09:45
*** Luzi has quit IRC09:52
*** ccamacho has quit IRC09:53
*** ccamacho has joined #openstack-nova10:04
*** zhubx has quit IRC10:06
*** zhubx has joined #openstack-nova10:07
*** zhubx has quit IRC10:08
*** zhubx has joined #openstack-nova10:08
*** zhubx has quit IRC10:09
*** wolverineav has quit IRC10:09
*** zhubx has joined #openstack-nova10:10
*** jangutter has joined #openstack-nova10:21
*** rcernin has quit IRC10:24
*** xek_ has joined #openstack-nova10:37
*** tbachman has quit IRC10:39
*** xek has quit IRC10:39
*** cdent has joined #openstack-nova10:42
openstackgerritSylvain Bauza proposed openstack/nova master: Add doc on VGPU allocs and inventories for nrp  https://review.openstack.org/64751910:45
*** psachin has joined #openstack-nova10:50
*** owalsh has quit IRC10:59
*** luksky has joined #openstack-nova11:02
*** owalsh has joined #openstack-nova11:04
*** wolverineav has joined #openstack-nova11:07
openstackgerritElod Illes proposed openstack/nova stable/stein: Use Selection object to fill request group mapping  https://review.openstack.org/64771311:10
openstackgerritMerged openstack/nova master: Add functional regression test for bug 1669054  https://review.openstack.org/64600611:11
openstackbug 1669054 in OpenStack Compute (nova) "RequestSpec.ignore_hosts from resize is reused in subsequent evacuate" [Medium,In progress] https://launchpad.net/bugs/1669054 - Assigned to Matt Riedemann (mriedem)11:11
*** udesale has quit IRC11:16
*** tetsuro has quit IRC11:19
*** ileixe has quit IRC11:20
*** mvkr has joined #openstack-nova11:22
*** cdent has quit IRC11:22
*** cdent has joined #openstack-nova11:23
*** priteau has quit IRC11:29
*** derekh has joined #openstack-nova11:39
*** wolverineav has quit IRC11:40
*** tiendc has quit IRC11:41
openstackgerritMerged openstack/nova master: Remove last use of rc_fields  https://review.openstack.org/64599111:46
*** ratailor has quit IRC11:49
*** ratailor has joined #openstack-nova11:50
*** ttsiouts has quit IRC12:00
*** ttsiouts has joined #openstack-nova12:00
*** ttsiouts has quit IRC12:05
*** tbachman has joined #openstack-nova12:15
*** ttsiouts has joined #openstack-nova12:26
*** markvoelker has quit IRC12:31
*** liuyulong has joined #openstack-nova12:32
*** wolverineav has joined #openstack-nova12:37
*** mchlumsky has joined #openstack-nova12:43
*** janki has quit IRC12:45
*** janki has joined #openstack-nova12:45
*** lbragstad has joined #openstack-nova12:46
*** janki has quit IRC12:46
*** janki has joined #openstack-nova12:46
*** baclawski has quit IRC12:50
*** jmlowe has quit IRC12:52
*** udesale has joined #openstack-nova13:00
*** awalende has quit IRC13:01
*** awalende has joined #openstack-nova13:01
*** READ10 has joined #openstack-nova13:02
*** awalende has quit IRC13:05
*** mriedem has joined #openstack-nova13:09
*** wolverineav has quit IRC13:11
*** dpawlik has quit IRC13:12
*** Luzi has joined #openstack-nova13:15
openstackgerritsean mooney proposed openstack/nova master: extend libvirt video model support  https://review.openstack.org/64773313:15
*** aluria has joined #openstack-nova13:16
*** jistr is now known as jistr|afk13:16
mriedembauzas: want to tweak this quick? https://review.openstack.org/#/c/647519/13:17
*** ratailor has quit IRC13:19
mriedemtonyb: is it ok to land the stable/stein upper-constraints change even if we end up doing an rc2 https://review.openstack.org/#/c/645462/ ?13:23
mriedemi would think so13:23
dansmithmriedem: mnaser lyarwood: have we actually got the deployment tools able to deploy and upgrade to new placement yet?13:28
dansmithusually upgrades to new releases don't even start happening until a while after all the projects have cut their final releases, AFAIK, so I would think we haven't seen that yet13:28
mriedemtripleo supports deploy of extracted placement but not yet upgrade from placement-in-nova according to lyarwood's last update13:29
*** awalende has joined #openstack-nova13:30
mriedemkolla supports deploy extracted placement from source / pypi i think13:30
mriedemcdent: ^13:30
mriedemsome of the deployment tools can't deploy from packages yet b/c there aren't packages yet13:30
dansmithright, deploy but not upgrade right?13:30
dansmithright13:30
mriedemas far as i know13:30
cdentyeah, deploy but not upgrade is my understanding as well13:30
cdentI haven't checked in the last week or so, though13:30
dansmithI'm just thinking that this early into train, nothing has changed from the situation a month ago where the deployment tools are the sticking point13:31
cdentis this referencing some earlier discussion? which "we haven't seen that yet" is this?13:32
dansmithjust the placement delete patch13:32
cdentah, okay, I thought perhaps that was it, but wasn't sure13:32
dansmithwe can cut and run and they'll just have to catch up, but it means they'll have to implement that and/or resolve issues before they can get their stein->master upgrades working at all13:34
dansmithso just wondering what progress they have made or how close that is for them13:34
*** awalende has quit IRC13:34
shilpasdmriedem: Hi, going through https://bugs.launchpad.net/nova/+bug/1784020, can you help me to understand more points mentioned here in 2.a and 2.b, want to reproduce and figure out the fixes for it13:34
openstackLaunchpad bug 1784020 in OpenStack Compute (nova) "Shared storage providers are not supported and will break things if used" [Medium,Fix released]13:34
cdentI kinda feel like some sort of forcing function is required, but that may be either unrealistic or rude13:34
*** dpawlik has joined #openstack-nova13:35
dansmithyeah could be, but before the previous release is even out might be a teensy bit aggresive13:35
dansmith(but spelled properly)13:36
dansmiththe deployment projects also receive the forcing-function love from all projects simultaneously, which isn't super fun :)13:36
cdentif we choose to wait, I will keep the patch up to date13:37
cdentsomebody just keep me informed please13:37
*** jmlowe has joined #openstack-nova13:37
mriedemoh i didn't realize there were +2s stacked up on https://review.openstack.org/#/c/618215/13:37
mriedemyeah that's way too aggressive IMO13:38
dansmitheasy for me to say, but it doesn't look like there are too many places that it will be conflict-heavy13:38
dansmithmriedem: yeah, that was my concern13:38
cdentI think only conf and cmd stuff is likely to conflict, if at all13:39
* dansmith nods13:39
mriedemhonestly i'd probably proceduraly -2 that for now13:39
mriedemefried: stephenfin: ^13:39
cdentit was until yesterday13:40
*** dpawlik has quit IRC13:40
stephenfinmriedem: Yup, I wasn't sure so I held off on the +W. Happy for someone -2 that for now, yup13:40
cdentIf this turns into nova-net, I'm going to burn a hole through the earth13:41
dansmithcould we maybe tell the deployment projects that we're going to merge this the week of the PTG, so they should either get it done before then or show up with lots of reasons and gifts if not?13:41
cdentseems a reasonable plan13:41
* stephenfin _would_ rather like a new laptop13:41
*** janki has quit IRC13:42
mriedemi think dropping at the ptg is a good idea b/c there is a forum session on extraction right before that as well13:43
mriedemas a final 'in 2 days we're going to drop this code from nova, any questions?'13:43
dansmithyeah13:43
*** eharney has joined #openstack-nova13:43
*** zhubx has quit IRC13:46
*** zhubx has joined #openstack-nova13:46
*** dpawlik has joined #openstack-nova13:53
*** awaugama has joined #openstack-nova13:56
mnaserdansmith, mriedem, cdent: OSA hasn’t progressed unfortunately.  However, having an official timeline would be nice13:56
mnaserSo I’m supportive of at least knowing when it will be done.13:56
mnaserWe can probably get deploy from extracted easily but orchestrating the upgrade is the hard part13:56
*** ircuser-1 has quit IRC13:57
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: add more details to confirmResize troubleshooting  https://review.openstack.org/64774113:57
*** dpawlik has quit IRC13:58
mriedemefried: there is some wording for the API reference ^13:58
*** mvkr has quit IRC13:59
dansmithmnaser: oh I thought deploy from extracted was already working, but I'm sure I was just confusing that with tripleo or something13:59
*** mlavalle has joined #openstack-nova14:01
mnaserYeah, well, we have an ansible role that deploys placement only. But we have not integrated the role with our playbooks14:01
dansmithah, okay14:01
*** udesale has quit IRC14:03
*** kaisers has quit IRC14:05
*** kaisers has joined #openstack-nova14:06
*** wolverineav has joined #openstack-nova14:07
mriedemmdbooth: if you're busy do you care if i add the unit test for https://review.openstack.org/#/c/645546/ ?14:12
tridentHmm. Is trait:CUSTOM_FOO=forbidden set as flavor property expected to work in stable/rocky? I get instances spun up with flavors with that set ending up on resource providers (compute nodes) with that trait set. The other way around seem to work - a flavor with trait:CUSTOM_FOO=required makes instances with that flavor end up on the resource providers with that trait set.14:14
*** jistr|afk is now known as jistr14:15
mriedemtrident: says it should https://docs.openstack.org/nova/rocky/user/flavors.html14:15
mriedem"Added in the 18.0.0 Rocky release."14:15
mdboothmriedem: I'd be delighted :) I'm currently working on https://bugs.launchpad.net/nova/+bug/182137314:16
openstackLaunchpad bug 1821373 in OpenStack Compute (nova) "Most instance actions can be called concurrently" [Undecided,New]14:16
*** dpawlik has joined #openstack-nova14:17
tridentmriedem: Yeah, that's what I thought as well, so I am a bit surprised. Didn't really find any bugs that seem relevant either.14:17
mriedemmdbooth: ok will do. another thing was https://review.openstack.org/#/c/551349/ just needs some tweaks i think, but it looks like maybe lyarwood is working that now14:17
mriedemunless lyarwood is stuck in tripleo limbo for the next month14:18
openstackgerritSylvain Bauza proposed openstack/nova master: Add doc on VGPU allocs and inventories for nrp  https://review.openstack.org/64751914:18
mdboothmriedem: He's away this week, but he has taken that patch.14:18
mriedemtrident: if it's helpful, here is a functional test that does traits-based scheduling https://github.com/openstack/nova/blob/358776a303b0c5dba3c279f242c82b7a12ff5ce8/nova/tests/functional/test_servers.py#L445114:19
tridentmriedem: Butt still the API call to placement /allocation_candidates doesn't seem to pass the forbidden trait (which if I understand it correctly should be "required=!CUSTOM_FOO" while it does pass it for the flavor with trait required: "required=CUSTOM_FOO"14:20
tridentmriedem: Thanks, I'll keep looking then, just wanted to check that I am not missing anything obvious.14:21
mriedemhmm, looking at the test in there that uses a forbidden trait on a flavor https://github.com/openstack/nova/blob/358776a303b0c5dba3c279f242c82b7a12ff5ce8/nova/tests/functional/test_servers.py#L4824 it doesn't seem like the forbidden trait is actually testing anything useful14:21
mriedemmeaning, a better test would be to have something like 2 computes, one with trait CUSTOM_FOO and one without, and disable the one without so we can't schedule there, then schedule with forbidden CUSTOM_FOO and it should have a NoValidHost result,14:22
mriedembut you're saying it sounds like it might just pass14:22
*** Luzi has quit IRC14:23
tridentmriedem: I'll see what I find continuing the troubleshooting and get back...14:23
mriedemack, this is the code that builds the GET /allocation_candidates query string fwiw https://github.com/openstack/nova/blob/358776a303b0c5dba3c279f242c82b7a12ff5ce8/nova/scheduler/utils.py#L25714:24
*** udesale has joined #openstack-nova14:24
*** xek_ has quit IRC14:24
*** xek_ has joined #openstack-nova14:24
*** mvkr has joined #openstack-nova14:26
*** dpawlik has quit IRC14:27
tridentmriedem: Thanks. that's exactly where I am currently looking :)14:28
*** dpawlik has joined #openstack-nova14:29
*** brandor5 has joined #openstack-nova14:34
*** wolverineav has quit IRC14:40
*** zbr has quit IRC14:47
*** zbr has joined #openstack-nova14:50
*** ccamacho has quit IRC14:51
*** munimeha1 has joined #openstack-nova14:53
*** xek_ has quit IRC14:53
*** cfriesen has joined #openstack-nova14:59
brandor5hello everyone: I'm trying to use hypervisor_hostname as a scheduler hint and it's worked in the past but I'm running into issues now... I get a 400 bad request back... anyone have any ideas why that could be?15:01
openstackgerritMatt Riedemann proposed openstack/nova master: Fix incomplete instance data returned after build failure  https://review.openstack.org/64554615:02
*** jmlowe has quit IRC15:02
mriedemmdbooth: done ^15:02
mdboothmriedem: Thanks! You ought to add a co-authored line, esp considering the tests are longer than the fix :)15:03
*** gbarros has joined #openstack-nova15:04
openstackgerritMatt Riedemann proposed openstack/nova master: Fix incomplete instance data returned after build failure  https://review.openstack.org/64554615:04
mriedemmelwitt: want to hit ^ when you get a chance?15:06
mriedemgoes back to pike15:06
mriedembrandor5: would be helpful if you shared the actual 400 error message15:06
brandor5mriedem: I'm trying to dig it out, 1 second :)15:07
*** jmlowe has joined #openstack-nova15:07
mriedemscheduler hints are open-ended for the most part, and hypervisor_hostname is not a known hint https://github.com/openstack/nova/blob/358776a303b0c5dba3c279f242c82b7a12ff5ce8/nova/api/openstack/compute/schemas/servers.py#L9315:07
mriedemso i'm assuming you're using some out of tree scheduler filter for this hint?15:08
brandor5http://sprunge.us/9pGzX915:08
mriedemlike forcing to build on a specific baremetal node?15:08
brandor5exactly15:08
mriedemthat doesn't give the error message15:08
brandor5query: '["=","$hypervisor_hostname","b23bbc4a-9a43-4ed8-b028-43d2aca3a6d7"]'15:08
brandor5that's the only 400 I see anywhere15:09
*** jmlowe has quit IRC15:09
brandor5when i'm running the command15:09
mriedemyou don't get an error message on the client side?15:09
brandor5ansible returns:  msg: 'BadRequestException: 400'15:09
brandor5if I remove the filter from the call it works fine15:09
mriedemthere is nothing else in the logs for req-f5f66dc0-8035-4c07-a631-a438fb79ba42 ?15:09
mriedemso you're hitting something with this https://github.com/openstack/nova/blob/358776a303b0c5dba3c279f242c82b7a12ff5ce8/nova/api/openstack/compute/schemas/servers.py#L12515:10
mriedemi want to say i've heard this reported bfore15:10
mriedem*before15:10
brandor5nothing else found in the logs with that request15:11
brandor5the 400 would be coming from nova, correct?15:11
brandor5I shouldn't be looking down in ironic?15:11
mriedemit's coming from nova15:11
brandor5that's what I thought15:11
mdboothIs it worth writing new code whose only consumer would be cellsv1?15:12
mriedemmdbooth: nope15:12
mdboothSpecifically, it looks like the only user of expected_vm_state is cellsv115:12
mdboothi.e. instance.save(expected_vm_state=FOO)15:13
mriedemprobably because cells v1 was racy15:13
mriedemsyncing state between child and parent cell15:13
mdboothOk. It'll look a bit odd to add new expected_task_state code without expected_vm_state code, though, so I'll add a comment15:13
mriedembrandor5: i don't know what would be returning that 400 except the json schema validation code, but that query hint is just marked as a string or object, and you're passing a json string15:15
*** awaugama has quit IRC15:15
mriedemthe actual query hint isn't used until we hit the JsonFilter scheduler filter, which is long after you'd get that 40015:16
mriedemso my guess is the request schema validation is somehow puking15:16
brandor5ok, i'll keep digging and will report back if I figure it out15:16
mriedemi very much doubt we have any functional testing for JsonFilter15:17
mriedembrandor5: i think the last person that had this problem just used availability_zone=nova::b23bbc4a-9a43-4ed8-b028-43d2aca3a6d715:17
mriedemthe only problem with that is it bypasses the filters and forces to that node15:17
brandor5that would be okay in this case15:17
mriedemhttps://docs.openstack.org/nova/latest/admin/availability-zones.html15:18
brandor5so I'll try that if the thing im about to test doesn't work15:18
mriedemthen force away15:18
brandor5mriedem: it turns out it was a bug in ansible, updating to latest and I don't see that issue now15:23
*** janki has joined #openstack-nova15:23
*** jmlowe has joined #openstack-nova15:25
mriedemhuh15:26
mriedemwell you made me find our docs are busted at least :) https://bugs.launchpad.net/nova/+bug/182176415:26
openstackLaunchpad bug 1821764 in OpenStack Compute (nova) "docs: JsonFilter query hint examples do not use valid json strings" [Medium,Triaged]15:26
brandor5yeah, weird15:26
brandor5hehe glad to be of assistance :D15:26
sean-k-mooneymriedem: [">=", "$free_ram_mb", 1024] is a vlaid json array15:28
mriedemsean-k-mooney: did you read the bug?15:29
sean-k-mooneyim just looking at it but '[">=", "$free_ram_mb", 1024]' is not the same thing as json.loads("[>=,$free_ram_mb,1024]")15:30
mriedemserenity now15:30
openstackgerritMerged openstack/nova stable/pike: Fix WeighedHost logging regression  https://review.openstack.org/64139815:32
sean-k-mooneyoh you are saying its correct in the schduler doc but not in the create server example15:33
mriedemi don't see how what i wrote in the bug report is unclear15:34
sean-k-mooneyi was confused by the fact that it was correct in the osc command but not in the api example in the json filter doc15:36
*** wolverineav has joined #openstack-nova15:37
sean-k-mooneyi intially looked at it and didnt see teh api example was wrong as my brain just filled in teh missing quots ared the args15:37
sean-k-mooneyits why im terible are prof readign my own stuff if i read two things that are almost identical and expect them to be the same i have troble seeign the difference15:38
openstackgerritMatt Riedemann proposed openstack/nova master: Fix JsonFilter query hint examples in docs  https://review.openstack.org/64777815:38
*** hemna has quit IRC15:40
*** hemna has joined #openstack-nova15:42
mriedemooo dansmith i could use the JsonFilter to schedule directly to a cell, that's fun15:42
sean-k-mooneymriedem: you can you the json filter to do a lot of horribel things15:43
*** hongbin has joined #openstack-nova15:43
dansmithmriedem: you can probably use it to do all kinds of things15:43
*** Sundar has joined #openstack-nova15:45
sean-k-mooneyas far as i rememebr it give you acess to any value contined in the hoststate object15:45
mriedemcorrect15:46
mriedemima add a warning in the docs15:48
sean-k-mooneythe main usecase i was aware of was shduing based in the cpu info or hypervior type.15:49
mriedemi think the ImagePropertiesFilter already handles hv type15:49
mriedemand cpu info should be solved with traits15:49
sean-k-mooneyya it does15:49
sean-k-mooneythe json filter is really a filter that we shoudl be able to remove in the near future but some peopel still use it. i think oath have an internal version too.15:50
*** mrch_ has quit IRC15:56
mdboothHmm, my grand plan to fix instance.save() is scuppered because I forgot we don't have access to the old values of fields which have changed but not saved.15:57
*** gbarros has quit IRC15:58
*** liuyulong is now known as liuyulong_zzzZ15:59
sean-k-mooneymdbooth: can you not just always save all changed field when we call save and simply mark any filed we write to as chaged ignoring if the value is the same or not16:01
mdboothsean-k-mooney: No, because we've already lost the old value before we called save()16:02
mdboothsean-k-mooney: There will be another way.16:03
melwittmriedem: can do16:06
*** mrch_ has joined #openstack-nova16:06
*** cdent has quit IRC16:07
*** Sundar has quit IRC16:10
*** wolverineav has quit IRC16:10
openstackgerritMatt Riedemann proposed openstack/nova master: Fix JsonFilter query hint examples in docs  https://review.openstack.org/64777816:14
openstackgerritMatt Riedemann proposed openstack/nova master: Document a warning about using the JsonFilter  https://review.openstack.org/64779616:14
*** psachin has quit IRC16:20
*** jmlowe has quit IRC16:21
*** imacdonn has quit IRC16:22
*** imacdonn has joined #openstack-nova16:22
*** gyee has joined #openstack-nova16:25
*** gbarros has joined #openstack-nova16:26
openstackgerritBalazs Gibizer proposed openstack/nova master: Add flavor to requested_resources in RequestSpec  https://review.openstack.org/64739616:27
*** eharney has quit IRC16:49
*** ttsiouts has quit IRC16:49
*** ttsiouts has joined #openstack-nova16:49
*** gbarros has quit IRC16:52
*** tssurya has quit IRC16:53
*** ttsiouts has quit IRC16:54
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for the JsonFilter  https://review.openstack.org/64781116:57
*** luksky has quit IRC16:57
*** mriedem has quit IRC16:57
*** tjgresha has quit IRC16:58
*** rpittau is now known as rpittau|afk16:59
*** gbarros has joined #openstack-nova17:01
*** brandor5 has left #openstack-nova17:04
*** janki has quit IRC17:09
*** tesseract has quit IRC17:13
*** igordc has joined #openstack-nova17:13
*** igordc has quit IRC17:13
*** igordc has joined #openstack-nova17:13
*** wolverineav has joined #openstack-nova17:15
*** gbarros has quit IRC17:22
openstackgerritStephen Finucane proposed openstack/nova stable/queens: [Stable Only] hardware: Handle races during pinning  https://review.openstack.org/64783117:22
*** gbarros has joined #openstack-nova17:25
*** zbr|ssbarnea has joined #openstack-nova17:29
*** zbr has quit IRC17:31
*** janki has joined #openstack-nova17:33
*** tbachman has quit IRC17:34
*** Sundar has joined #openstack-nova17:37
*** ivve has quit IRC17:39
*** udesale has quit IRC17:40
*** wolverineav has quit IRC17:44
*** eharney has joined #openstack-nova17:47
*** janki has quit IRC17:49
*** mvkr has quit IRC17:50
*** jmlowe has joined #openstack-nova17:51
*** mvkr has joined #openstack-nova17:53
*** ttsiouts has joined #openstack-nova17:54
*** tbachman has joined #openstack-nova17:55
*** helenafm has quit IRC17:59
*** manjeets_ has joined #openstack-nova18:01
*** manjeets_ has quit IRC18:02
*** wolverineav has joined #openstack-nova18:02
*** wolverineav has quit IRC18:04
*** wolverineav has joined #openstack-nova18:04
*** dtantsur is now known as dtantsur|afk18:10
*** wolverineav has quit IRC18:14
openstackgerritEric Fried proposed openstack/nova master: PrivsepFixture so sys_admin_pctxt methods can be tested  https://review.openstack.org/64784918:14
*** ttsiouts has quit IRC18:17
*** wolverineav has joined #openstack-nova18:17
*** ttsiouts has joined #openstack-nova18:17
openstackgerritmelanie witt proposed openstack/nova master: Add a reference PTL guide to the contributor docs  https://review.openstack.org/64785418:20
*** ttsiouts has quit IRC18:22
*** jmlowe has quit IRC18:23
*** derekh has quit IRC18:25
*** READ10 has quit IRC18:25
*** jmlowe has joined #openstack-nova18:29
*** gbarros has quit IRC18:34
*** gbarros has joined #openstack-nova18:39
*** jmlowe has quit IRC18:39
*** gbarros has quit IRC18:41
efriedmelwitt: What's an "RPC version alias"?18:41
dansmithefried: the per-release aliases for whatever version was current18:43
*** gbarros has joined #openstack-nova18:43
dansmithso you can say "compat=queens" when upgrading to rocky18:43
*** jmlowe has joined #openstack-nova18:43
dansmithinstead of compat=5.2 or something18:43
efrieddansmith: is this a thing that's set in code, or a git tag, or...18:45
dansmithefried: it's in compute/rpcapi.py18:45
efriedgot it, thanks.18:46
dansmithyou want to do those as late and close to release as possible in case we had to add an rpc version for a critical rc9 bug18:46
dansmiththey can be changed of course, but ideally they wouldn't18:46
*** jmlowe has quit IRC18:54
*** wolverineav has quit IRC18:55
*** wolverineav has joined #openstack-nova18:55
*** wolverineav has quit IRC18:58
*** jmlowe has joined #openstack-nova19:03
*** awalende has joined #openstack-nova19:03
*** awalende has quit IRC19:07
*** ttsiouts has joined #openstack-nova19:08
dansmithefried: have we let this sit long enough? https://review.openstack.org/#/c/644625/19:09
efrieddansmith: add my sentence and I'll fast approve it.19:09
*** wolverineav has joined #openstack-nova19:10
dansmithsigh, okay19:10
efriedyou think you're pissed, I just -1'd mikal's privsep series on account of missing unit tests.19:11
dansmithI think I'm done reviewing his patches for a while19:11
*** jmlowe has quit IRC19:11
*** wolverineav has quit IRC19:12
*** wolverin_ has joined #openstack-nova19:12
openstackgerritDan Smith proposed openstack/nova-specs master: Add request-filter-image-types spec  https://review.openstack.org/64462519:13
dansmithefried: ^19:13
dansmithI added "in other words" in front of your sentence as my highly passive-aggressive "saying this again with different words" notation19:13
*** mvkr has quit IRC19:15
openstackgerritEric Fried proposed openstack/nova-specs master: Add request-filter-image-types spec  https://review.openstack.org/64462519:15
efriedI noticed19:16
dansmithlet me have it.19:16
efriedI did19:16
efriedin the literal rather than figurative sense19:16
* dansmith nods19:17
*** ttsiouts has quit IRC19:18
*** ttsiouts has joined #openstack-nova19:19
sean-k-mooneyif people have dansmith's spec still in there mind https://review.openstack.org/#/c/647578/1/specs/train/approved/image-metadata-prefiltering.rst is very similar but different uscase.19:20
dansmithsean-k-mooney: I have it up to look at19:20
*** luksky has joined #openstack-nova19:21
sean-k-mooneydansmith: cool the main open question i have in the spec is realy how the use would formulate the quested that triggers the feature. 3b which is my perfeence i think is basicly the same as you were proposing for the image type19:22
*** tbachman has quit IRC19:23
openstackgerritmelanie witt proposed openstack/nova master: Add a reference PTL guide to the contributor docs  https://review.openstack.org/64785419:23
sean-k-mooney.i.e. they have to do nothing because a prefilter magicaly adds the traits for them19:23
*** ttsiouts has quit IRC19:23
*** jmlowe has joined #openstack-nova19:29
*** ttsiouts has joined #openstack-nova19:33
*** jmlowe has quit IRC19:33
openstackgerritMerged openstack/nova-specs master: Add request-filter-image-types spec  https://review.openstack.org/64462519:37
*** mriedem has joined #openstack-nova19:41
*** avolkov has joined #openstack-nova19:48
*** cdent has joined #openstack-nova19:55
*** panda|ko is now known as panda|off19:55
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Fix incomplete instance data returned after build failure  https://review.openstack.org/64790219:58
*** ttsiouts has quit IRC20:00
*** ttsiouts has joined #openstack-nova20:01
*** ttsiouts has quit IRC20:05
*** jmlowe has joined #openstack-nova20:06
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Fix incomplete instance data returned after build failure  https://review.openstack.org/64791120:06
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Fix incomplete instance data returned after build failure  https://review.openstack.org/64791320:12
*** IvensZambrano has quit IRC20:18
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Fix incomplete instance data returned after build failure  https://review.openstack.org/64791620:19
*** IvensZambrano has joined #openstack-nova20:20
*** BjoernT has joined #openstack-nova20:20
*** wolverin_ has quit IRC20:24
*** wolverineav has joined #openstack-nova20:25
*** tbachman has joined #openstack-nova20:26
*** dustinc has joined #openstack-nova20:27
*** wolverineav has quit IRC20:30
*** wolverineav has joined #openstack-nova20:31
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Update instance.availability_zone during live migration  https://review.openstack.org/64763020:32
*** BjoernT_ has joined #openstack-nova20:33
*** BjoernT has quit IRC20:33
*** mriedem has quit IRC20:36
openstackgerritMerged openstack/nova stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64546220:38
*** BjoernT_ has quit IRC20:38
efriedmdbooth: https://review.openstack.org/#/c/554195/ <== has your colleague sanity-checked this yet?20:38
*** ttsiouts has joined #openstack-nova20:41
*** eharney has quit IRC20:48
*** lbragstad has quit IRC21:01
*** ralonsoh has quit IRC21:02
*** munimeha1 has quit IRC21:05
tonybmriedem: Yup with the new constraints stuff we did it's fine to land that patch as soon as it's made.21:17
tonybmriedem: We do the actual cut over in the releases repo21:18
tonybmriedem: infact it would be possile to so the 'train' chnage today21:18
tonybmriedem: I need to email the list to dicsuss all that.21:18
*** hongbin has quit IRC21:19
*** mdbooth_ has joined #openstack-nova21:19
*** mdbooth has quit IRC21:23
*** pcaruana has quit IRC21:33
*** whoami-rajat has quit IRC21:37
*** IvensZambrano has quit IRC21:38
*** kaiokmo has quit IRC21:39
*** mchlumsky has quit IRC21:43
openstackgerritEric Fried proposed openstack/nova master: WIP: Use aggregate_add_host in nova-manage  https://review.openstack.org/64008021:47
*** avolkov has quit IRC21:55
*** ttsiouts has quit IRC22:07
*** ttsiouts has joined #openstack-nova22:14
*** ttsiouts has quit IRC22:18
*** panda|off has quit IRC22:24
*** ircuser-1 has joined #openstack-nova22:31
*** gbarros has quit IRC22:32
*** gbarros has joined #openstack-nova22:33
*** slaweq has quit IRC22:37
*** gbarros has quit IRC22:37
openstackgerritMerged openstack/nova master: Fix incomplete instance data returned after build failure  https://review.openstack.org/64554622:38
*** rcernin has joined #openstack-nova22:38
openstackgerritEric Fried proposed openstack/nova master: Use aggregate_add_host in nova-manage  https://review.openstack.org/64008022:39
efriedmriedem: this bud's for you ^22:39
*** wolverineav has quit IRC22:45
*** cdent has quit IRC22:45
*** wolverineav has joined #openstack-nova22:48
*** wolverineav has quit IRC22:50
*** wolverineav has joined #openstack-nova22:50
*** tkajinam has joined #openstack-nova22:57
*** tosky has quit IRC23:02
*** wolverineav has quit IRC23:19
*** wolverineav has joined #openstack-nova23:26
*** luksky has quit IRC23:28
*** mlavalle has quit IRC23:30
*** wolverineav has quit IRC23:31
*** wolverineav has joined #openstack-nova23:32
*** gyee has quit IRC23:53
« Monday, 2019-03-25 Index Wednesday, 2019-03-27 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!