| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-11-20 | |||
| 13:26:15 | lpetrut | sean-k-mooney: definitely | |
| 13:31:21 | lpetrut | sean-k-mooney: I think a specless blueprint should be enough: https://blueprints.launchpad.net/nova/+spec/hyperv-rbd | |
| 13:31:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make nova-compute hard fail on db config https://review.opendev.org/763559 | |
| 13:37:34 | sean-k-mooney | lpetrut: if the chagne was larger it would not be but in this case i might be its really up to gibi and the rest of the core team to decide | |
| 13:38:00 | sean-k-mooney | lpetrut: do you have a third party ci in the works to test this | |
| 13:38:20 | sean-k-mooney | lpetrut: i dont see an os-brick-rbd voting on either change | |
| 13:38:41 | sean-k-mooney | i expect we will want to see that pass before merging this | |
| 13:41:13 | lpetrut | atm we're only testing smb, iscsi and FC volumes but we'll add another os-brick job for RBD | |
| 13:42:42 | sean-k-mooney | and a nova job? ideally tempest full with rbd and hyperv? | |
| 13:44:29 | lpetrut | atm the nova job only covers lvm. if by any chance a nova patch breaks support for one of the volume backends (very unlikely considering the minimal nova code for that), we'll notice it when the os-brick CI starts failing | |
| 13:44:36 | sean-k-mooney | it could be the same job just triggering it on nova | |
| 13:45:27 | gibi | lpetrut: thanks for the bp. Could you join to an upcoming Nova weekly meeting to shortly dicuss your bp and approve it to Wallaby? | |
| 13:45:49 | lpetrut | gibi: definitely | |
| 13:46:06 | gibi | lpetrut: then please add your bp to the OpenDiscussion section of the meeting agenda https://wiki.openstack.org/wiki/Meetings/Nova | |
| 13:46:14 | gibi | so we won't forget it | |
| 13:46:30 | lpetrut | gibi: awesome, thanks for the tip | |
| 13:46:38 | gibi | thanks! | |
| 14:24:29 | bauzas | stephenfin: thanks, looking | |
| 14:24:39 | bauzas | for the open spec | |
| 14:32:51 | openstackgerrit | Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/763550 | |
| 14:37:39 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Image and flavor defined ephemeral storage encryption https://review.opendev.org/752284 | |
| 14:40:55 | openstackgerrit | Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/763550 | |
| 14:52:01 | openstackgerrit | Dan Smith proposed openstack/nova master: RFC: Update the install documents for cells and less distro specific https://review.opendev.org/763412 | |
| 15:11:09 | sean-k-mooney | oh i forgot about that | |
| 15:12:04 | gibi | weekend starts early today :) | |
| 15:16:57 | dansmith | heh yeah | |
| #openstack-nova - 2020-11-23 | |||
| 03:53:15 | openstackgerrit | chengsheng proposed openstack/nova master: CPU selection with hypervisor consideration https://review.opendev.org/c/openstack/nova/+/762330 | |
| 06:19:42 | openstackgerrit | YumengBao proposed openstack/nova-specs master: libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116 | |
| 08:15:38 | gibi | good morning Nova | |
| 08:15:56 | gibi | new gerrit... interesting | |
| 08:33:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support sys.argv in wsgi app https://review.opendev.org/c/openstack/nova/+/763750 | |
| 08:33:40 | gibi | and gerrit works :) | |
| 09:37:23 | openstackgerrit | Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/c/openstack/nova/+/763550 | |
| 09:40:59 | lpetrut | gibi: I had to double check to make sure I haven't opened android-review.googlesource.com :) | |
| 09:41:29 | gibi | lpetrut: :) | |
| 09:41:46 | gibi | please note that not everything works https://etherpad.opendev.org/p/gerrit-3.2-post-upgrade-notes | |
| 09:42:19 | gibi | the biggest missing piece for me is the launchpad integration. You have to manually set bug status | |
| 09:43:29 | lpetrut | interesting, good to know. looks like there's quite some work going on in order to bring back those features | |
| 10:21:31 | lyarwood | bauzas: https://review.opendev.org/c/openstack/nova/+/752486 - would you mind hitting this and the regression test below it? | |
| 10:25:53 | kashyap | Oh, shiny new Gerrit | |
| 10:41:16 | stephenfin | why does everything need to be in bold? | |
| 10:42:15 | bauzas | lyarwood: ack, sorry I was taxi for my kids | |
| 10:49:32 | bauzas | how can I see in which releases was merged https://review.opendev.org/c/openstack/nova/+/752247 ? | |
| 10:49:55 | bauzas | in the previous Gerrit version, we had a 'included in' value | |
| 10:50:41 | gibi | bauzas: good question | |
| 10:50:46 | gibi | I havn't found it | |
| 10:51:05 | stephenfin | here's a bit of fun - right click on an element in a path list patch and "inspect" it | |
| 10:51:24 | stephenfin | takes at least 10 seconds for Firefox to catch up on my machine :-D | |
| 10:51:26 | gibi | bauzas: above the file list there is an included-in link | |
| 10:52:08 | bauzas | gibi: haha, thanks ! | |
| 10:52:20 | bauzas | indeed, that helps me | |
| 10:52:22 | bauzas | gibi: <3 | |
| 10:52:57 | bauzas | so, this change was merged during the victoria release, so we don't need to have a victoria backport too | |
| 10:53:10 | bauzas | that's what I wanted to kew | |
| 10:53:13 | bauzas | knew | |
| 11:16:00 | openstack | bug 1895696 in Cinder "unable to boot instance from encrypted volume created from a glance image of an encrypted volume" [Undecided,New] https://launchpad.net/bugs/1895696 | |
| 11:16:00 | openstackgerrit | Merged openstack/nova stable/ussuri: Add regression test for bug #1895696 https://review.opendev.org/c/openstack/nova/+/752485 | |
| 11:24:36 | gibi | the new gerrit supports emojis | |
| 11:24:57 | gibi | I'm not fan of emojis | |
| 11:25:55 | gibi | and gerrit makes suggestion how to make good review comments like: "When disagreeing, explain the advantage of your approach" | |
| 11:26:20 | gibi | I hope I can trun them off after I saw all of them | |
| 11:32:25 | lyarwood | I think he was hoping to be able to review *only* using emojis | |
| 11:32:44 | gibi | :D | |
| 11:32:48 | stephenfin | ❤️❤️❤️ | |
| 11:33:00 | gibi | I like the text base emojis | |
| 11:33:06 | gibi | like ':D' | |
| 11:33:13 | gibi | maybe I'm oldscool | |
| 11:33:28 | stephenfin | gibi: I think you mean 😃 | |
| 11:34:00 | stephenfin | Ideally, you'd be able to toggle between the emoji and their textual representation | |
| 11:34:19 | stephenfin | shortcode, rather | |
| 11:34:31 | stephenfin | but those shortcodes are platform specific :-( | |
| 11:34:33 | stephenfin | e.g. :smiley: | |
| 11:35:52 | gibi | stephenfin: https://pasteboard.co/JBGqpj6.png :D | |
| 11:36:20 | gibi | stephenfin: yeah, I like how hangouts can be forced to translate the emojis back | |
| 11:36:41 | gibi | (at least my emojois not the incoming ones) | |
| 11:40:36 | lyarwood | is someone going to try to rework the gerrit ui to look something like the previous ui we had btw? | |
| 11:41:14 | lyarwood | and the quick query/dashboard list | |
| 11:42:00 | stephenfin | lyarwood: They're working to restore the CI table, but it requires a new plugin so I don't know how long that will take | |
| 11:42:19 | gibi | lyarwood: I might look into the CI table thing at some point as a user script | |
| 11:42:35 | gibi | lyarwood: what is quick query? | |
| 11:42:58 | stephenfin | I assume he means the access to dashboards that didn't require a dropdown | |
| 11:43:28 | lyarwood | yeah the list of editable queries at the top of the page | |
| 11:43:43 | lyarwood | I had a few dashboard listed etc, but now it's behind a dropdown | |
| 11:43:45 | stephenfin | Dashboards are now hidden in the "Your" dropdown. One more click to get those frequently used link | |
| 11:43:59 | lyarwood | yeah | |
| 11:44:28 | gibi | ahh I see | |
| 11:48:45 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/760971 - would you mind taking a look at this today if you have time, just a simple docs change. | |
| 11:48:55 | gibi | looking | |
| 11:49:16 | lyarwood | many thanks | |
| 11:53:21 | gibi | lyarwood: +A | |
| 12:04:39 | gibi | dansmith: when you are up, I pushed a change to https://github.com/kk7ds/openstack-gerrit-dashboard/pull/30 to adapt it to the new gerrit | |
| 12:05:56 | sean-k-mooney | gibi: the old gerrit supported emojis | |
| 12:06:32 | sean-k-mooney | gibi: stephenfin used the poop emoji in his hypervisors api spec | |
| 12:06:40 | gibi | sean-k-mooney: I hope nobody will start using them more frequently than before, but I'm affraid that having a context popup for emojis will not help | |
| 12:08:09 | stephenfin | sean-k-mooney: https://that.guru/blog/plain-text-in-evolution-3-38/ | |
| 12:08:28 | stephenfin | just FYI, since I think you use Evolution | |
| 12:08:45 | sean-k-mooney | i have plain text enabled | |
| 12:08:50 | sean-k-mooney | not sure if its the same | |
| 12:09:08 | sean-k-mooney | i need to change the line wrapping to something more resonable then 80 though | |
| 12:09:22 | stephenfin | So did I | |