| yoctozepto | #startmeeting masakari | 06:03 |
|---|---|---|
| opendevmeet | Meeting started Tue Jul 27 06:03:13 2021 UTC and is due to finish in 60 minutes. The chair is yoctozepto. Information about MeetBot at http://wiki.debian.org/MeetBot. | 06:03 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 06:03 |
| opendevmeet | The meeting name has been set to 'masakari' | 06:03 |
| yoctozepto | o/ | 06:03 |
| jopdorp | o/ | 06:03 |
| suzhengwei | o/ | 06:03 |
| XinxinShen | o/ | 06:05 |
| yoctozepto | ok, let's start | 06:08 |
| yoctozepto | agenda for today | 06:08 |
| yoctozepto | * CI status | 06:08 |
| yoctozepto | * Important pending reviews (important bugfixes, backports) | 06:08 |
| yoctozepto | * Next release planning | 06:08 |
| yoctozepto | ** Cleaned up blueprints https://blueprints.launchpad.net/masakari Watch for priority-assignee-series triple. | 06:08 |
| yoctozepto | ** [shenxinxin, suzhengwei] vm evacuations for host recovery https://review.opendev.org/c/openstack/masakari-specs/+/789432 | 06:08 |
| yoctozepto | * Open discussion | 06:08 |
| yoctozepto | #topic CI status | 06:08 |
| yoctozepto | I used it a bit recently and it seems green still | 06:09 |
| yoctozepto | good winds | 06:09 |
| yoctozepto | #topic Important pending reviews (important bugfixes, backports) | 06:09 |
| yoctozepto | https://review.opendev.org/c/openstack/masakari-monitors/+/794162 | 06:09 |
| yoctozepto | thanks suzhengwei for re-reviewing | 06:09 |
| yoctozepto | merging this now | 06:09 |
| yoctozepto | one note - please backport only *after* the change merges on master to get the proper commit id | 06:10 |
| suzhengwei | I have chery-pick it to train/victoria/ussuri. | 06:10 |
| yoctozepto | another note - remember we have also stable/wallaby now and train is in EM so there is no urgency to patch it | 06:11 |
| yoctozepto | in other words, let's wait for the patch to merged and then retry the cherrypicks please | 06:11 |
| suzhengwei | ok | 06:11 |
| yoctozepto | ok, another important patch https://review.opendev.org/c/openstack/masakari/+/796732 | 06:12 |
| yoctozepto | and two others - ready bandit patches: | 06:13 |
| yoctozepto | https://review.opendev.org/c/openstack/masakari-dashboard/+/791922 | 06:13 |
| yoctozepto | https://review.opendev.org/c/openstack/python-masakariclient/+/791924 | 06:13 |
| yoctozepto | I am also merging this old trivialfix by XinxinShen https://review.opendev.org/c/openstack/python-masakariclient/+/793595 | 06:14 |
| yoctozepto | I think I have gone through all of them now | 06:14 |
| yoctozepto | I will give you a minute or two to go over them | 06:14 |
| yoctozepto | and then we move onto the next release planning | 06:15 |
| yoctozepto | ok, I hope you had the time to get acquainted with them | 06:18 |
| yoctozepto | please review | 06:18 |
| yoctozepto | #topic Next release planning | 06:18 |
| yoctozepto | two subtopics in here | 06:18 |
| yoctozepto | first of all | 06:18 |
| yoctozepto | Cleaned up blueprints https://blueprints.launchpad.net/masakari Watch for priority-assignee-series triple. | 06:18 |
| yoctozepto | I also mailed to the openstack-discuss mailing list about that | 06:18 |
| yoctozepto | http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023811.html | 06:19 |
| yoctozepto | this gives a better overview over what's really happening in the Masakari world | 06:19 |
| yoctozepto | please try to keep it up to date | 06:19 |
| yoctozepto | any questions so far? | 06:21 |
| opendevreview | Merged openstack/python-masakariclient master: Replace deprecated UPPER_CONSTRAINTS_FILE variable https://review.opendev.org/c/openstack/python-masakariclient/+/793595 | 06:21 |
| yoctozepto | ok | 06:22 |
| yoctozepto | the next topic is | 06:22 |
| yoctozepto | [shenxinxin, suzhengwei] vm evacuations for host recovery https://review.opendev.org/c/openstack/masakari-specs/+/789432 | 06:22 |
| yoctozepto | I have left a lot of comments there | 06:22 |
| yoctozepto | please ask me any questions to help resolve them | 06:22 |
| yoctozepto | we also have a thread on the openstack-discuss ml on enhancing nova: http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023819.html | 06:23 |
| opendevreview | Merged openstack/masakari-monitors master: Fix hostmonitor hanging forever after certain exceptions https://review.opendev.org/c/openstack/masakari-monitors/+/794162 | 06:23 |
| suzhengwei | I have read the mail. | 06:24 |
| suzhengwei | My opnion, we can still push forward the work without nova-api changed. | 06:26 |
| yoctozepto | probably, we can at least create the data structure as I described | 06:27 |
| suzhengwei | For priority evcuation, it will need firshly calculate and record the instance in masakari DB, and then evacuation. | 06:28 |
| yoctozepto | indeed | 06:30 |
| yoctozepto | so it starts with null migration_uuid anyway | 06:30 |
| suzhengwei | It need to create the data structure between notifications and instancese. | 06:30 |
| suzhengwei | It is ok without migration_uuid. | 06:31 |
| yoctozepto | ok | 06:33 |
| yoctozepto | let me see the refreshed spec then | 06:33 |
| yoctozepto | perhaps we just have most of what we need anyway | 06:33 |
| suzhengwei | Moreover, it doesn't used migration_uuid to check whether evacuation finished. | 06:33 |
| yoctozepto | though migration_uuid would be great for provenance | 06:34 |
| yoctozepto | yeah, we need that info for quick queries | 06:34 |
| yoctozepto | so it is stored in our database anyhow | 06:34 |
| suzhengwei | yes. | 06:35 |
| yoctozepto | ok | 06:37 |
| yoctozepto | so I am just waiting for the refreshed spec | 06:38 |
| yoctozepto | good | 06:38 |
| yoctozepto | as for the consul, I will try to get to it later this week | 06:38 |
| yoctozepto | do we have anything else on release planning? | 06:38 |
| suzhengwei | Priority evcuation and migration back both based on notification event, so we need one more table to record the releation between instance evacautin and notification. | 06:39 |
| suzhengwei | Do we need guide for consul usage? | 06:40 |
| yoctozepto | suzhengwei: don't we have this foreign key already? I guess we would just need an extra column to track if the evacuation was "undone" | 06:42 |
| yoctozepto | suzhengwei: yes, please, let it look better on us than it does with pacemaker; I will help with language | 06:43 |
| suzhengwei | Priority is for both notifications and instances releated to one host notificaion. | 06:45 |
| suzhengwei | What do you means "foreign key"? | 06:46 |
| suzhengwei | Sorry, I can't catch you. | 06:47 |
| yoctozepto | suzhengwei: I meant we already reference the notification for the particular evacuation via the notification uuid (this is the "foreign key", it's a database theory term for that) | 06:47 |
| * yoctozepto is hard to catch :-) | 06:47 | |
| jopdorp | 🤸♂️ | 06:48 |
| suzhengwei | Yes, it used notification_uuid as the foreign key. | 06:49 |
| suzhengwei | jopdorp: Hi, what do you think? | 06:49 |
| jopdorp | I'm not sure | 06:51 |
| yoctozepto | suzhengwei: yes, I know :-) | 06:51 |
| yoctozepto | ok, just waiting for spec update | 06:51 |
| yoctozepto | we should be good | 06:51 |
| yoctozepto | the idea is great | 06:51 |
| yoctozepto | ok, let's move onto open discussion | 06:51 |
| yoctozepto | #topic Open discussion | 06:51 |
| yoctozepto | all questions allowed | 06:52 |
| yoctozepto | ok, I'm wrapping this up then, thanks | 06:59 |
| yoctozepto | #endmeeting | 06:59 |
| opendevmeet | Meeting ended Tue Jul 27 06:59:18 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 06:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/masakari/2021/masakari.2021-07-27-06.03.html | 06:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/masakari/2021/masakari.2021-07-27-06.03.txt | 06:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/masakari/2021/masakari.2021-07-27-06.03.log.html | 06:59 |
| jopdorp | thanks! | 06:59 |
| yoctozepto | suzhengwei: let's merge all backports ~> https://review.opendev.org/c/openstack/masakari-monitors/+/802351 | 07:05 |
| yoctozepto | I will do a release later then | 07:05 |
| suzhengwei | all right. | 07:05 |
| yoctozepto | thanks, | 07:06 |
| yoctozepto | stable releases proposed: https://review.opendev.org/c/openstack/releases/+/802447 | 09:56 |
| *** diablo_rojo__ is now known as diablo_rojo | 11:21 | |
| *** diablo_rojo is now known as Guest2424 | 11:22 | |
| *** Guest2424 is now known as diablo_rojo | 11:23 | |
| *** diablo_rojo is now known as Guest2459 | 17:24 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!