| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-07 | |||
| 20:55:31 | efried | dev & review. Yes. | |
| 20:55:46 | dansmith | yeah, I get that | |
| 20:55:54 | dansmith | anyway, don't want to get back into it right now | |
| 21:06:35 | dansmith | mriedem: there's a lot more boilerplate required in API patches since the last time I wrote one.. any chance we have a checklist or something I can follow? | |
| 21:06:45 | dansmith | api samples, api refs, schema crap, etc, etc | |
| 21:07:34 | mriedem | yeah sec | |
| 21:07:56 | mriedem | https://docs.openstack.org/nova/latest/contributor/#the-nova-api | |
| 21:10:24 | dansmith | thanks | |
| 21:32:24 | mriedem | efried: is this what you were asking for and expecting in PS9? https://review.opendev.org/#/c/667133/10/doc/source/user/index.rst@75 | |
| 21:32:38 | mriedem | if what i'm saying is something you agree with, i can approve and fup | |
| 21:33:15 | efried | looking. | |
| 21:34:46 | efried | mriedem: I agree, though this makes it *slightly* easier to find, since the first link in the admin doc is to the user side. | |
| 21:34:57 | efried | I didn't want the existing link removed fwiw, I just wanted a new one added. | |
| 21:35:00 | efried | but also | |
| 21:35:24 | efried | when it comes to docs, as long as we're not injecting anything wrong/regressive, I like the idea of merging quickly even if imperfect just to move things forward. | |
| 21:35:51 | efried | anyway, if you want to respin quick and self+2 I'll +A. | |
| 21:35:57 | efried | or fup, either way. | |
| 21:38:17 | mriedem | yeah i'll post a follow up | |
| 21:45:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: link to user/availability-zones from user home page https://review.opendev.org/687165 | |
| 21:54:14 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/train: docs: Highlight the current broken state of SEV https://review.opendev.org/686414 | |
| 21:54:52 | mriedem | efried: what are your thoughts on cutting rc2? it's due thursday, so i guess eod tomorrow if we don't hear from aspiers? | |
| 21:56:25 | efried | mriedem: yes | |
| 21:56:43 | efried | mriedem: as in, merge the 'SEV busted' patch tomorrow and cut the RC as soon as it's in. | |
| 21:57:01 | mriedem | ok | |
| 21:57:12 | openstackgerrit | Merged openstack/nova master: fixtures: Beef up NeutronFixture https://review.opendev.org/686795 | |
| 21:57:17 | mriedem | i'll +2 the docs patch tomorrow | |
| 21:57:22 | dansmith | how invasive is it to just revert large chunks of it? | |
| 21:57:25 | mriedem | likely too risky to rush the other fix at this point anyway | |
| 21:57:39 | efried | mriedem: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-10-04.log.html#t2019-10-04T14:50:21 | |
| 21:57:49 | mriedem | dansmith: i'm assuming there would be a lot of pus and blood | |
| 21:57:55 | efried | mriedem: note that the doc patch is in stable so I can't help you. | |
| 21:58:11 | mriedem | sure, but dan can | |
| 21:58:16 | dansmith | yep | |
| 21:58:17 | efried | dansmith: riskier than not doing that. | |
| 21:58:25 | efried | and it works for common configs | |
| 21:58:54 | dansmith | well, I mean like reverting the piece that even lets you turn it on kinda thing | |
| 21:59:07 | dansmith | sounds like abandonware before delivery event | |
| 22:01:08 | mriedem | you'd just change the libvirt driver _sev_enabled method to return False | |
| 22:01:20 | dansmith | yeah, that doesn't sound so bad | |
| 22:01:44 | dansmith | so imagine it being two months from now and we still have radio silence on a fix and maintenance | |
| 22:01:58 | dansmith | if we reverted the renos and disabled it, we could remove it without much fanfare | |
| 22:02:06 | dansmith | I know that's fairly nuclear, but just worth considering I think | |
| 22:02:41 | mriedem | it would elicit a response at least :) | |
| 22:02:46 | dansmith | also that | |
| #openstack-nova - 2019-10-08 | |||
| 00:30:17 | openstackgerrit | Merged openstack/nova master: docs: Rewrite host aggregate, availability zone docs https://review.opendev.org/667133 | |
| 00:39:37 | openstackgerrit | Merged openstack/nova master: fixtures: Handle iterable params for 'NeutronFixture.list_*' https://review.opendev.org/686796 | |
| 00:45:03 | openstackgerrit | Merged openstack/nova master: fixtures: Store 'device_id' when creating port in NeutronFixture https://review.opendev.org/686797 | |
| 02:17:16 | openstackgerrit | Jing Zhang proposed openstack/nova master: [WIP] Default mem_page_size to small for pinned instances https://review.opendev.org/686079 | |
| 02:43:34 | openstackgerrit | ya.wang proposed openstack/nova-specs master: Re-propose expose-auto-converge-post-copy spec for Ussuri https://review.opendev.org/687199 | |
| 04:57:35 | openstack | bug 1709118 in OpenStack Compute (nova) "_ContextAuthPlugin needs a refresh" [Low,Invalid] https://launchpad.net/bugs/1709118 | |
| 04:57:35 | openstackgerrit | Merged openstack/nova master: Remove workaround for bug #1709118 https://review.opendev.org/524316 | |
| 05:42:43 | openstackgerrit | Merged openstack/nova master: [Gate fix] Avoid use cell_uuid before assignment https://review.opendev.org/686996 | |
| 06:53:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/train: Error out interrupted builds https://review.opendev.org/687216 | |
| 07:02:17 | bauzas | good morning Nova | |
| 07:13:43 | gibi | bauzas: good morning | |
| 07:19:36 | bauzas | gibi: I was a bit on and off upstream last days so if you need some eyes, tell me | |
| 07:21:20 | gibi | bauzas: nothing urgent at the moment. I still want to get back to your audit patch | |
| 07:21:40 | bauzas | gibi: I need to write a reno note and some docs like matt asked | |
| 07:23:46 | gibi | bauzas: I saw mriedem's comments. I more interested in trying your code in my devstack | |
| 07:25:32 | bauzas | gibi: that'd be awesome | |
| 07:25:42 | bauzas | gibi: functional tests don't really check children allocations | |
| 07:26:09 | bauzas | gibi: but hopefully it will work this time given you found the root cause last time :) | |
| 07:27:53 | gibi | bauzas: I can try that with bandwith allocation on child RPs. | |
| 07:28:00 | bauzas | ++ | |
| 07:28:37 | bauzas | gibi: I can also do this with VGPU allocations, but i need to reboot my environment :) | |
| 07:29:10 | gibi | bauzas: fortunetly the bandwidth setup does not need any special hardware so it is easy for me to build a devstack | |
| 07:29:21 | bauzas | yeah | |
| 07:29:46 | bauzas | hopefully, VGPU allocations would work with Intel GPU | |
| 07:29:57 | bauzas | but I didn't have time to test those | |
| 07:39:42 | openstackgerrit | Akihiro Motoki proposed openstack/nova master: doc: Improve PDF document structure https://review.opendev.org/682746 | |
| 07:53:50 | dr_gogeta86 | good morning all | |
| 09:05:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Remove stale BDMs on reserve_block_device_name failure https://review.opendev.org/682594 | |
| 09:56:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Ignore volume exceptions during post_live_migration https://review.opendev.org/682622 | |
| 09:56:24 | openstack | bug 1843639 in OpenStack Compute (nova) "libvirt: post_live_migration failures to disconnect volumes result in the rollback of live migrations" [Medium,In progress] https://launchpad.net/bugs/1843639 - Assigned to Artom Lifshitz (notartom) | |
| 09:56:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1843639 https://review.opendev.org/682621 | |
| 10:08:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Improve PDF document structure https://review.opendev.org/682746 | |
| 10:10:08 | stephenfin | bauzas: Could I ask you to push through a few trivial patches for me, please? I've a lot of stuff in flight /o\ | |
| 10:11:04 | stephenfin | If you could hit https://review.opendev.org/686211 https://review.opendev.org/686901 https://review.opendev.org/684774 and https://review.opendev.org/#/c/684775/ that's be great | |
| 10:11:19 | stephenfin | https://imgflip.com/s/meme/That-Would-Be-Great.jpg | |
| 10:17:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Take an instance.uuid lock when rebooting https://review.opendev.org/673463 | |
| 10:38:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Support move operations with qos ports - Ussuri https://review.opendev.org/686779 | |
| 10:39:53 | gibi | efried: fixed your comments in ^^ | |
| 10:42:21 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Remove 'adv-config', 'system-admin' subdocs https://review.opendev.org/684402 | |
| 10:42:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Strip '.rst' suffix https://review.opendev.org/687264 | |
| 10:42:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Replacing underscores with dashes https://review.opendev.org/685929 | |
| 10:45:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove @safe_connect from put_allocations https://review.opendev.org/686579 | |
| 10:55:23 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: refactor volume bdm rollback error handling https://review.opendev.org/656500 | |
| 10:55:24 | lyarwood | stephenfin: ^ finally updated that if you have time this wekk. | |
| 10:55:29 | lyarwood | week* | |
| 11:20:36 | sean-k-mooney | stephenfin: dumb question but how to i add blueprint to the nova team meeting agenda again. there is a wiki page i think right? | |
| 11:20:55 | sean-k-mooney | https://wiki.openstack.org/wiki/Meetings/Nova? | |
| 11:23:22 | sean-k-mooney | im not sure if that is the agenda for last weeks meetin or thi seeks meting | |
| 11:29:06 | gibi | sean-k-mooney: add your stuff to that agenda with a timestamp so when efried goes and updates the agenda for the next meeting he will see that your item is for the upcoming meeting | |
| 11:29:59 | sean-k-mooney | thanks | |
| 11:33:34 | sean-k-mooney | gibi: i bacically want to know if https://blueprints.launchpad.net/nova/+spec/inplace-rebuild-of-numa-instances can proceed as a specless blueprint or if i should write a spec | |
| 11:33:57 | sean-k-mooney | but i have updated the agenda with the timestamp as you suggesteded | |
| 11:36:09 | gibi | sean-k-mooney: after reading the bp, for me it seems OK to have this as specless as it is only fixing bugs | |
| 11:42:28 | sean-k-mooney | gibi: that is what i was hoping. do you think it makes sense to group them as a blueprint as i have | |
| 11:43:29 | sean-k-mooney | i intend to fix the bug as seperate patches in the order i suggested (prevent broken behavior then skip filter if safe) | |
| 11:51:53 | gibi | sean-k-mooney: it is easier to track multiple bugs as a single bp so I agree with your syle | |