| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-11-09 | |||
| 05:48:48 | xinranwang | gibi: Hi gibi, thanks for your comments for smartnic spec. For bandiwidth qos, can I assume that the co-existance is not support at the first step. | |
| 05:51:31 | xinranwang | maybe we can add this restirction into document for the first phase | |
| 05:52:18 | xinranwang | Is it acceptable? | |
| 07:50:13 | gibi | xinranwang: yes, such limitation is acceptable for me | |
| 07:59:48 | xinranwang | gibi: great, thanks. I will update the spec soon.:) | |
| 08:00:46 | gibi | I guess you mean coexistens for the same physical device. | |
| 08:54:30 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Reintroduce nova-dsvm-multinode-base https://review.opendev.org/761870 | |
| 09:05:25 | bauzas | good morning Nova | |
| 09:12:02 | gibi | bauzas: o/ | |
| 10:22:54 | openstackgerrit | Lee Yarwood proposed openstack/nova master: rbd: Only log import failures when the RbdDriver is used https://review.opendev.org/761762 | |
| 10:26:26 | lyarwood | stephenfin: ^ hopefully fixed btw | |
| 10:29:53 | lyarwood | gibi / stephenfin / bauzas ; https://review.opendev.org/761870 could use reviews today if you have time, for whatever reason zuul didn't raise an error with the original change, it just removes the job silently. I'll file a bug later for this. | |
| 10:30:05 | stephenfin | already done :) | |
| 10:30:15 | gibi | I will look shortly | |
| 10:30:34 | lyarwood | thanks | |
| 10:40:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add os-volume_attachments reference docs https://review.opendev.org/760971 | |
| 10:41:20 | lyarwood | stephenfin: ^ some basic ref docs for volume attachment btw if you're interested, came up as part of a spec. | |
| 10:41:59 | lyarwood | plan on adding lots of additional detail later in the cycle but the locking part is a good enough start at the moment | |
| 10:42:25 | stephenfin | sure thing, I can take a look | |
| 10:42:28 | lyarwood | thanks | |
| 12:45:52 | openstackgerrit | Merged openstack/nova master: zuul: Reintroduce nova-dsvm-multinode-base https://review.opendev.org/761870 | |
| 13:53:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Warn when starting services with older than N-1 computes https://review.opendev.org/761923 | |
| 13:54:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/761924 | |
| 14:18:03 | stephenfin | lyarwood: Thanks for the review on the 'openstack server evacuate' patch btw. I sent it on its way. Long overdue. | |
| 14:18:38 | stephenfin | I need to go through the API with a fine comb but I _think_ we have the vast majority of the API covered at this point | |
| 14:20:45 | gibi | stephenfin: what is the equvivalent for nova instance-action-list ? I allways fall back to the nova client to do that | |
| 14:20:59 | lyarwood | gibi: openstack server event list | |
| 14:21:01 | lyarwood | I think | |
| 14:21:05 | gibi | /o\ | |
| 14:21:10 | stephenfin | yeah, that | |
| 14:21:23 | gibi | indeed | |
| 14:21:25 | gibi | thanks | |
| 14:21:31 | gibi | I need to build muscle memory for that | |
| 14:21:35 | lyarwood | stephenfin: I don't think we have a version of that for migrations in osc btw | |
| 14:21:45 | stephenfin | lyarwood: we do now | |
| 14:22:08 | lyarwood | oh sweet | |
| 14:22:15 | lyarwood | oh wait do you mean you're going to write it? | |
| 14:22:56 | stephenfin | nope, already done | |
| 14:22:57 | stephenfin | https://review.opendev.org/#/c/675304/ | |
| 14:23:04 | stephenfin | don't know if its released yet, mind, but soon | |
| 14:23:29 | lyarwood | ah right that might be why it's not in the docs yeat | |
| 14:23:31 | lyarwood | yet* | |
| 14:24:18 | gibi | sweet | |
| 14:26:58 | gibi | I guess then we can check our docs and replace the nova CLI exemples with openstack CLI examples: http://paste.openstack.org/show/799826/ | |
| 14:27:27 | gibi | bahh that paste contains duplicates | |
| 14:28:08 | gibi | nah, much better now http://paste.openstack.org/show/799828/ | |
| 14:30:04 | stephenfin | sounds reasonable; I could probably do that this week | |
| 14:30:19 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Create a fixture around fake_notifier https://review.opendev.org/758446 | |
| 14:30:19 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use the non polling notification waiter in func test https://review.opendev.org/758445 | |
| 14:30:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use NotificationFixture for legacy notifications too https://review.opendev.org/758448 | |
| 14:31:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move fake_notifier impl under NotificationFixture https://review.opendev.org/758451 | |
| 14:31:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test the NotificationFixture https://review.opendev.org/758450 | |
| 14:31:50 | gibi | stephenfin: cool. you can plug me in for review | |
| 14:32:25 | gibi | I think the (only) agreement on the PTG about OSC was that we try to fix our docs to use OSC examples | |
| 14:33:00 | brinzhang_ | stephenfin: +1, good start | |
| 14:34:13 | sean-k-mooney | gibi: once we have parity however i really would like to look at deprecating novaclient and requireing osc to be updated | |
| 14:34:35 | sean-k-mooney | we can defer that for a cycle or so but i would like that to be our medium term goal | |
| 14:34:44 | sean-k-mooney | even if we dont do it this cycle | |
| 14:34:53 | stephenfin | sean-k-mooney: that's happening already | |
| 14:35:04 | sean-k-mooney | not quite | |
| 14:35:04 | gibi | sean-k-mooney: agree | |
| 14:35:16 | stephenfin | gtema is doing a lot of work here | |
| 14:35:25 | sean-k-mooney | stephenfin: wehn i say deprecate novaclint i mean stop accpeting any new features/commands being added | |
| 14:35:50 | sean-k-mooney | so its bugfix only and require that they are added to osc instead | |
| 14:36:18 | stephenfin | to the CLI, sure, but we should keep adding to the library code until we're sure openstacksdk is good enough | |
| 14:36:25 | sean-k-mooney | preferably doing one final realse of novaclinet when we freeze the features | |
| 14:36:47 | sean-k-mooney | stephenfin: maybe i was not making that distintion really | |
| 14:36:52 | stephenfin | I'd like to move pretty much all of OSC first, since we should have no issues doing that once openstacksdk is good enough | |
| 14:37:54 | sean-k-mooney | since you have put this effort in to close the gap i just dont want it to reopen | |
| 14:44:14 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.opendev.org/602432 | |
| 14:44:48 | sean-k-mooney | stephenfin: lyarwood can ye take a look at ^ | |
| 14:44:58 | sean-k-mooney | fixed the pep8 issue otherwise its the same | |
| 14:47:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: doc: require openstack client change for every new API microversion https://review.opendev.org/717727 | |
| 14:47:54 | gibi | stephenfin, sean-k-mooney: this ^^ is my contribution to the OSC topic | |
| 14:49:21 | sean-k-mooney | :) | |
| 14:51:50 | gibi | "Delay in Elastic Search: Indexing behind by 141 hours" :( | |
| 14:54:50 | sean-k-mooney | thats only slightly longer then normally its normally 72 hours i think | |
| 14:55:26 | sean-k-mooney | if you are fering to logstash/kibana upstream | |
| 14:55:32 | sean-k-mooney | *refering | |
| 15:01:35 | gibi | it remember when it was close to 0 | |
| 15:01:41 | gibi | last week it hanged around 100 hours | |
| 15:02:34 | sean-k-mooney | it does tend to vary but i think the target was no more the 72 hours | |
| 15:02:45 | sean-k-mooney | it does catch up from time to time | |
| 15:03:45 | gibi | I hope so | |
| 15:13:12 | brinzhang_ | stephenfin: thanks fix that issue from openstack server migration list CLI, I am sure I was tested ti, but the strange thing is that this issue was not found | |
| 15:21:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove compute service level check for qos ops https://review.opendev.org/735570 | |
| 15:21:52 | gibi | stephenfin: I think you will like this code removal patch ^ | |
| 15:32:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Warn when starting services with older than N-1 computes https://review.opendev.org/761923 | |
| 15:42:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/761924 | |
| 15:45:17 | stephenfin | gibi: Done. I assume we're okay to merge things like that now that we've got the service version check? | |
| 15:45:31 | gibi | stephenfin: that is my idea too | |
| 15:45:56 | dansmith | the per-release service version check should catch anything that merged N-2 releases ago, | |
| 15:46:11 | dansmith | so it should be fine to remove older ones and rely on the macro one yeah | |
| 15:46:50 | dansmith | (assuming the qos one is old enough, I didn't look) | |
| 15:47:05 | gibi | qos move ops are added in Ussuri | |
| 15:47:35 | gibi | I mean that last one | |
| 15:48:15 | gibi | so in Victoria we could have removed the service level check. But we did not for extra safety | |
| 15:49:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056 | |
| 15:52:22 | dansmith | stephenfin: we're not really breaking RPC specifically here, because we're not changing any rpc versions or signatures or anything, but this is one of those "not covered by the rpc versions" behaviors.. it's breaking service-to-service interaction, but only for older computes (not even older RPC versions), but for which we've already said isn't supported | |
| 15:53:03 | stephenfin | that makes sense | |