Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-03-18
10:20:35 gibi stephenfin: the current way is fine by me but maybe there is other possibilities
10:23:03 stephenfin gibi: That works, but we could/should probably just update the release note in novaclient
10:23:28 stephenfin i.e. add https://review.opendev.org/c/openstack/nova/+/780401/1/nova/api/openstack/compute/rest_api_version_history.rst#392
10:24:44 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Change error code when vDPA is unsupported https://review.opendev.org/c/openstack/nova-specs/+/781259
10:24:44 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: trivial: Fix spelling, formatting of vDPA spec https://review.opendev.org/c/openstack/nova-specs/+/781260
10:26:02 gibi stephenfin: so have an explicit well named anchor in the nova doc and update the novaclient reno to refer to the well named anchor?
10:26:14 stephenfin I think that would be a better approach, yes
10:26:20 stephenfin Personally
10:26:47 gibi OK lets do that
10:26:57 gibi I can propose the fix
10:27:05 stephenfin Sounds good. Happy to review
10:27:09 gibi cool
11:10:30 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/781138
11:12:15 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/602432
11:20:23 lyarwood gibi: mind if I add a cinder cross project session in the PTG pad, have a topic.
11:20:38 gibi lyarwood: please do
11:28:10 lyarwood done thanks
11:50:07 lyarwood elod: https://review.opendev.org/c/openstack/nova/+/758733/ is ready for review btw
12:31:08 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add a resource limits guide https://review.opendev.org/c/openstack/nova/+/781433
12:31:09 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add a real-time guide https://review.opendev.org/c/openstack/nova/+/781434
12:31:09 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove duplicate TPM extra spec info https://review.opendev.org/c/openstack/nova/+/781435
12:31:10 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove duplicated PCI passthrough extra spec info https://review.opendev.org/c/openstack/nova/+/781436
12:31:10 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add SEV guide https://review.opendev.org/c/openstack/nova/+/781437
12:31:11 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add CPU models guide https://review.opendev.org/c/openstack/nova/+/781438
12:31:11 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Change formatting of hypervisor config guides https://review.opendev.org/c/openstack/nova/+/781439
12:31:12 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add libvirt misc doc https://review.opendev.org/c/openstack/nova/+/781440
12:31:29 stephenfin Some good docs there. I was holding off until after feature freeze ^
12:36:45 kashyap stephenfin: Cool; a separate CPU models guide is good; instead of that info buried elsewhere
12:37:02 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add image metadata property reference guide https://review.opendev.org/c/openstack/nova/+/756867
12:37:10 kashyap stephenfin: We could also link docs like these into it: something like "See also: https://docs.openstack.org/nova/latest/admin/mitigation-for-Intel-MDS-security-flaws.html"
12:37:31 kashyap If only to make it more discoverable; /me adds a note in the review
12:37:40 stephenfin I'll probably move that doc into the cpu-models one (with a redirect) at some point
12:37:59 stephenfin It would fit nicely in there as a concrete use case
12:39:31 kashyap Yeah. Idd
12:41:24 kashyap stephenfin: There's additional things we can mention on 'host-model' that can be very useful for people to know _why_ we default to it
12:49:19 elod lyarwood: +2+W'd
13:02:39 lyarwood elod: thanks
13:10:27 kashyap stephenfin: Sorry for the wall o' text here, but I hope some useful crumbs in there: https://review.opendev.org/c/openstack/nova/+/781438
13:13:20 kashyap s/hope/hope there are/
13:23:26 dansmith lyarwood: ugh, yeah sorry.. fix a bug, get punished I guess
13:24:13 dansmith lyarwood: assuming that's going to affect all older branches, we may want to make tempest only rely on that behavior for new enough glance
13:28:09 gibi stephenfin: it seems '.. _2.88 microversion:' does not generate an anchor but '.. _microversion 2.88:' does generate 'microversion-2-88' as an anchor
13:28:53 gibi so I'm wondering how our other anchors starting with numbers in the microversion history document ever worked
13:28:59 lyarwood dansmith: no issues, no idea about the glance jobs but Nova only started testing that in stable/victoria so the backports should be enough for us
13:29:40 dansmith lyarwood: ahh, right
13:31:21 dansmith man gerrit is s l o w today
13:56:52 openstackgerrit Balazs Gibizer proposed openstack/python-novaclient master: Use well named anchor into the microversion history https://review.opendev.org/c/openstack/python-novaclient/+/781484
14:03:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: doc: mark the max microversion for wallaby https://review.opendev.org/c/openstack/nova/+/780401
14:04:51 gibi stephenfin: ^^ your turn
14:07:28 mnaser weirdest thing ever
14:07:46 mnaser we changed the db for a system (literally, just mysqldump | mysql)
14:07:59 mnaser and now, limit= from nova api is always returning exactly half of what we are supposed to get
14:08:16 mnaser https://www.irccloud.com/pastebin/BEMJghhM/
14:08:20 mnaser anyone seen something like this? o_O
14:18:12 openstackgerrit Lee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent https://review.opendev.org/c/openstack/nova/+/720769
14:25:54 gmann lyarwood: gibi stephenfin IMO, 400 is right one and if we change existing 409/403 to 400 it does not require microversion bump.
14:44:18 lyarwood mnaser: odd running master?
14:44:47 lyarwood stephenfin: ^ are you cool with gmann's suggestion?
14:57:07 dansmith elod: [07:53:00] <abhishekk> lyarwood, dansmith may be Elod will approve the patches while I am gone
14:57:16 elod lyarwood: I think https://review.opendev.org/781493 needs to be squashed with https://review.opendev.org/781426/ otherwise gate won't pass, am I right?
14:57:52 elod dansmith: sure
14:57:55 abhishekk elod, right
14:58:34 lyarwood jebus
14:58:36 abhishekk dansmith, so to avoid all this dance, can we just drop lower constraints job?
14:58:36 lyarwood okay sorry
14:59:01 dansmith abhishekk: yeah, for sure, I was just trying to avoid the dance since it was supposed to all be headed to the gate
14:59:13 dansmith abhishekk: but if you want to push that in underneath (or get lyarwood to do it) that's fine with me
14:59:16 abhishekk and sqash the dropping of nova-ceph job with action wrapper
14:59:39 dansmith I wasn't 100% sure we had settled on that as a wider community, as was asserted in the meeting, but I was definitely not following that whole thread
14:59:58 abhishekk ok, so lets keep it as it is at the moment
15:00:14 lyarwood I'll squash things now
15:00:25 abhishekk lyarwood, yep, go ahead
15:00:31 dansmith abhishekk: to be clear, I'm saying at this point, we're not headed for gate, so we can do whatever you want :)
15:00:51 dansmith lyarwood: has nova dropped the lower-constraints job(s)?
15:01:06 abhishekk dansmith, yes, lets merge that dropping lower constraint patch differently
15:01:16 dansmith abhishekk: ack
15:01:22 abhishekk after this is done
15:01:35 elod no, lc jobs are still there (only disabled on rocky(?))
15:01:42 lyarwood dansmith: no
15:01:44 dansmith ack, okay
15:01:47 gmann dansmith: abhishekk we still have not concluded that what to do on l-c job
15:01:59 gmann discussing in today TC meeting
15:02:03 dansmith gmann: okay I thought it wasn't quite decided yet, but others seemed to think it was
15:02:03 abhishekk gmann, ack, thanks
15:43:30 bauzas gibi: dansmith: fwiw, I removed my -2 from https://review.opendev.org/c/openstack/nova/+/761452 as we're after the FF
15:43:32 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Use proper headers in nova-manage man page https://review.opendev.org/c/openstack/nova/+/726191
15:43:33 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage db' docs to use proper directives https://review.opendev.org/c/openstack/nova/+/677509
15:43:34 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage api_db', 'cell_v2' docs https://review.opendev.org/c/openstack/nova/+/726192
15:43:34 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage placement' docs https://review.opendev.org/c/openstack/nova/+/726193
15:43:35 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Document options for remaining nova-* commands https://review.opendev.org/c/openstack/nova/+/726195
15:43:35 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Populate "See Also" section with meaningful links https://review.opendev.org/c/openstack/nova/+/729917
15:43:36 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add man pages for 'nova-policy' https://review.opendev.org/c/openstack/nova/+/729918
15:43:36 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage libvirt' docs https://review.opendev.org/c/openstack/nova/+/781504
15:44:04 bauzas dansmith: still working on removing 5.0 but given we only have a week, maybe we should just merge the above ? ^
15:45:29 dansmith bauzas: ack, I'm out tomorrow and won't get to revisiting that today I expect, so whatever gibi wants to do
15:45:57 bauzas dansmith: given we will remove the V5proxy in Xena, the 5.0 support would only be in the Wallaby release
15:46:17 gibi dansmith bauzas: if we land what we have today, does that pose any serious limitation?
15:46:45 bauzas gibi: mmmm, I don't know
15:47:07 dansmith gibi: well, I haven't reviewed the latest rev, but no, it's just more than we need and also it lies a bit about older versions we say we can support but can't really, but in practice it's probably not a big deal
15:47:11 bauzas we have a lof of merge conflicts that said => https://review.opendev.org/c/openstack/nova/+/761452

Earlier   Later