| opendevreview | Alfredo Moralejo proposed openstack/python-watcherclient master: Ignore eventlet deprecation warning messages https://review.opendev.org/c/openstack/python-watcherclient/+/986830 | 07:52 |
|---|---|---|
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove unused methods from KeystoneHelper https://review.opendev.org/c/openstack/watcher/+/987126 | 08:38 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/987127 | 08:38 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-keystoneclient dependency https://review.opendev.org/c/openstack/watcher/+/987128 | 08:38 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Update configuration docs for client sections https://review.opendev.org/c/openstack/watcher/+/987129 | 08:38 |
| opendevreview | Alfredo Moralejo proposed openstack/python-watcherclient master: Ignore eventlet deprecation warning messages https://review.opendev.org/c/openstack/python-watcherclient/+/986830 | 09:27 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/987127 | 09:56 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-keystoneclient dependency https://review.opendev.org/c/openstack/watcher/+/987128 | 09:56 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Update configuration docs for client sections https://review.opendev.org/c/openstack/watcher/+/987129 | 09:56 |
| opendevreview | Alfredo Moralejo proposed openstack/python-watcherclient master: Ignore eventlet deprecation warning messages https://review.opendev.org/c/openstack/python-watcherclient/+/986830 | 09:59 |
| opendevreview | David proposed openstack/watcher-tempest-plugin master: Add comprehensive tests for action precondition validation https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/966860 | 10:02 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add scenario tests for boot from volume instances https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/986616 | 10:33 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Freeze data_model API response fields https://review.opendev.org/c/openstack/watcher/+/986777 | 10:35 |
| amoralej | Hi, I have one doubt about how to check status of VMs in zone_migration - https://github.com/openstack/watcher/blob/master/watcher/decision_engine/strategy/strategies/zone_migration.py#L367-L377 - We are using two attributes vm_state and status coming directly from openstacksdk instance objects | 11:26 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/watcher/+/987127 | 11:28 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove python-keystoneclient dependency https://review.opendev.org/c/openstack/watcher/+/987128 | 11:28 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Update configuration docs for client sections https://review.opendev.org/c/openstack/watcher/+/987129 | 11:28 |
| amoralej | while trying to remove nova api calls, i was trying to mimic those comparison, in model we only have vm_state, so i was adding the status field. However, i realized status is not included in the nova notifications, https://github.com/openstack/nova/blob/2e0bfb1102d8af376c110a169eba42e0905dae07/nova/notifications/objects/instance.py#L25-L61 | 11:28 |
| amoralej | althout there is a power_state | 11:31 |
| jgilaber | looking at the nova api docs https://docs.openstack.org/api-ref/compute/#show-server-details I'm not sure of the difference between vm_state and status | 11:35 |
| amoralej | actually, notifications have another one power_state | 11:36 |
| jgilaber | status="The server status.", OS-EXT-STS:vm_state="The VM state" | 11:36 |
| amoralej | it's weird nova didn't add status which is top level attribute but added the OS-EXT-STS:vm_state ... | 11:37 |
| amoralej | I may call nova to get the status while processing the notifications i tend to think that, if nova folks only add the vm_state there may be a reason .. | 11:45 |
| dviroel | it is wierd indeed, may we could just ping them in the nova channel and ask | 11:46 |
| dviroel | it seems something with historical reason | 11:47 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Update configuration docs for client sections https://review.opendev.org/c/openstack/watcher/+/987129 | 12:45 |
| winiciusallan[m] | people, will we need a blueprint for the preemptible instances spec? | 12:58 |
| winiciusallan[m] | or is just the spec enough | 12:58 |
| dviroel | winiciusallan[m]: both, the blueprint help us to track the development phase | 13:51 |
| winiciusallan[m] | dviroel: ack. created here https://blueprints.launchpad.net/watcher/+spec/preemptible-instances | 13:55 |
| dviroel | winiciusallan[m]: ack, we can approve once we agree on the spec too | 13:57 |
| winiciusallan[m] | good. I put it in drafting state because I'm still writing it | 13:58 |
| winiciusallan[m] | btw I'll submit a first version for early review on initial sections | 13:59 |
| opendevreview | Winicius Allan Bezerra da Silva proposed openstack/watcher-specs master: [WIP] Add spec for Preemptible Instances feature https://review.opendev.org/c/openstack/watcher-specs/+/987171 | 14:00 |
| winiciusallan[m] | here it is ^ | 14:01 |
| dviroel | winiciusallan[m]: thanks. yeah, we can provide an early feedback which is good for both sides | 14:02 |
| winiciusallan[m] | dviroel: thanks! | 14:03 |
| opendevreview | Merged openstack/watcher master: run hacking via local hook https://review.opendev.org/c/openstack/watcher/+/983925 | 16:44 |
| opendevreview | Merged openstack/watcher master: Convert remote pre-commit hooks to local pip hooks https://review.opendev.org/c/openstack/watcher/+/983933 | 17:24 |
| opendevreview | Merged openstack/watcher master: Add hook smoke-test script https://review.opendev.org/c/openstack/watcher/+/983940 | 17:24 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!