| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-10 | |||
| 19:53:11 | dansmith | mriedem: maybe you'll be more careful with this one | |
| 19:53:26 | mriedem | ? | |
| 19:53:37 | dansmith | you lost the last one I gave you | |
| 19:54:19 | mriedem | i'm pretty sure this is the last time i have to rebase this series | |
| 19:56:57 | efried | Do we cover revert_resize in any of our CI jobs? | |
| 19:57:20 | mriedem | yes | |
| 19:57:58 | mriedem | definitely single node, the trickier is multi-node coverage | |
| 19:58:13 | efried | I'm looking at https://review.opendev.org/#/c/665138/ and not finding out a way to tell whether it's doing... anything at all. | |
| 19:58:54 | mriedem | http://logs.openstack.org/39/669539/1/check/tempest-full-py3/1555a09/job-output.txt.gz#_2019-07-05_22_24_52_926869 | |
| 19:58:56 | artom | efried, you stay away from that stuff. Run, I tell you. RUN! | |
| 19:59:20 | artom | efried, you could try using https://review.opendev.org/#/c/664442/ somehow | |
| 19:59:34 | sean-k-mooney | efried: for what its worth i have poposed that we dont backfil the libvirt pos feature and instead use any bandwith it frees up to work on things like numa in placment or novas use of placement in general | |
| 19:59:51 | efried | sean-k-mooney: proposed internally at RH you mean? | |
| 19:59:58 | sean-k-mooney | ya | |
| 20:00:01 | efried | that's wonderful. | |
| 20:00:27 | efried | I would really like to see us focus on moving forward that way rather than adding to the burden of stuff that helps us stay where we are. | |
| 20:00:38 | sean-k-mooney | we will see if that helps or not but my intent at least is to finish off the work i have started and then see what time is left in the cycle | |
| 20:00:53 | mriedem | efried: the tempest-slow job has a cold migration test that is doing revert resize but it's skipped http://logs.openstack.org/39/669539/1/check/tempest-slow-py3/cd3cb0e/job-output.txt.gz#_2019-07-05_22_44_58_146668 until sean-k-mooney/artom's fix lands... | |
| 20:00:57 | mriedem | so a catch-22 | |
| 20:01:25 | mriedem | https://review.opendev.org/#/c/663405/ | |
| 20:02:09 | sean-k-mooney | mriedem: dansmith merged your change to add teh iptables job to the experimenatl pipline right so that can be used for testing revert for now | |
| 20:02:17 | mriedem | but yeah https://review.opendev.org/#/c/664442/ is the canary | |
| 20:02:24 | mriedem | sean-k-mooney: oh yeah that's right | |
| 20:02:39 | artom | sean-k-mooney, I thought that was single node? | |
| 20:02:41 | mriedem | see? there are only like 5 need-to-know manual on-demand things | |
| 20:03:03 | artom | (That being the reason it failed - we didn't test same host revert with OVS hybrid plug) | |
| 20:03:19 | mriedem | artom: it is | |
| 20:03:39 | mriedem | so neutron-tempest-iptables_hybrid tests ovs hybrid plug same-host resize, | |
| 20:03:52 | mriedem | tempest-slow (will eventually) test multi-node revert | |
| 20:03:56 | mriedem | but not ovs hybrid plug | |
| 20:04:04 | artom | We have same-host resize in tempest-full-py3 too, no? | |
| 20:04:07 | mriedem | and tempest-full tests non-hybrid plug same host resize revert | |
| 20:04:11 | efried | So I could update https://review.opendev.org/#/c/665138/ to depend-on https://review.opendev.org/#/c/664442/ and that would tell me... what? | |
| 20:04:12 | artom | jynx | |
| 20:04:12 | mriedem | yes, but not ovs hybrid plug | |
| 20:04:14 | mriedem | and single node | |
| 20:04:16 | sean-k-mooney | artom: the iptables one is singel node yes | |
| 20:04:33 | artom | My brain is leaking | |
| 20:04:37 | efried | I'm probably too dense today to understand it anyway; maybe I'll just push it to the back of the queue again until I'm feeling sharp. | |
| 20:04:40 | mriedem | and the dnm patch at the top of the series for multi-node ovs hybrid plug=true | |
| 20:04:47 | sean-k-mooney | so same host resize revert can be tested which is basically cold migrate | |
| 20:04:54 | mriedem | efried: you're likely best leaving it to me and dansmith at this point | |
| 20:05:05 | efried | okay | |
| 20:05:11 | dansmith | save. yourself. | |
| 20:05:12 | mriedem | sean-k-mooney: .... we don't test same-host cold migration in the gate | |
| 20:05:14 | sean-k-mooney | technically we might allow same host "migrate" too but never bothered to try that | |
| 20:05:16 | artom | dansmith, that's what I said! | |
| 20:05:17 | mriedem | b/c the libvirt driver doesn't support that | |
| 20:05:29 | mriedem | sean-k-mooney: only the vcenter driver allows that | |
| 20:05:36 | artom | But noooo, we need "quality" and for things to "work" | |
| 20:05:36 | sean-k-mooney | mriedem: well same host cold migrate isnt really useful | |
| 20:05:48 | sean-k-mooney | wait they allow it because its a clustered hypervior | |
| 20:05:52 | mriedem | yes | |
| 20:05:58 | sean-k-mooney | so im guessing its not really the same host right | |
| 20:06:02 | sean-k-mooney | at least internally | |
| 20:06:05 | mriedem | different esxi host | |
| 20:06:15 | sean-k-mooney | right of course it is | |
| 20:06:17 | mriedem | same nova-compute service host | |
| 20:06:44 | mriedem | wonder if i could get some questions onto the openstack COA exam | |
| 20:07:04 | mriedem | "what is a compute node?" BZZZT! | |
| 20:08:42 | sean-k-mooney | when is disk_available_least bigger then free_gb | |
| 20:08:55 | efried | a) the same as a compute host | |
| 20:08:55 | efried | b) the physical system on which the nova-compute service runs | |
| 20:08:55 | efried | c) an ironic instance | |
| 20:08:55 | efried | d) all of the above | |
| 20:09:02 | dansmith | please, you guys are depressing me | |
| 20:09:21 | efried | (btw, I would totally fail that question) | |
| 20:11:20 | efried | mriedem: backporting... | |
| 20:11:36 | openstackgerrit | Eric Fried proposed openstack/nova stable/stein: Perf: Use dicts for ProviderTree roots https://review.opendev.org/670179 | |
| 20:12:02 | efried | I think the rocky backport is going to be dirty, because the mdev stuff I "fixed" wasn't there yet. | |
| 20:12:35 | efried | yup | |
| 20:14:06 | efried | oh, worse than that | |
| 20:15:36 | mriedem | unrelated, but while looking at ironic n-cpu job logs for that change on master, i see we hit a provider inventory conflict and we must refresh and re-push and it works | |
| 20:15:37 | mriedem | http://logs.openstack.org/25/637225/2/check/ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa/fee37e3/controller/logs/screen-n-cpu.txt.gz#_Jul_10_00_32_51_413640 | |
| 20:15:43 | mriedem | b/c the reserved value changed on the node | |
| 20:15:58 | mriedem | only question is, does that 409 need to be an error if we're going to refresh associations and try again? | |
| 20:16:12 | mriedem | like, "ERROR! .... oh nvm i've got this." | |
| 20:17:34 | efried | it's the kind of thing where we would sure want the last one (when we give up retrying) to be an error. | |
| 20:17:49 | efried | but yeah, that one could certainly be warning or even info | |
| 20:22:48 | openstackgerrit | Eric Fried proposed openstack/nova stable/rocky: Perf: Use dicts for ProviderTree roots https://review.opendev.org/670182 | |
| 20:23:47 | efried | mriedem: ^ | |
| 20:23:57 | efried | conflicts pare it down to just one file. | |
| 20:24:05 | efried | Should we go to queens? | |
| 20:25:47 | mriedem | belmiro's bug was against rocky b/c that's what cern was using at the time, | |
| 20:25:58 | mriedem | i'd say maybe wait for someone to ask for it in queens | |
| 20:26:53 | efried | ight | |
| 21:01:03 | openstackgerrit | sean mooney proposed openstack/nova master: DNM demonstrate that get_domain_capabilities is broken. https://review.opendev.org/670189 | |
| 21:02:21 | sean-k-mooney | efried: aspiers ^ im going rework get_domain_capabilities function so that it does not explode at runtime | |
| 21:15:43 | efried | neat | |
| 21:23:24 | sean-k-mooney | the test case is a bit contrived as i would otherwise be assert that if i trow an excpetion that it thing explode | |
| 21:23:51 | sean-k-mooney | but hopefully its clear that there reason it was passing wa we were hardcoding stuff | |
| 21:24:01 | sean-k-mooney | and we dont call it | |
| 21:26:54 | openstackgerrit | Merged openstack/nova master: docs: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164 | |
| 21:28:14 | sean-k-mooney | im going to change the api of _get_domain_capabilities to match libvirt which allows all args to be set to none so we dont need to invent a default if we dont have one and can just use libvirts instead | |
| 21:36:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert nova-next to a zuul v3 job https://review.opendev.org/670196 | |
| 21:44:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert nova-next to a zuul v3 job https://review.opendev.org/670196 | |
| 22:48:57 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664 | |
| 22:48:57 | openstackgerrit | Eric Fried proposed openstack/nova master: Use OpenStack SDK for placement https://review.opendev.org/656023 | |
| 22:58:26 | openstackgerrit | Merged openstack/nova master: nova-lvm: Disable [validation]/run_validation in tempest.conf https://review.opendev.org/662176 | |
| 23:17:05 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: harden get_domain_capabilities https://review.opendev.org/670189 | |