| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-15 | |||
| 19:23:05 | fungi | it was disconnected for the past 50 minutes due to some shakeup in freenode | |
| 19:23:11 | fungi | but should be fine now | |
| 19:23:21 | efried | you da person | |
| 19:23:24 | fungi | efried: thanks for communicating the moxtermination! | |
| 19:23:44 | fungi | i think we've all been looking forward to it for a *very* long time | |
| 19:24:23 | fungi | heh | |
| 19:25:21 | sean-k-mooney | fungi: while you are here. do you know what the plan will be regarding ubuntu 20.04 i assume we will move to it sometime in V | |
| 19:26:30 | sean-k-mooney | i guess that decsion wont be made untill closer to the start of V but just said i would ask | |
| 19:26:45 | fungi | sean-k-mooney: it's slated to be released around ussuri rc1 last i looked, so we'll likely have images available before ussuri releases but maybe not as soon as master reopens for victoria commits | |
| 19:27:29 | sean-k-mooney | is it actully victoria. i did not see a poll for the name although i might have missed it | |
| 19:28:17 | fungi | aiui, the tc is ready to declare ubuntu 20.04 a release platform for victoria though: https://review.opendev.org/693743 | |
| 19:29:28 | sean-k-mooney | ok cool we were discussing a kerenl 5.4 feature downstream earlier and the reason i asked is i think 20.04 will be based on 5.4 | |
| 19:30:46 | fungi | sean-k-mooney: yeah, voting on the v cycle name happened in early december, collecting ideas started in mid-november | |
| 19:30:49 | fungi | http://lists.openstack.org/pipermail/openstack-discuss/2019-November/010880.html | |
| 19:30:55 | fungi | http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011947.html | |
| 19:31:19 | sean-k-mooney | ah ok i saw the tread on the proposal but i obviously missed the vote likely when i went on vaction | |
| 19:31:19 | fungi | announcement was ^ last week after legal was done with trademark searches and risk eval | |
| 19:32:07 | fungi | and the w cycle naming process is about to get underway: https://review.opendev.org/702414 | |
| 19:36:37 | sean-k-mooney | cool thanks for the update :) | |
| 19:38:19 | sean-k-mooney | efried: dansmith i currently have sundars latest update stacking. ill likely test it out tomorow however since im about to call it a day | |
| 19:38:28 | dansmith | cool | |
| 20:02:27 | openstackgerrit | Merged openstack/os-traits master: Add a trait that compute manager can handle accelerator requests. https://review.opendev.org/698961 | |
| 20:33:58 | openstackgerrit | Merged openstack/nova master: nova-net: Remove unused exceptions https://review.opendev.org/697149 | |
| 20:34:03 | openstackgerrit | Merged openstack/nova master: nova-net: Remove unnecessary 'neutronv2' prefixes https://review.opendev.org/696776 | |
| 20:42:59 | openstackgerrit | waleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support https://review.opendev.org/665965 | |
| 20:46:05 | efried | lyarwood: I updated your os-traits release patch to pick up COMPUTE_ACCELERATORS which just merged. | |
| 20:46:53 | sean-k-mooney | ya i had to use libs_from_git to pull that in for testing the cyborg changes so that would be useful | |
| 21:27:37 | openstackgerrit | Merged openstack/nova stable/rocky: Improve metadata server performance with large security groups https://review.opendev.org/697517 | |
| 21:28:44 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 21:28:45 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 21:45:34 | efried | msg diablo_rojo | |
| 21:45:38 | efried | sigh | |
| 21:46:25 | openstackgerrit | Merged openstack/nova stable/stein: block_device: Copy original volume_type when missing for snapshot based volumes https://review.opendev.org/696686 | |
| 22:18:06 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 22:49:04 | efried | dustinc: ready for re-review? | |
| #openstack-nova - 2020-01-16 | |||
| 00:58:18 | openstackgerrit | Merged openstack/nova master: libvirt: Remove MIN_QEMU_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/695915 | |
| 01:39:19 | xiaolin | hello guys, I'm trying to run tempest test "rally verify start --pattern set=volume", but got an error "Forbidden: Policy doesn't allow os_compute_api:os-assisted-volume-snapshots:create to be performed. (HTTP 403)", can anyone give me some advice ? | |
| 01:39:31 | xiaolin | when i modify base.RULE_ADMIN_API to base.RULE_ANY in /usr/lib/python2.7/site-packages/nova/policies/assisted_volume_snapshots.py, the test passed | |
| 01:39:48 | xiaolin | Is this permission setting reasonable? | |
| 01:59:56 | openstackgerrit | wangjiajing proposed openstack/nova master: Optimize the comments for the "test_resize_cross_cell_weigher_filtered_to_target_cell_by_spec" method. https://review.opendev.org/702781 | |
| 02:05:00 | alex_xu | xiaolin: that API I thought should be called by cinder side, probably can go to check the cinder side's nova config, whether it is admin user | |
| 04:33:17 | openstackgerrit | Brin Zhang proposed openstack/nova master: FUP Populate the 'updated_at' when aggregate metadata is updated https://review.opendev.org/702790 | |
| 06:15:06 | LiangFang | hi folks, regarding the volume local cache spec. https://review.opendev.org/#/c/689070/ | |
| 06:16:37 | LiangFang | I'm thinking to simply the logic. If the compute node have cache capability, then leverage it. If not have the capability, then just go ahead without caching. | |
| 06:17:31 | LiangFang | We can use aggregate to group the servers that has cache capability | |
| 06:20:17 | LiangFang | this way is not perfect for every scenarios, e.g. only support libvirt, not schedule by flavor. But in some private cloud, it is simple and useful. | |
| 06:25:25 | LiangFang | in the cinder spec https://review.opendev.org/#/c/684556/, lots of concerns if we fail the attach volume operation | |
| 07:51:12 | openstackgerrit | Marek Lyčka proposed openstack/nova-specs master: Enable REST quiescing https://review.opendev.org/702810 | |
| 08:07:16 | openstackgerrit | Marek Lyčka proposed openstack/nova-specs master: Enable REST quiescing https://review.opendev.org/702810 | |
| 08:21:39 | lyarwood | efried: ack thanks for that, I missed that another trait had or at least was about to land. | |
| 08:26:28 | openstackgerrit | Marek Lyčka proposed openstack/nova-specs master: Enable REST quiescing https://review.opendev.org/702810 | |
| 08:47:05 | gibi | dansmith: hi! stephenfin would like you to look at one of the qos patch https://review.opendev.org/#/c/702261/ and if you are there already the rest is also ready for review | |
| 09:14:47 | openstackgerrit | Marek Lyčka proposed openstack/nova-specs master: Enable REST quiescing https://review.opendev.org/702810 | |
| 09:22:14 | stephenfin | efried: Probably would have waited til https://review.opendev.org/#/c/702440/ was merged to mark that BP as complete, personally - I'd like to get the whole thing closed out | |
| 09:22:26 | stephenfin | but hey, the hardest work is done | |
| 09:31:26 | openstackgerrit | Riccardo Pittau proposed openstack/nova master: Fix typos for update_available_resource reference https://review.opendev.org/702602 | |
| 09:39:42 | stephenfin | yo, sean-k-mooney, looks like you added https://github.com/openstack/nova/blob/b44b540fc70504f3869ef23022642095de0ea99e/nova/virt/libvirt/vif.py#L299-L301 | |
| 09:41:41 | stephenfin | sean-k-mooney: but that function has existed since it was removed in commit 92323586b5d03b31c18657ad64646a8ce3b8a742 | |
| 09:42:02 | stephenfin | sean-k-mooney: so that means that code is not being called by anything yet. In either case, want to work on the fix or will I? | |
| 09:42:40 | stephenfin | sean-k-mooney: fwiw, I'm running pytype over the code base to see what it reveals. That's one of two actual issues it has found so far | |
| 09:42:51 | openstackgerrit | Marek Lyčka proposed openstack/nova-specs master: Enable REST quiescing https://review.opendev.org/702810 | |
| 10:00:46 | kashyap | stephenfin: Hey, any clue how to resolve this "duplicate label" thing?: | |
| 10:00:54 | kashyap | Warning, treated as error: | |
| 10:00:54 | kashyap | /home/zuul/src/opendev.org/openstack/nova-specs/doc/source/specs/ussuri/approved/allow-secure-boot-for-qemu-kvm-guests.rst:72:duplicate label proposed change, other instance in /home/zuul/src/opendev.org/openstack/nova-specs/doc/source/specs/train/approved/allow-secure-boot-for-qemu-kvm-guests.rst | |
| 10:01:00 | kashyap | --- | |
| 10:04:03 | stephenfin | kashyap: Use anonymous labels instead | |
| 10:04:10 | stephenfin | `foo`__ | |
| 10:04:17 | stephenfin | __ https://example.com/ | |
| 10:04:33 | stephenfin | (double underscore) | |
| 10:04:38 | kashyap | stephenfin: Ah-ha! you, sir, are the Main Man! | |
| 10:07:19 | kashyap | Okay, good to know: "... However, this feature [Anonymous Hyperlinks] is easily abused and can result in unreadable plaintext and/or unmaintainable documents. Caution is advised." | |
| 10:08:58 | kashyap | stephenfin: Oh, wait -- | |
| 10:09:11 | kashyap | stephenfin: I see that anonymous labels are only for URLs | |
| 10:09:23 | kashyap | stephenfin: I'm trying to refer to a section in the _same_ document; thus no URL here | |
| 10:11:46 | kashyap | Here is a small concrete example: http://paste.openstack.org/show/788455/ | |
| 10:20:27 | kashyap | Okay, that works. | |
| 10:20:52 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Ussuri https://review.opendev.org/693844 | |
| 12:29:07 | openstackgerrit | waleed mousa proposed openstack/os-vif master: Add delete_netdev field in VIFPortProfileOpenVSwitch profile https://review.opendev.org/702857 | |
| 12:38:15 | openstackgerrit | waleed mousa proposed openstack/os-vif master: Add delete_netdev field in VIFPortProfileOpenVSwitch profile https://review.opendev.org/702857 | |
| 12:53:25 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Ussuri https://review.opendev.org/693844 | |
| 13:00:31 | openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/nova master: Add ironic hypervisor doc https://review.opendev.org/701998 | |
| 13:36:43 | efried | stephenfin: I'm confused. Is https://review.opendev.org/#/c/702440/ related to mox somehow? | |
| 13:36:51 | efried | I marked the mox blueprint complete, not the nova-net one. | |
| 13:37:02 | efried | ...at least that's what I meant to do... | |
| 13:38:02 | efried | lyarwood: I don't think you missed anything; that other trait merged later. I just wanted to pick it up in the same release since the timing worked out. | |
| 13:54:43 | efried | stephenfin: tidy docs patch for the ironic virt driver, should be an easy +A for you: https://review.opendev.org/#/c/701998/ (Note that more is possible, but the information there is accurate afaict and it has nod from ironic cores) | |
| 14:08:38 | iurygregory | efried, thanks for the +2 =) | |
| 14:09:05 | efried | iurygregory: yahyoubetcha. Appreciate the patch. | |
| 14:09:54 | iurygregory | \o/ | |
| 14:14:57 | efried | dansmith: I started to look at https://review.opendev.org/#/c/580271/ and successor and they seem fine, but I don't have the confidence to approve. Would you please cast your discerning eye? Should be an easy +A for you, if it's right. | |
| 14:25:07 | dansmith | efried: so I've always been told we shouldn't mess with those fields from python land, so I'm not confident to +W that either | |
| 14:25:25 | efried | glad I asked | |
| 14:25:25 | dansmith | I would think we'd want someone from oslo db land to say they're comfortable with them | |
| 14:25:48 | dansmith | maybe stephenfin has already consulted the expert or is an expert himself since he's already +2? | |
| 14:26:27 | stephenfin | I was expecting SQLA to handle those fields for us, tbh, but I couldn't find anything that suggested it would | |
| 14:28:07 | stephenfin | I didn't think about looking in oslo.db though /o\ /me goes to investigate | |
| 14:30:43 | dansmith | the update *is* handled by SQLA fwiw | |
| 14:30:49 | sean-k-mooney | dansmith: efried it looks like there are some issues with the arq binding on the cyborg side http://paste.openstack.org/show/788470/ | |
| 14:31:18 | stephenfin | Yup, have spotted the use of 'TimestampMixin' in NovaBase | |
| 14:32:05 | efried | gibi: I'd like to queue up some review time from you for the provider config file bp if possible please. It's pretty straightforward stuff, kind of fun to look at IMO. It's very close to ready, if you want to preview: https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/provider-config-file | |