Posted	Nick	Remark
#openstack-nova - 2018-09-21
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	imacdonn	see also https://bugs.launchpad.net/nova/+bug/1757207
22:32:41	openstack	Launchpad bug 1757207 in OpenStack Compute (nova) "compute resource providers not equal to compute nodes in deployment" [Undecided,Incomplete]
#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.
11:39:57	kashyap	johnthetubaguy: ^ You have a comment here?
11:43:10	gibi	kashyap: so you will propose a patch that set MIN based on NEXT_MIN but for that we have to figure out what will be the new value of NEXT_MIN
11:44:09	kashyap	gibi: Yes. First we need to pick NEXT_MIN_* versions
11:44:17	kashyap	gibi: Like I sent out an email to 'operators' list in the past: http://lists.openstack.org/pipermail/openstack-operators/2018-April/015089.html
11:44:42	gibi	kashyap: OK this make sense
11:44:52	kashyap	(Operators + Dev list)
11:44:55	kashyap	Do we have a name for the 'T' release yet?
11:45:44	gibi	kashyap: I think we are in the name suggestion phase
11:45:48	kashyap	gibi: I just need to spend some tedious time to go through all distros that matter / upstream cares about, and arrive at the new enough version that is supported by all distros..

Earlier   Later