Posted	Nick	Remark
#openstack-nova - 2018-09-21
20:47:24	mriedem	like how data migrations were done during db sync in the long ago
20:47:34	mriedem	which meant operators could have substantial downtime in the control plane
20:48:39	imacdonn	ok
20:49:28	mriedem	which project are you looking to do the upgrade checks for btw?
20:49:40	imacdonn	so I need to digest the whole upgrade flow a bit more (maybe after I digest lunch) .... but I still think it's worth at least considering pre/post-upgrade checks in general
20:50:21	imacdonn	I'm trying to be a good citizen and 'do the right thing' for all projects .. or all of the ones that I uses, which is mostly the core ones
20:51:40	imacdonn	based on your ML posts a while back, I added 'nova-status upgrade check', but it's not quite right, as I'm running it after starting the services
20:52:15	imacdonn	maybe mostly because currently placement is considered a nova service, at least in my deployment mechanism
20:53:03	imacdonn	I may try hacking in sometihng to start placement individually before doing the upgrade check, then start the rest of the (nova) services
20:53:47	imacdonn	or maybe I'll worry about that for stein
20:54:47	mriedem	"I added 'nova-status upgrade check', but it's not quite right, as I'm running it after starting the services" - as i said, that's also acceptable in the case of nova and it's even in our install docs: https://docs.openstack.org/nova/latest/install/verify.html
20:54:55	mriedem	for a fresh install i mean
20:55:29	mriedem	flexibility and idempotency are kind of the key things here
20:55:54	imacdonn	Well, the description does say "Performs a release-specific readiness check before restarting services with new code"
20:56:01	imacdonn	https://docs.openstack.org/nova/latest/cli/nova-status.html
20:56:39	imacdonn	but yeah ... I was basically using it as a validation that "everything went OK with the nova upgrade"
20:56:40	openstackgerrit	Matt Riedemann proposed openstack/nova master: DNM: Test against mdbooth's evacuate patch https://review.openstack.org/604423
20:58:14	imacdonn	re fresh install vs upgrade, I basically perform the same steps in both cases (that idempotence thing)
21:01:54	mriedem	yeah some of the wording is a bit dated and was written when the initial change in ocata was really for before the new code was started up
21:02:12	mriedem	but that's still how we do it in grenade
21:03:36	imacdonn	wondering if there might be a case where you really do want to check for something before starting the new code, because otherwise the new code might try to do something that would turn out to be catastrophic
21:04:15	mriedem	well, yes, you want to run it after starting new code
21:05:17	mriedem	guh, sorry, typo
21:05:21	mriedem	before starting new code
21:05:22	mriedem	that's the idea
21:05:33	mriedem	put it in a venv or container and run it
21:47:53	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add some more docs for upgrade checkers https://review.openstack.org/604486
22:26:28	imacdonn	mriedem_away: On a slightly related note; is there a reason that this do like .where(compute_nodes.c.deleted==0) ? https://github.com/openstack/nova/blob/master/nova/cmd/status.py#L111-L130
22:29:43	imacdonn	erm, I mean; ... that this doesn't* do like .....
22:32:41	openstack	Launchpad bug 1757207 in OpenStack Compute (nova) "compute resource providers not equal to compute nodes in deployment" [Undecided,Incomplete]
22:32:41	imacdonn	see also https://bugs.launchpad.net/nova/+bug/1757207
#openstack-nova - 2018-09-22
00:10:24	karimull	Hi mriedem : a qk question , can we add plugins in nova-compute , which can be called before libvirt is executed for special processing during instance launch?
00:59:18	openstackgerrit	iain MacDonnell proposed openstack/nova master: nova-status - don't count deleted compute_nodes https://review.openstack.org/604495
01:19:05	openstackgerrit	Merged openstack/nova stable/rocky: Filter deleted computes from get_all_by_uuids() https://review.openstack.org/604367
04:30:36	openstackgerrit	Merged openstack/nova master: remove virt driver requires_allocation_refresh https://review.openstack.org/602481
04:30:43	openstackgerrit	Merged openstack/nova master: Nix update_instance_allocation, _allocate_for_instance https://review.openstack.org/604186
06:50:21	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
11:29:17	openstackgerrit	Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
16:06:25	mriedem	imacdonn: replied
16:07:03	mriedem	karimull: there are "hooks" so you could write something out of tree to run before/after the driver spawn in the compute service, however, hooks are deprecated and your out of tree code could be broken at any time
16:07:46	mriedem	https://github.com/openstack/nova/blob/master/nova/hooks.py
16:24:52	imacdonn	mriedem: thanks!
16:30:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: Ignore VirtDriverNotReady in _sync_power_states periodic task https://review.openstack.org/604376
17:20:41	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
19:57:06	openstackgerrit	Merged openstack/nova stable/rocky: Fix service list for disabled compute using MC driver https://review.openstack.org/592310
20:32:29	openstackgerrit	Vlad Gusev proposed openstack/nova stable/pike: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/604039
#openstack-nova - 2018-09-23
05:17:24	openstackgerrit	Merged openstack/nova master: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/526558
06:47:15	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
06:53:37	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/604577
08:08:05	openstackgerrit	Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
10:04:17	openstackgerrit	Brin Zhang proposed openstack/nova master: Rename "polling_changes-since_parameter.rst" https://review.openstack.org/604606
10:43:24	openstackgerrit	Brin Zhang proposed openstack/nova master: Rename "polling_changes-since_parameter.rst" https://review.openstack.org/604606
10:52:09	openstackgerrit	Hamdy Khader proposed openstack/nova master: Set defult value of num_nvme_discover_tries=5 https://review.openstack.org/602351
15:01:18	openstackgerrit	melissaml proposed openstack/python-novaclient master: Update the URL in doc https://review.openstack.org/604629
15:28:55	openstackgerrit	Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844
15:48:16	openstackgerrit	Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844
#openstack-nova - 2018-09-24
02:15:15	openstackgerrit	Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844
06:23:18	openstackgerrit	Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687
06:57:44	openstackgerrit	Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687
06:58:55	gibi	mriedem: thanks for the review, I'm planning to focus on that patch series this week
06:59:34	gibi	cdent: bedisdes the use-nested-allocation-candidates I think bauzas working on the vgpu support that will utilize nested scheduling too
07:05:16	gibi	s/bedisdes/besides/ :)
08:03:09	openstackgerrit	Lee Yarwood proposed openstack/nova master: placement: Always reset conf.CONF when starting the wsgi app https://review.openstack.org/604693
08:03:47	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: placement: Always reset conf.CONF when starting the wsgi app https://review.openstack.org/604694
08:31:37	bauzas	good morning nova
08:31:41	bauzas	I wave a bit late tho
08:46:15	gibi	bauzas: good morning
08:52:09	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
09:05:55	zioproto	hello
09:06:12	zioproto	I am upgrading my cluster from Newton to Ocata, and I see weird failures in Nova like this http://paste.openstack.org/show/730608/
09:06:24	zioproto	does anyone had such problems with database foreign keys ?
09:06:29	zioproto	this seems to be related to heat
09:34:01	openstackgerrit	Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687
09:57:59	ondrejme	Hello, I am trying to build Rocky from source and deploy with Kolla-ansible. I have tried to create image for nova (rhel-based), but there's a version requirement for qemu - a "qemu-img-ev". I don't want to use -ev so i built it with older version called "qemu-img". What problems can I encounter? Thanks ahead.
09:59:35	openstackgerrit	Merged openstack/osc-placement master: import zuul job settings from project-config https://review.openstack.org/601374
10:04:28	openstackgerrit	Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360
10:06:25	egonzalez	ondrejme better use #openstack-kolla channel, sort question is to remove package and append your with template overrides {% set nova_compute_packages_remove = ['qemu-img-ev'] %} {% set nova_compute_packages_append = ['qemu-img'] %}
10:06:39	openstackgerrit	Merged openstack/osc-placement master: switch documentation job to new PTI https://review.openstack.org/601375
10:06:41	openstackgerrit	Merged openstack/osc-placement master: add python 3.6 unit test job https://review.openstack.org/601376
10:09:25	johnthetubaguy	gibi: I had thoughts on this one, but really close to approve, what do you think, if you have time to take a peak at that? https://review.openstack.org/#/c/476459/29
10:11:15	ondrejme	egonzales: thats what I did, and I managed to deploy as well with successful instance launch, that instance is up and running just fine, I was just wondering if you know of any functions that will not work that way
10:23:53	gibi	johnthetubaguy: looking
10:34:33	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
10:41:57	gibi	johnthetubaguy: responeded inline. I think you raised the same issue melwitt raised in PS21 then had some back and forth and ended up passing delete_type instead on NotificationAction
10:57:36	johnthetubaguy	gibi: cool, I was struggling to work out all the context in the previous revisions
10:58:00	johnthetubaguy	gibi: I am OK with add the host param, but wasn't sure what you thought, can be a follow up patch
10:58:59	gibi	johnthetubaguy: yeah I can do a followup right away
10:59:31	johnthetubaguy	gibi: sounds good, I wasn't 100% if my reading of that was correct
10:59:46	gibi	johnthetubaguy: I double checked and I think you are correct
10:59:56	johnthetubaguy	gibi: given its been cooking since jun17, lets just merge where we are now!
11:00:04	gibi	johnthetubaguy: I totally agree
11:00:13	johnthetubaguy	cool
11:11:24	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Follow up for Ib6f95c22ffd3ea235b60db4da32094d49c2efa2a https://review.openstack.org/604743
11:11:38	gibi	johnthetubaguy: thanks for the reviews, here is the followup ^
11:36:56	kashyap	gibi: Thanks for the review here: https://review.openstack.org/#/c/562313/3
11:37:51	gibi	kashyap: don't worry, it is easy to mix those variables
11:38:08	kashyap	So two things there: (a) first bump the version for Stein, as advertized in commit: 28d337b (Pick next minimum libvirt / QEMU versions for "Stein")
11:38:26	openstackgerrit	Vladyslav Drok proposed openstack/nova stable/pike: Fix resize_instance rpcapi call https://review.openstack.org/603439
11:39:41	kashyap	gibi: ... (b) Need to pick version for the 'T' release.
11:39:52	kashyap	gibi: Can't bump without first picking the future MIN versions for the 'T' release.

Earlier   Later