Earlier  
Posted Nick Remark
#openstack-nova - 2020-04-17
23:23:10 openstackgerrit melanie witt proposed openstack/nova stable/pike: Require stestr < 3.0.0 in test-requirements https://review.opendev.org/720849
23:25:03 openstackgerrit melanie witt proposed openstack/nova stable/pike: Require stestr < 3.0.0 in test-requirements https://review.opendev.org/720849
23:25:04 openstackgerrit melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600
#openstack-nova - 2020-04-18
04:22:53 openstackgerrit Merged openstack/python-novaclient stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/719164
05:23:45 openstackgerrit Takashi Natsume proposed openstack/python-novaclient stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719165
07:44:42 openstackgerrit Merged openstack/python-novaclient stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719165
12:50:36 openstackgerrit Merged openstack/nova master: Follow-ups for host_status:unknown-only policy rule https://review.opendev.org/713295
12:50:43 openstackgerrit Merged openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607
14:55:09 openstackgerrit Lee Yarwood proposed openstack/nova master: block_device: Use the attachment update API to refresh connection_info https://review.opendev.org/720769
15:24:17 openstackgerrit Merged openstack/nova stable/train: Avoid spurious error logging in _get_compute_nodes_in_db https://review.opendev.org/702902
16:16:15 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Init HostState.failed_builds https://review.opendev.org/669538
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 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Add functional recreate test for regression bug 1825537 https://review.opendev.org/682725
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:16 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Add functional recreate test for bug 1764556 https://review.opendev.org/673830
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:24 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Add functional regression test for bug 1778305 https://review.opendev.org/673832
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: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 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:13:14 bauzas but AFAIR, we don't expose the service versions, just the package versions
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

Earlier   Later