| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-09 | |||
| 15:03:56 | opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/zed https://review.opendev.org/c/openstack/os-vif/+/856782 | |
| 15:03:57 | opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Add Python3 antelope unit tests https://review.opendev.org/c/openstack/os-vif/+/856783 | |
| 15:04:03 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856784 | |
| 15:04:04 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856785 | |
| 15:04:06 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856786 | |
| 15:04:07 | opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: Add Python3 antelope unit tests https://review.opendev.org/c/openstack/osc-placement/+/856787 | |
| 15:04:18 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856788 | |
| 15:04:20 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856789 | |
| 15:04:21 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856790 | |
| 15:04:22 | opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: Add Python3 antelope unit tests https://review.opendev.org/c/openstack/python-novaclient/+/856791 | |
| 22:48:32 | opendevreview | Christian Rohmann proposed openstack/nova master: db: Drop redundant indeces on instances and console_auth_tokens tables https://review.opendev.org/c/openstack/nova/+/856757 | |
| 22:56:55 | opendevreview | Christian Rohmann proposed openstack/placement master: Db: Drop redundant indexes for columns already having unique constraint https://review.opendev.org/c/openstack/placement/+/856770 | |
| #openstack-nova - 2022-09-10 | |||
| 09:48:19 | opendevreview | Rajesh Tailor proposed openstack/nova master: Update nova-manage doc page https://review.opendev.org/c/openstack/nova/+/856894 | |
| 09:49:10 | opendevreview | Rajesh Tailor proposed openstack/nova master: Update Availability zone doc page https://review.opendev.org/c/openstack/nova/+/846463 | |
| 10:53:37 | opendevreview | Takashi Natsume proposed openstack/nova master: Update min supported service version for 2023.1 https://review.opendev.org/c/openstack/nova/+/856895 | |
| 18:25:02 | opendevreview | Merged openstack/nova master: Gracefully ERROR in _init_instance if vnic_type changed https://review.opendev.org/c/openstack/nova/+/850003 | |
| #openstack-nova - 2022-09-12 | |||
| 07:11:06 | opendevreview | Christian Rohmann proposed openstack/nova master: db: Drop redundant indeces on instances and console_auth_tokens tables https://review.opendev.org/c/openstack/nova/+/856757 | |
| 07:46:14 | Uggla | Good morning Nova. | |
| 07:51:51 | bauzas | good morning | |
| 07:52:00 | gibi | o/ | |
| 07:52:20 | gibi | Uggla: do you still have a question on my comment on the manila series ? | |
| 07:53:11 | Uggla | hi gibi , no that's ok for the moment. :) | |
| 07:53:30 | gibi | Uggla: cool. Sorry for not responding last week I was deep in some k8s discussions | |
| 07:54:03 | Uggla | gibi, no worries that's fine. | |
| 08:43:05 | sahid | o/ I have specific use-case regarding nova host-evacuate, we would like evacuate host to run but that, all instances scheduled to be forced as shutdown | |
| 08:43:18 | sahid | is there a way to have this happening? | |
| 08:48:13 | sean-k-mooney | no | |
| 08:48:38 | sean-k-mooney | evacuate at the api level result in the vm being evacuated to the same state its in in the db currently | |
| 08:48:46 | sean-k-mooney | so you would need an api change for that. | |
| 08:49:06 | sean-k-mooney | novaclient's shell is deprecated so we are not adding or alterign any commands | |
| 08:49:22 | sean-k-mooney | and nova host-evacuate is intentionally not supported in osc | |
| 08:49:36 | sean-k-mooney | so at this point we should not alter/extend its behavior | |
| 08:50:03 | gibi | I'm wondering what happens if you try to first stop the VM then evacuate it | |
| 08:53:27 | sahid | gibi if the host is down, nothing is happening | |
| 08:53:51 | sahid | the other idea was to extend resetState | |
| 08:54:43 | gibi | I feel like this might be a new microversion to the evacuate action, adding a flag to instruct nova to evacuate but not start the VM on the dest | |
| 08:56:22 | sahid | it's what I was thinking as-well but for host-evacuate it seems that you don't want we make any changes | |
| 08:56:56 | gibi | host-evacuate is a client side concept. You can replace that with a shell script calling the openstack client | |
| 08:57:37 | sahid | side question, why host-evacuate is not supported in openstack client? | |
| 08:57:48 | gibi | what you cannot do is to make a active VM evacuated as stopped via the nova REST API today, hence my microversion thinking | |
| 08:57:51 | sean-k-mooney | sahid: because of what gibi said | |
| 08:58:02 | sean-k-mooney | you its a client side implemation and we did not want to support it any more | |
| 08:58:08 | sean-k-mooney | the error handeling is terrible | |
| 08:58:12 | gibi | sahid: because it is considered orcestration | |
| 08:58:20 | sean-k-mooney | well that too | |
| 08:58:22 | sahid | yes that makes sense, i understand now | |
| 08:58:38 | sean-k-mooney | but more because if one of the evacuation fails its kind of undefiend what the end result of the commnd is | |
| 08:59:01 | sahid | so back to the original use-case, does that would make sense to have evacuate with a flag to force the state? | |
| 08:59:12 | sean-k-mooney | it wont be one of (all evacuated or all still on orginal host) it will be a mix | |
| 08:59:29 | sean-k-mooney | sahid: i would say target state | |
| 08:59:33 | sean-k-mooney | rahter then force | |
| 08:59:46 | sean-k-mooney | that has been requested before at the last inperson ptg i think | |
| 09:00:02 | sean-k-mooney | i would not be apposed to a eveacuate to stopped option | |
| 09:00:12 | sean-k-mooney | im not sure that shelved makes sense | |
| 09:00:21 | sean-k-mooney | but started/stopped i can see | |
| 09:00:26 | gibi | I think target_state enum (AsBefore,Stopped) | |
| 09:00:28 | gibi | make sense | |
| 09:00:44 | gibi | AsBefore=NoChange | |
| 09:01:16 | gibi | can we evacuate a shelved instance? | |
| 09:01:26 | sean-k-mooney | on reset-sate while i would like to expand what it can do so that you can specify somehting other then aviable/error im not sure this is the right way to do this | |
| 09:01:31 | sean-k-mooney | gibi: no | |
| 09:01:38 | sean-k-mooney | gibi: because its not on a host | |
| 09:01:43 | gibi | OK, cool then :D | |
| 09:01:48 | gibi | I started worrying :) | |
| 09:02:01 | sean-k-mooney | im ment it would not make sense to evacuate to shelve_offloaded | |
| 09:02:21 | sean-k-mooney | we could allow shelve_offloading when its down instead | |
| 09:02:27 | sean-k-mooney | but its not really evacuate | |
| 09:02:47 | sean-k-mooney | evaucate is ment to move the vm form one host to another | |
| 09:03:06 | sean-k-mooney | where as shelve/unshleve is moving form on a host to not and vise versa | |
| 09:03:20 | gibi | yeah | |
| 09:03:40 | sean-k-mooney | its kind of a pendantic distinction but i dont quite consider them equal | |
| 09:03:47 | sean-k-mooney | you could argue it either way | |
| 09:04:15 | sean-k-mooney | so i would not be agaisn allowing stop ot work in a host dwonstate by the way | |
| 09:04:30 | sean-k-mooney | you woudl update the db and treat it kind of like local delete | |
| 09:04:43 | sean-k-mooney | when the compute agent comes back up it woudl reconsile the vm state | |
| 09:05:13 | sean-k-mooney | if you stoped it then evacuated that would solve sahid's case | |
| 09:05:20 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/854499 | |
| 09:05:21 | opendevreview | Amit Uniyal proposed openstack/nova master: [compute] always set instnace.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/791135 | |
| 09:05:43 | sahid | sean-k-mooney: yes it's also a possibility | |
| 09:06:56 | sean-k-mooney | the one thing to keep in mind i guess is that even if we allow stop | |
| 09:07:08 | sean-k-mooney | it doen not chnage the responsiblity for the admin | |
| 09:07:28 | sean-k-mooney | that is you are requried as an admin to ensure a host is fenced or all vms are stopped before you evacuate | |
| 09:08:00 | sean-k-mooney | if we allow stop in a down host state the admin still need to ensure it is stoped to prevent data currpption | |
| 09:08:17 | sean-k-mooney | but if they can then that woudl allwo them to evacuate without start the vm again | |
| 09:08:24 | gibi | this is why I would connect the stopping to the evacuation action, that way it is clear that on the source host it is not stopped | |
| 09:08:51 | sean-k-mooney | ack ya that cleaner | |
| 09:09:04 | sean-k-mooney | and the existing check for is it safe to evacute woudl also be checked | |
| 09:09:13 | gibi | yes | |
| 09:09:15 | sean-k-mooney | e.g. the heatbeat has been missed or you set force_down | |
| 09:09:54 | sean-k-mooney | i think johnthetubaguy expressed interest in this in the past | |
| 09:10:17 | sean-k-mooney | or at least supprot for the people that were askign for it in the past | |
| 09:11:09 | sean-k-mooney | oh that reminds me i guess we are not merging my default change? | |
| 09:11:34 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/830829 | |
| 09:12:36 | sean-k-mooney | i would either like to merge that before RC1 or after we create the stable branch | |
| 09:17:38 | sahid | thank you guys - are we agree to extend evacuate with target state (AsBefore, Stopped) ? Can I report a bug with destailled description or should I share a spec? | |
| 09:21:17 | sean-k-mooney | sahid: all api changes require a spec regardless of how trivial | |
| 09:21:25 | sean-k-mooney | this would need a spec and a new microverion | |
| 09:22:24 | sahid | ack I make this happen for A. | |
| 09:22:24 | sean-k-mooney | it can be a pretty short spec but there will at least be a conductor rpc change and likely a compute one too pass the target state | |
| 09:22:43 | sahid | sure no worries | |