| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-06 | |||
| 10:08:08 | sean-k-mooney | i think it would be in the release notes but i can quickly check the code | |
| 10:08:38 | alexe9191 | Fantastic, if I could also know which file you are checking in the source tree that would be great as well. | |
| 10:09:07 | sean-k-mooney | we have each verion bump documented with a comment https://github.com/openstack/nova/blame/master/nova/compute/rpcapi.py#L76 | |
| 10:09:37 | sean-k-mooney | https://github.com/openstack/nova/blame/master/nova/compute/rpcapi.py#L385 | |
| 10:09:42 | alexe9191 | Awesome! | |
| 10:09:45 | sean-k-mooney | the aliase are the quickest way to check | |
| 10:10:23 | alexe9191 | Interesting, so K to P is on the same Major RPC release. | |
| 10:10:32 | sean-k-mooney | so kilo was 4.0 | |
| 10:10:39 | sean-k-mooney | rocky was 5.0 | |
| 10:10:57 | sean-k-mooney | technically 5.0 is the ame as 4.22 | |
| 10:11:46 | sean-k-mooney | so if rocky still has the 4.22 compat code then you could pin the rpc version to 4.0 in principal | |
| 10:11:56 | sean-k-mooney | but it proably will have bugs | |
| 10:12:36 | alexe9191 | This is really helpful! | |
| 10:13:35 | sean-k-mooney | honestly i doubt we actully manged to keep perfect rpc compate and i suspect rocky has had the 4.x compat code removed | |
| 10:14:31 | sean-k-mooney | but looking quickly it looks like the compat code was maybe removed in stine | |
| 10:14:58 | sean-k-mooney | without testign it first however i would not bet on it it fully working | |
| 10:15:07 | alexe9191 | If I am not mistaken, this could mean that in theory I could run Newton API on kilo nova compute. Since the RPC API is on the same major version. | |
| 10:15:39 | sean-k-mooney | in theory yes | |
| 10:16:02 | sean-k-mooney | in practice you would likely be the first to try that and you would have to let us know if it works :) | |
| 10:16:27 | alexe9191 | I will most definitely do that :) | |
| 10:16:33 | sean-k-mooney | you would pin the RPC version of the newton contolplane to 4.0 | |
| 10:16:49 | alexe9191 | This should be done in nova.conf I could imagine ? | |
| 10:16:50 | sean-k-mooney | you can do that via the upgrade levels in the nova.conf | |
| 10:19:02 | alexe9191 | I am gonna let you guys know once I test that! Thanks a lot for all of the helpful information. | |
| 10:19:44 | bauzas | sean-k-mooney: alexe9191: well, Kilo is very prehistoric | |
| 10:20:09 | bauzas | and from the RPC PoV, we only started to introduce massively nova objects by this time | |
| 10:20:15 | sean-k-mooney | it is but so is newton :P | |
| 10:20:33 | bauzas | at newton, we mostly stopped to use dicts on RPC calls | |
| 10:20:48 | bauzas | while during kilo, most of the values were those | |
| 10:20:50 | sean-k-mooney | mostly but we still have a few even on master | |
| 10:21:21 | bauzas | speaking of RPC compat, I'd say a nova boot *could* work | |
| 10:21:22 | sean-k-mooney | bauzas: having random dicts of stings migth actully help in this case | |
| 10:21:33 | sean-k-mooney | provide the required data is there | |
| 10:21:51 | bauzas | maybe | |
| 10:22:48 | bauzas | anyway, all of this is very experimental | |
| 10:23:25 | bauzas | but I sincerely hope alexe9191 is not expecting live-migration to work with this env :) | |
| 10:23:26 | sean-k-mooney | oh yes it is. | |
| 10:26:15 | alexe9191 | bauzas no and this is not a production environment where I will be testing. Our of curiosity i am going to run an experiment but for production I am probably gonna do the rolling upgrade. | |
| 10:28:31 | bauzas | alexe9191: anyway, I'm very curious about your experiment results :) | |
| 10:29:11 | bauzas | I'm personnally not opposed to have operators breaking the N+1 rule by running very old computes if they wish | |
| 10:29:44 | bauzas | provided they understand the maintenance price | |
| 10:31:15 | bauzas | but... Kilo/Rocky is like a gymnast front oversplit to me :) | |
| 10:31:40 | bauzas | front side* even | |
| 10:31:42 | sean-k-mooney | one of the cost is really you can only safely use the max api version of the oldest compute | |
| 10:31:50 | sean-k-mooney | as in microversion | |
| 10:32:08 | sean-k-mooney | you might be able to use later microverions but if they needed an rpc change it wont work | |
| 10:32:28 | sean-k-mooney | we will backlevel the request but you wont get teh behavior the new microverion implies | |
| 10:33:10 | gibi | stephenfin: I finished reviewing the migration compacting series, nice work! | |
| 10:33:18 | stephenfin | gibi: Thanks :) | |
| 10:33:42 | gibi | I have couple of questions in the middle but overall I think we are good | |
| 10:33:44 | stephenfin | If you want me to review anything, just let me know. I'm almost done on the os-hypervisors fixup (many rabbit holes) | |
| 10:33:58 | stephenfin | Sweet. Will take a look shortly | |
| 10:35:09 | sean-k-mooney | bauzas: by the way i have not got around to doing the full code review for the cyborg shevle patches but did my respoce make sense | |
| 10:35:20 | sean-k-mooney | regarding the conductor chagnes | |
| 10:37:24 | bauzas | sean-k-mooney: I'll look at them later today, I was on a meeting just before and I want to rebase my own routed networks series | |
| 10:37:37 | sean-k-mooney | no worries | |
| 10:37:38 | bauzas | in other words, I lag :D | |
| 10:38:46 | gibi | stephenfin: there is a trivial review https://review.opendev.org/c/openstack/nova/+/769467 you can lok at | |
| 10:39:06 | sean-k-mooney | hehe well i was indening to do a fully review yesterady and havent started other then to respond ot your comments so your still ahead of me | |
| 10:39:15 | stephenfin | gibi: have it open already :) | |
| 10:40:34 | gibi | besides that I have nothing :) | |
| 10:40:45 | gibi | I need to do a bunch of rebases | |
| 10:49:14 | sean-k-mooney | speaking of rebases i wil be rebasing my vdpa spec in a day or so but if people have time it would be nice if ye could review that spec https://review.opendev.org/c/openstack/nova-specs/+/764999 and also the numa port polices spec https://review.opendev.org/c/openstack/nova-specs/+/765901 | |
| 10:50:03 | sean-k-mooney | i intend to start pushing some standalone patches for those this week namely the xml generation for vdpa and some constants for the newtuon extensions | |
| 10:51:25 | sean-k-mooney | the larger change will still be a few weeks out but i would like to start submitting an merging some of the smaller patches which would be dead code but easilly reviewable while i work on the more complex part but the spec need to be approved first before that can happen | |
| 11:07:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Create a fixture around fake_notifier https://review.opendev.org/c/openstack/nova/+/758446 | |
| 11:07:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use the non polling notification waiter in func test https://review.opendev.org/c/openstack/nova/+/758445 | |
| 11:07:51 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use NotificationFixture for legacy notifications too https://review.opendev.org/c/openstack/nova/+/758448 | |
| 11:09:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test the NotificationFixture https://review.opendev.org/c/openstack/nova/+/758450 | |
| 11:09:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move fake_notifier impl under NotificationFixture https://review.opendev.org/c/openstack/nova/+/758451 | |
| 11:16:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Drop statistics-style fields from os-hypervisors https://review.opendev.org/c/openstack/nova/+/764040 | |
| 11:16:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Normalize exception handling for os-hypervisors https://review.opendev.org/c/openstack/nova/+/769520 | |
| 11:16:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Remove 'test_extended_hypervisors' https://review.opendev.org/c/openstack/nova/+/769519 | |
| 11:17:13 | stephenfin | gibi: Done ^ Sorry for the delay | |
| 11:17:22 | gibi | on it | |
| 11:29:10 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068 | |
| 11:29:10 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add requested_networks field to RequestSpec object https://review.opendev.org/c/openstack/nova/+/749977 | |
| 11:43:00 | sean-k-mooney | does anywone know if shareing resouce providers actully work by the way | |
| 11:43:36 | sean-k-mooney | its in relateion to http://lists.openstack.org/pipermail/openstack-discuss/2021-January/019647.html | |
| 11:43:48 | sean-k-mooney | specifically for the RBD image backend | |
| 11:44:28 | sean-k-mooney | if we used provider.yaml to set the local disk_gb inventory to 0 and created a shareing resocue provider with a disk_gb inventory for the ceph pool | |
| 11:44:55 | lyarwood | I don't think anyone was working on that | |
| 11:45:02 | sean-k-mooney | then added the MISC_SHARES_VIA_AGGREGATE trait to the RP and adedd the RP and the comptue to an aggreate would it work | |
| 11:45:14 | sean-k-mooney | lyarwood: they have not been for like 2 years | |
| 11:45:25 | sean-k-mooney | but i tought the placment side was done | |
| 11:45:37 | sean-k-mooney | i just dont know if anyone has ever tested it | |
| 11:45:58 | gibi | sean-k-mooney: we don't have functional or other integration tests for it so I assume that it does not work | |
| 11:46:12 | lyarwood | no idea sorry | |
| 11:46:17 | sean-k-mooney | hehe ya that my base assumtion with openstack too | |
| 11:46:21 | sean-k-mooney | if its not tested its broken | |
| 11:47:07 | sean-k-mooney | if it did work technially operators/installer tooll could configure it and actully get correct reporting for shared storage this way | |
| 11:47:55 | sean-k-mooney | the real question is will the placement allocation candiate include the inventory form the sharing provider | |
| 11:48:04 | sean-k-mooney | using the request we currently make | |
| 11:48:27 | sean-k-mooney | it "should" but i have no idea if it will actully work | |
| 11:49:30 | sean-k-mooney | placment has functrion api test for shared resouces https://github.com/openstack/placement/blob/master/placement/tests/functional/gabbits/shared-resources.yaml | |
| 11:50:51 | sean-k-mooney | https://github.com/openstack/placement/blob/master/placement/tests/functional/gabbits/shared-resources.yaml#L135-L143 | |
| 11:50:57 | sean-k-mooney | it looks like it should work | |
| 11:51:09 | sean-k-mooney | if you read those top to bottom | |
| 11:52:44 | gibi | sean-k-mooney: I'm not sure that our resize and migrate logic works properly with sharing providers | |
| 11:52:46 | sean-k-mooney | if i had time i would really like to test that and write that up in docs because it would be greate do document how to correctly deploy using sharing agggreats to model storage | |