Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-10
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 mriedem yes, but not ovs hybrid plug
20:04:12 artom jynx
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 sean-k-mooney mriedem: well same host cold migrate isnt really useful
20:05:36 artom But noooo, we need "quality" and for things to "work"
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 d) all of the above
20:08:55 efried c) an ironic instance
20:08:55 efried b) the physical system on which the nova-compute service runs
20:08:55 efried a) the same as a compute host
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: Use OpenStack SDK for placement https://review.opendev.org/656023
22:48:57 openstackgerrit Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664
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
23:43:09 openstackgerrit Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899
23:43:10 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028
23:43:10 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
23:43:11 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691
23:43:11 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690
23:43:12 openstackgerrit Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for getting network metadata from node https://review.opendev.org/670213
#openstack-nova - 2019-07-11
00:00:12 openstackgerrit Matt Riedemann proposed openstack/nova master: Convert nova-next to a zuul v3 job https://review.opendev.org/670196
00:31:48 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
00:31:48 openstackgerrit Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177
00:31:49 openstackgerrit Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
03:05:25 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support for changing deleted_on_termination after boot https://review.opendev.org/580336
04:03:37 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: doc: Fix a parameter of NotificationPublisher https://review.opendev.org/670225
04:05:32 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: doc: Fix a parameter of NotificationPublisher https://review.opendev.org/670226

Earlier   Later