| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-18 | |||
| 16:24:07 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Fix false ERROR message at compute restart https://review.opendev.org/699496 | |
| 16:24:11 | openstack | bug 1825537 in OpenStack Compute (nova) pike "finish_resize failures incorrectly revert allocations" [Medium,Confirmed] https://launchpad.net/bugs/1825537 | |
| 16:24:11 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Add functional recreate test for regression bug 1825537 https://review.opendev.org/682725 | |
| 16:24:16 | openstack | bug 1764556 in OpenStack Compute (nova) pike ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress] https://launchpad.net/bugs/1764556 - Assigned to Matt Riedemann (mriedem) | |
| 16:24:16 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Add functional recreate test for bug 1764556 https://review.opendev.org/673830 | |
| 16:24:24 | openstack | bug 1778305 in OpenStack Compute (nova) pike "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Medium,In progress] https://launchpad.net/bugs/1778305 - Assigned to Matt Riedemann (mriedem) | |
| 16:24:24 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Add functional regression test for bug 1778305 https://review.opendev.org/673832 | |
| 16:24:33 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Don't generate service UUID for deleted services https://review.opendev.org/673833 | |
| 16:43:58 | openstackgerrit | Merged openstack/nova-specs master: [Trivial] Remove note for the implementation https://review.opendev.org/714817 | |
| 17:47:53 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers attributes Policies https://review.opendev.org/719729 | |
| 18:03:29 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers attributes policies https://review.opendev.org/719730 | |
| 18:03:57 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remaining servers policies https://review.opendev.org/720104 | |
| 18:04:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remaining servers Policies https://review.opendev.org/720106 | |
| 18:04:20 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remaining servers policies https://review.opendev.org/720116 | |
| 22:34:56 | openstackgerrit | Merged openstack/nova stable/train: Add config option for neutron client retries https://review.opendev.org/715010 | |
| 22:39:48 | openstackgerrit | Merged openstack/nova stable/train: bug-fix: Reject live migration with vpmem https://review.opendev.org/714064 | |
| 22:39:53 | openstackgerrit | Merged openstack/nova stable/stein: Fix intermittently failing regression case https://review.opendev.org/713682 | |
| 22:40:00 | openstackgerrit | Merged openstack/nova stable/stein: Lowercase ironic driver hash ring and ignore case in cache https://review.opendev.org/713982 | |
| #openstack-nova - 2020-04-19 | |||
| 02:52:53 | openstackgerrit | Merged openstack/nova stable/train: Abort live-migration during instance_init https://review.opendev.org/720414 | |
| 15:15:44 | openstackgerrit | Merged openstack/nova master: doc: mark the max microversion for ussuri https://review.opendev.org/719313 | |
| #openstack-nova - 2020-04-20 | |||
| 02:59:57 | openstackgerrit | Qitao proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/721145 | |
| 07:24:08 | bauzas | good morning Nova | |
| 07:24:09 | gibi | good morning Nova | |
| 07:25:34 | lyarwood | morning morning | |
| 07:56:23 | lyarwood | https://review.opendev.org/#/c/718964/ & https://review.opendev.org/#/c/701756/ could use stable core review so I can cut the release ahead of RC if anyone has time today btw | |
| 08:10:34 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP nova-next: Start testing the q35 machine type https://review.opendev.org/708701 | |
| 08:48:08 | bauzas | lyarwood: any idea why https://review.opendev.org/699291 is not backported ? https://review.opendev.org/#/q/topic:bug/1855927+(status:open+OR+status:merged) | |
| 08:49:01 | bauzas | lyarwood: ah, nevermind, found the backport https://review.opendev.org/#/c/701756/2 | |
| 08:52:08 | bauzas | something is actually weird with those changes, trying to untangle it | |
| 08:54:39 | bauzas | ah-ah, okay, got it | |
| 08:54:47 | bauzas | https://review.opendev.org/#/q/topic:bug/1856925+(status:open+OR+status:merged) vs. https://review.opendev.org/#/q/topic:bug/1855927+%28status:open+OR+status:merged%29 | |
| 09:02:19 | lyarwood | bauzas: sorry was head down on something downstream | |
| 09:02:36 | bauzas | lyarwood: no worries, commenting with a +1 | |
| 09:03:19 | lyarwood | right because https://review.opendev.org/#/c/699291/ hasn't landed yet | |
| 09:03:21 | lyarwood | sorry | |
| 09:04:15 | lyarwood | does anyone know if we dump the compute service version somewhere in the logs during n-cpu startup? | |
| 09:05:55 | bauzas | lyarwood: I just proposed to provide another revision of https://review.opendev.org/#/c/701756/2 that would be in a separate branch and just isolated | |
| 09:06:02 | tosky | lyarwood: you can check the package version, I guess | |
| 09:06:05 | bauzas | because that's confusing | |
| 09:06:34 | bauzas | lyarwood: AFAIR we do provide the compute service version, lemme grab you the logs | |
| 09:10:39 | bauzas | lyarwood: you can get the compute package version here which allows you to get the compute service version by looking at the code https://zuul.opendev.org/t/openstack/build/96486cc66f7542318bbcfc4a43784d56/log/controller/logs/screen-n-cpu.txt#861 | |
| 09:12:27 | lyarwood | bauzas: do we not dump anything in the API or scheduler about the compute versions they are aware of | |
| 09:12:44 | lyarwood | bauzas: I'm specifically trying to find the service version, not the package version btw. | |
| 09:12:50 | lyarwood | bauzas: https://github.com/openstack/nova/blob/stable/queens/nova/compute/api.py#L3976-L3998 for this | |
| 09:12:56 | bauzas | lyarwood: yup, I understood your question | |
| 09:13:14 | bauzas | but AFAIR, we don't expose the service versions, just the package versions | |
| 09:13:14 | lyarwood | bauzas: I'm thinking that there's an older compute still registered somewhere that's causing that to use the old legacy path | |
| 09:14:10 | lyarwood | kk thanks | |
| 09:15:54 | bauzas | lyarwood: this being said, the version field of the Service object is maybe expose thru the REST API | |
| 09:15:59 | bauzas | exposed* | |
| 09:16:59 | bauzas | meh, nvm | |
| 09:17:07 | bauzas | this would be a nova-manage thing | |
| 09:18:13 | bauzas | at least we don't return it to the API https://docs.openstack.org/api-ref/compute/?expanded=list-compute-services-detail#id368 | |
| 09:21:04 | bauzas | lyarwood: I think I found something interesting for your problem | |
| 09:22:27 | bauzas | lyarwood: the logs can emit some information about an old service when it starts https://github.com/openstack/nova/blob/master/nova/service.py#L72-L81 | |
| 09:34:05 | nightmare_unreal | hello is there a way to get nova cli output in json ? for e.g. openstack server list -f json give output in json. but there is no such thing(-format) in nova cli | |
| 09:38:32 | gibi | bauzas, lyarwood: the version notifications contain the service version https://github.com/openstack/nova/blob/master/doc/notification_samples/common_payloads/ServiceStatusPayload.json#L12 | |
| 09:39:02 | bauzas | gibi: ah good point, forgot it | |
| 09:39:04 | lyarwood | gibi: ah! thanks | |
| 09:39:44 | lyarwood | gibi: but that's only for active services right? | |
| 09:39:51 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741 | |
| 09:40:04 | lyarwood | gibi: I'm assuming these computes are dead tbh so maybe the db is the only real way to tell | |
| 09:40:16 | bauzas | I had to modify again something I fixed locally | |
| 09:40:45 | bauzas | for some reason, my editor provided me a stale version of an old file | |
| 09:41:02 | bauzas | so my last commit reverted another fix I made | |
| 09:41:04 | bauzas | strange... | |
| 09:41:13 | bauzas | (I have to be cautious now) | |
| 09:41:18 | bauzas | Atom-- | |
| 09:44:34 | gibi | lyarwood: if you can interact with the service via the REST API then you can get the above notificaiton | |
| 09:44:40 | gibi | like when you disable it | |
| 09:45:09 | lyarwood | ack thanks | |
| 09:46:51 | bauzas | gibi: lyarwood: I don't think it's a crucial and secret information bit to expose the service object version thru the logs | |
| 09:47:47 | lyarwood | bauzas: for admins no I guess not | |
| 09:48:05 | gibi | bauzas: agree, this is not a secret for admins | |
| 09:48:08 | bauzas | just sayin', we could log something there https://github.com/openstack/nova/blob/stable/queens/nova/service.py#L166 (for DEBUG purposes) | |
| 09:48:26 | gibi | sure | |
| 09:48:49 | bauzas | we already do https://github.com/openstack/nova/blob/stable/queens/nova/service.py#L158 | |
| 09:49:04 | bauzas | lyarwood: feel free to file a patch ;) | |
| 09:57:55 | brinzhang | good morning all | |
| 09:58:39 | brinzhang | gibi: I am not sure which is true, but I replied something in https://review.opendev.org/#/c/720670/ | |
| 09:59:28 | brinzhang | Indeed, nova-cinder interaction and nova-neutron interaction have different result, if the xxxclient reports exception | |
| 10:03:24 | bauzas | stephenfin: gibi: fwiw, we said we could let it go https://review.opendev.org/#/c/712741/ | |
| 10:03:40 | bauzas | (don't leave it frozen) | |
| 10:04:06 | bauzas | okay, it was a terrible play of words | |
| 10:04:15 | bauzas | => [] | |
| 10:06:41 | brinzhang | bauzas: a nit inline ^ | |
| 10:07:22 | bauzas | brinzhang: can't see your comment :) | |
| 10:07:34 | gibi | bauzas: I will reply after lunch | |
| 10:07:46 | brinzhang | bauzas: my net so .. slowly | |
| 10:07:51 | bauzas | thanks migi and brinzhang | |
| 10:07:53 | gibi | brinzhang: same to you | |
| 10:07:58 | bauzas | oh shit, wrong mix | |
| 10:08:05 | brinzhang | gibi: thanks | |
| 10:08:07 | bauzas | thanks gibi | |
| 10:08:11 | bauzas | migi, gibi, dammit | |
| 10:08:39 | bauzas | ah, sad, he's not connected here | |
| 10:08:49 | brinzhang | bauzas: done | |
| 10:09:42 | bauzas | brinzhang: ack, good, can respin since Zuul hasn't replied yet | |
| 10:11:14 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741 | |
| 10:16:00 | brinzhang | bauzas: thanks quickly update ^ | |