| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-02 | |||
| 17:04:58 | sean-k-mooney | rm_work: ok well i can see if we can turn that into a proper patch with some tests | |
| 17:05:07 | rm_work | well I +1'd | |
| 17:05:09 | rm_work | lol | |
| 17:05:40 | sean-k-mooney | well it work but i want to put a unit test that asssert we dont call the thing that explode when session is None | |
| 17:06:13 | rm_work | yep, makes sense | |
| 17:06:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 17:08:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 17:13:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 17:13:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 17:16:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 17:18:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 17:21:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 17:22:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 17:30:48 | opendevreview | sean mooney proposed openstack/osc-placement master: default to max version when no session https://review.opendev.org/c/openstack/osc-placement/+/794276 | |
| 17:33:28 | rm_work | \o/ | |
| 17:35:22 | sean-k-mooney | lol i broke the docs because thre app is non | |
| 17:35:49 | sean-k-mooney | File "/home/zuul/src/opendev.org/openstack/osc-placement/osc_placement/version.py", line 141, in get_version | |
| 17:35:51 | sean-k-mooney | if obj.app.client_manager.session is None: | |
| 17:35:53 | sean-k-mooney | AttributeError: 'NoneType' object has no attribute 'client_manager' | |
| 17:37:49 | opendevreview | sean mooney proposed openstack/osc-placement master: default to max version when no session https://review.opendev.org/c/openstack/osc-placement/+/794276 | |
| 17:38:07 | sean-k-mooney | rm_work: ok ^ should be the final patch unless peopel ask for feedback | |
| 17:38:12 | sean-k-mooney | well provide feedback | |
| 17:38:35 | sean-k-mooney | gibi: can you add https://review.opendev.org/c/openstack/osc-placement/+/794276 to your review queue | |
| 17:39:08 | rm_work | would docs not also want latest? | |
| 17:39:44 | sean-k-mooney | maybe | |
| 17:39:50 | rm_work | but, starts to be hard to code for every little case like that when you're basically making an inference about the caller | |
| 17:39:59 | sean-k-mooney | but they were previouly getting the oldest | |
| 17:40:01 | rm_work | not "if for_docs:" | |
| 17:40:06 | rm_work | yeah... | |
| 17:40:22 | sean-k-mooney | i can do this a different way | |
| 17:40:30 | sean-k-mooney | i can catch the other error type | |
| 17:40:40 | sean-k-mooney | PluginAtributeERROR | |
| 17:41:05 | sean-k-mooney | but i dont like that since it actully trys to connect the cloud | |
| 17:41:37 | stephenfin | melwitt: Won't have a chance now (I'm just finishing up) but I think I didn't have any concerns of my own, so if you're happy to +W then go for it :) | |
| 17:42:20 | melwitt | cool, thanks stephenfin | |
| 17:42:23 | sean-k-mooney | rm_work: let me see what the docs are using it for | |
| 17:48:39 | sean-k-mooney | rm_work: its from cliff/sphinxext.py | |
| 17:49:14 | rm_work | so not the client auto-doc stuff? | |
| 17:49:22 | rm_work | or that is it | |
| 17:49:41 | sean-k-mooney | well kind of | |
| 17:49:53 | sean-k-mooney | its form _generate_nodes_per_command | |
| 17:50:06 | sean-k-mooney | so it is the auto generation of docs for cliff commands | |
| 17:50:36 | sean-k-mooney | previously that was raising an atribute error but we were catching it and returning the oldest version | |
| 17:55:27 | sean-k-mooney | rm_work: lol i may have missed that comment https://review.opendev.org/c/openstack/osc-placement/+/794276/5/osc_placement/version.py#146 | |
| 17:55:37 | sean-k-mooney | that is why the try is there | |
| 17:56:21 | rm_work | heh yeah | |
| 17:56:49 | rm_work | I would say "let's use the maximum one" but that's just me | |
| 17:57:22 | opendevreview | Merged openstack/nova master: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/784166 | |
| 17:57:34 | sean-k-mooney | ya that would be my prefernce too | |
| 17:58:12 | rm_work | might be a bigger change than just "bugfix regression on bash completion" | |
| 17:58:56 | opendevreview | Merged openstack/nova-specs master: Allow provider re-parenting in placement https://review.opendev.org/c/openstack/nova-specs/+/788243 | |
| 17:59:25 | sean-k-mooney | rm_work: ya for now im not going to chagne it | |
| 17:59:34 | sean-k-mooney | but i left a review comment | |
| 18:00:49 | rm_work | yep, left a reply too :D | |
| 18:31:36 | opendevreview | Elod Illes proposed openstack/nova stable/wallaby: [stable-only] Fix ceph install in live migration hook https://review.opendev.org/c/openstack/nova/+/794000 | |
| 18:35:15 | opendevreview | Rodrigo Barbieri proposed openstack/nova stable/wallaby: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/794328 | |
| 22:13:49 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: zuul: Replace grenade and nova-grenade-multinode with grenade-multinode https://review.opendev.org/c/openstack/nova/+/794345 | |
| 22:32:55 | tosky | melwitt: crossing fingers :) | |
| 22:33:13 | melwitt | :) | |
| #openstack-nova - 2021-06-03 | |||
| 01:19:17 | gmann | melwitt: replied on this - https://review.opendev.org/c/openstack/nova/+/762013/2/doc/source/configuration/policy-concepts.rst#220 | |
| 01:20:03 | gmann | melwitt: I agree on your point. I will add the current limitation for force_host and requested_destination | |
| 01:20:41 | gmann | zero_disk and external-network seems good. later one depends on neutron policy but that is open for project reader also | |
| 01:37:05 | opendevreview | Ghanshyam proposed openstack/nova master: Remove PROJECT_ADMIN limitation from zero-disk and external-network policy https://review.opendev.org/c/openstack/nova/+/794360 | |
| 01:38:07 | gmann | melwitt: ^^ | |
| 02:18:06 | opendevreview | Ghanshyam proposed openstack/nova master: Improve policy doc for supported scope info https://review.opendev.org/c/openstack/nova/+/762013 | |
| 02:18:10 | gmann | melwitt: stephenfin ^^ updated | |
| 05:06:39 | opendevreview | melanie witt proposed openstack/nova master: Make test_refresh_associations_* deterministic https://review.opendev.org/c/openstack/nova/+/794396 | |
| 05:10:22 | melwitt | gmann: did you mean to remove the table when you went from PS2 to PS3? https://review.opendev.org/c/openstack/nova/+/762013/2..3/doc/source/configuration/policy-concepts.rst#b307 | |
| 05:17:05 | melwitt | added a comment on the review ^ | |
| 07:44:13 | bauzas | hmpf, reminder that we have an office hour in 15 mins , I guess | |
| 07:58:04 | gibi | yes | |
| 07:58:09 | gibi | in 2 mins :) | |
| 07:58:17 | gibi | ... | |
| 07:58:17 | gibi | I'm almost ready to start | |
| 08:00:16 | gibi | #startmeeting nova_extra | |
| 08:00:17 | opendevmeet | Meeting started Thu Jun 3 08:00:16 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 08:00:18 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 08:00:20 | opendevmeet | The meeting name has been set to 'nova_extra' | |
| 08:00:58 | gibi | welcome o/ | |
| 08:01:16 | suzhengwei__ | \o/ | |
| 08:01:38 | XinxinShen | o/ | |
| 08:01:51 | gibi | ... waiting a minute to lets everybody join | |
| 08:03:24 | gibi | So a quick summary. We agree to have this meeting time slot one a Months in every first Thursday here in #openstack-nova | |
| 08:03:33 | gibi | s/one/once/ | |
| 08:04:04 | gibi | but we haven't talked about yet what would be a good format for it | |
| 08:04:19 | gibi | should we go through the normal meeting agenda | |
| 08:04:33 | gibi | talking about bugs, CI, release, stable, | |
| 08:04:44 | gibi | or just go with an open agenda, like in an office hour | |
| 08:05:06 | gibi | what is the preference of the people present? | |
| 08:05:14 | gibi | I can do both if needed | |
| 08:07:07 | suzhengwei__ | This is the first extra meeting. My colleague didn't prepare for it enough. | |
| 08:07:18 | gibi | suzhengwei__: no worries | |
| 08:07:56 | suzhengwei__ | Open agenda will be all right. | |
| 08:08:09 | gibi | If no oppinions about the format then I would go with the simple open agenda | |
| 08:08:19 | gibi | suzhengwei__: cool, then we are on the same page | |
| 08:08:23 | suzhengwei__ | ok | |
| 08:08:27 | gibi | and we can change later if needed | |
| 08:08:59 | gibi | I have one agenda point for today | |
| 08:09:47 | gibi | just want to refresh the process we have in nova about the release | |
| 08:09:56 | gibi | so we just had Milestone 1 last week | |