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/!