| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-15 | |||
| 19:19:00 | fungi | openstackstatus is not present in here. checking to see if it's fallen victim to a netsplit or something, hold on | |
| 19:22:40 | fungi | efried: okay, try again! | |
| 19:22:53 | efried | #success With the merging of https://review.opendev.org/#/c/696518/ mox is finally purged from nova \o/ | |
| 19:22:54 | openstackstatus | efried: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | |
| 19:22:58 | efried | nice, thanks fungi! | |
| 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 | fungi | announcement was ^ last week after legal was done with trademark searches and risk eval | |
| 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: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 | /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:00:54 | kashyap | Warning, treated as error: | |
| 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 | dansmith | I would think we'd want someone from oslo db land to say they're comfortable with them | |
| 14:25:25 | efried | glad I asked | |
| 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 | |