| *** baojg has quit IRC | 00:09 | |
| *** baojg has joined #openstack-meeting-alt | 00:10 | |
| *** vishakha has quit IRC | 00:30 | |
| *** ayoung has quit IRC | 00:43 | |
| *** ayoung has joined #openstack-meeting-alt | 00:46 | |
| *** rcernin_ has joined #openstack-meeting-alt | 00:46 | |
| *** rcernin has quit IRC | 00:47 | |
| *** masahito has joined #openstack-meeting-alt | 01:04 | |
| *** tmazur has quit IRC | 01:10 | |
| *** ijw__ has quit IRC | 01:20 | |
| *** ijw has joined #openstack-meeting-alt | 01:20 | |
| *** ayoung has quit IRC | 01:29 | |
| *** ayoung has joined #openstack-meeting-alt | 01:30 | |
| *** ayoung has quit IRC | 01:34 | |
| *** baojg has quit IRC | 02:32 | |
| *** baojg has joined #openstack-meeting-alt | 02:33 | |
| *** rcernin_ has quit IRC | 03:19 | |
| *** rcernin_ has joined #openstack-meeting-alt | 03:35 | |
| *** masahito has quit IRC | 03:40 | |
| *** whoami-rajat__ has joined #openstack-meeting-alt | 03:55 | |
| *** apetrich has joined #openstack-meeting-alt | 04:36 | |
| *** masahito has joined #openstack-meeting-alt | 04:38 | |
| *** apetrich has quit IRC | 04:41 | |
| *** vishalmanchanda has joined #openstack-meeting-alt | 04:55 | |
| *** gyee has quit IRC | 05:07 | |
| *** baojg has quit IRC | 05:12 | |
| *** baojg has joined #openstack-meeting-alt | 05:13 | |
| *** kevinz has joined #openstack-meeting-alt | 05:34 | |
| *** slaweq has joined #openstack-meeting-alt | 05:53 | |
| *** lamt has quit IRC | 06:00 | |
| *** jopdorp has quit IRC | 06:01 | |
| *** jopdorp has joined #openstack-meeting-alt | 06:02 | |
| *** baojg has quit IRC | 06:02 | |
| *** rajinir__ has joined #openstack-meeting-alt | 06:02 | |
| *** rajinir has quit IRC | 06:03 | |
| *** rajinir__ is now known as rajinir | 06:03 | |
| *** baojg has joined #openstack-meeting-alt | 06:03 | |
| *** slaweq has quit IRC | 06:18 | |
| *** ralonsoh has joined #openstack-meeting-alt | 06:39 | |
| *** apetrich has joined #openstack-meeting-alt | 06:41 | |
| *** rcernin_ has quit IRC | 06:51 | |
| *** rcernin_ has joined #openstack-meeting-alt | 06:55 | |
| *** slaweq has joined #openstack-meeting-alt | 06:55 | |
| *** jtomasek has joined #openstack-meeting-alt | 07:00 | |
| *** ralonsoh has quit IRC | 07:04 | |
| *** ralonsoh has joined #openstack-meeting-alt | 07:09 | |
| *** e0ne has joined #openstack-meeting-alt | 07:26 | |
| *** rcernin_ has quit IRC | 07:35 | |
| *** tosky has joined #openstack-meeting-alt | 07:38 | |
| *** ralonsoh has quit IRC | 07:43 | |
| *** baojg has quit IRC | 07:53 | |
| *** baojg has joined #openstack-meeting-alt | 07:53 | |
| *** baojg has quit IRC | 07:59 | |
| *** rcernin_ has joined #openstack-meeting-alt | 08:40 | |
| *** rcernin_ has quit IRC | 08:46 | |
| *** derekh has joined #openstack-meeting-alt | 09:05 | |
| *** baojg has joined #openstack-meeting-alt | 09:28 | |
| *** zzzeek has quit IRC | 09:32 | |
| *** zzzeek has joined #openstack-meeting-alt | 09:33 | |
| *** zzzeek has quit IRC | 09:48 | |
| *** zzzeek has joined #openstack-meeting-alt | 09:49 | |
| *** baojg has quit IRC | 09:54 | |
| *** vhari has joined #openstack-meeting-alt | 09:59 | |
| *** zzzeek has quit IRC | 10:08 | |
| *** zzzeek has joined #openstack-meeting-alt | 10:10 | |
| *** ijw has quit IRC | 10:19 | |
| *** ijw has joined #openstack-meeting-alt | 10:20 | |
| *** ijw has quit IRC | 10:25 | |
| *** ijw has joined #openstack-meeting-alt | 10:39 | |
| *** ijw has quit IRC | 10:44 | |
| *** lpetrut has joined #openstack-meeting-alt | 10:47 | |
| *** zzzeek has quit IRC | 10:57 | |
| *** zzzeek has joined #openstack-meeting-alt | 11:00 | |
| *** masahito has quit IRC | 11:00 | |
| *** sridharg has joined #openstack-meeting-alt | 11:03 | |
| *** ijw has joined #openstack-meeting-alt | 11:11 | |
| *** ijw has quit IRC | 11:15 | |
| *** ijw has joined #openstack-meeting-alt | 11:42 | |
| *** ijw has quit IRC | 11:47 | |
| *** raildo has joined #openstack-meeting-alt | 11:56 | |
| *** derekh has quit IRC | 12:09 | |
| *** sridharg has quit IRC | 12:12 | |
| *** ijw has joined #openstack-meeting-alt | 12:14 | |
| *** dave-mccowan has joined #openstack-meeting-alt | 12:19 | |
| *** ijw has quit IRC | 12:19 | |
| *** lbragstad has quit IRC | 12:35 | |
| *** lbragstad has joined #openstack-meeting-alt | 12:38 | |
| *** ijw has joined #openstack-meeting-alt | 12:46 | |
| *** derekh has joined #openstack-meeting-alt | 13:03 | |
| *** tmazur has joined #openstack-meeting-alt | 13:31 | |
| *** enriquetaso has joined #openstack-meeting-alt | 13:47 | |
| *** kaisers has joined #openstack-meeting-alt | 13:50 | |
| *** rosmaita has joined #openstack-meeting-alt | 13:51 | |
| *** whoami-rajat has joined #openstack-meeting-alt | 13:58 | |
| *** Liang__ has joined #openstack-meeting-alt | 13:58 | |
| *** Liang__ is now known as LiangFang | 13:59 | |
| *** ijw has quit IRC | 14:00 | |
| rosmaita | #startmeeting cinder | 14:00 |
|---|---|---|
| openstack | Meeting started Wed Sep 2 14:00:20 2020 UTC and is due to finish in 60 minutes. The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: cinder)" | 14:00 | |
| openstack | The meeting name has been set to 'cinder' | 14:00 |
| rosmaita | #topic roll call | 14:00 |
| *** openstack changes topic to "roll call (Meeting topic: cinder)" | 14:00 | |
| whoami-rajat | Hi | 14:00 |
| e0ne | hi | 14:00 |
| kaisers | hi | 14:00 |
| enriquetaso | ji | 14:00 |
| LiangFang | hi | 14:00 |
| eharney | hi | 14:00 |
| enriquetaso | hi* | 14:00 |
| smcginnis | .o. | 14:00 |
| hemna | mep | 14:00 |
| rosmaita | good turnout | 14:01 |
| rosmaita | #link https://etherpad.openstack.org/p/cinder-victoria-meetings | 14:01 |
| rosmaita | #topic announcements | 14:01 |
| *** openstack changes topic to "announcements (Meeting topic: cinder)" | 14:01 | |
| rosmaita | reminder about Forum brainstorming | 14:02 |
| rosmaita | i sent something to the ML to solicit ideas from operators and other cinder users | 14:02 |
| rosmaita | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-August/016659.html | 14:02 |
| rosmaita | and here is where you can add ideas: | 14:02 |
| rosmaita | #link https://etherpad.opendev.org/p/2020-Wallaby-cinder-brainstorming | 14:02 |
| *** sfernand has joined #openstack-meeting-alt | 14:03 | |
| m5z | hi =] | 14:03 |
| rosmaita | we will pick what topics to propose at next week's meeting | 14:03 |
| rajinir | o/ | 14:03 |
| rosmaita | in the meantime, you need to be registered for the Summit to attend the Forum | 14:03 |
| rosmaita | it's free, so go ahead and register | 14:04 |
| rosmaita | #link https://openinfrasummit2020.eventbrite.com/ | 14:04 |
| rosmaita | also, we have the Wallaby PTG coming up | 14:05 |
| enriquetaso | is the deadline for proposing topics on Wed 9 sept ? | 14:05 |
| rosmaita | enriquetaso: i thought friday that week? | 14:05 |
| enriquetaso | thanks! | 14:05 |
| rosmaita | but if you have an idea, add it to the forum etherpad | 14:06 |
| enriquetaso | ++ | 14:06 |
| rosmaita | and the wallaby PTG etherpad is | 14:06 |
| rosmaita | #link https://etherpad.opendev.org/p/wallaby-ptg-cinder-planning | 14:06 |
| rosmaita | #topic updates - releases | 14:07 |
| *** openstack changes topic to "updates - releases (Meeting topic: cinder)" | 14:07 | |
| rosmaita | ok, os-brick for victoria has to be released tomorrow | 14:07 |
| rosmaita | we have one patch in the gate now | 14:07 |
| rosmaita | #link https://review.opendev.org/730376 | 14:07 |
| rosmaita | previous release was 3.2.1 | 14:08 |
| rosmaita | this will be 4.0.0 because we removed a bunch of connectors for drivers that were removed from cinder | 14:08 |
| e0ne | thanks everybody for the feedback to the patch above | 14:08 |
| rosmaita | we have some stuff piled up for stable/ussuri and train in os-brick | 14:08 |
| rosmaita | so i will put up patches for those releases soon | 14:09 |
| rosmaita | next up: python-cinderclient release next week | 14:09 |
| rosmaita | someone please look at this one! it's a 2 line change! cinderclient support for mv 3.61 (which has merged on cinder side) | 14:09 |
| rosmaita | #link https://review.opendev.org/#/c/742994/ | 14:09 |
| rosmaita | the big thing to focus on is cinder patch for per-project default volume types: | 14:09 |
| rosmaita | #link https://review.opendev.org/#/c/737707/ | 14:10 |
| * e0ne is feeling like a release-blocker person | 14:10 | |
| rosmaita | because that has a cinderclient support patch: | 14:10 |
| rosmaita | #link https://review.opendev.org/739223 | 14:10 |
| hemna | looks like that patch only bumps the microversion number ? | 14:10 |
| rosmaita | hemna: yes | 14:10 |
| e0ne | hemna: yes | 14:10 |
| hemna | ok done | 14:11 |
| rosmaita | whoami-rajat has revised https://review.opendev.org/#/c/742994/ | 14:11 |
| e0ne | smcginnis, hemna: thank you! | 14:11 |
| rosmaita | so it could use some eyes, need to make sure it's acceptable so that we know the cinderclient side is correct | 14:12 |
| whoami-rajat | rosmaita, that's not the patch ... | 14:12 |
| *** LiangFang has quit IRC | 14:12 | |
| rosmaita | oops | 14:12 |
| whoami-rajat | #link https://review.opendev.org/#/c/739223/ | 14:12 |
| rosmaita | #link https://review.opendev.org/#/c/737707/ | 14:12 |
| rosmaita | we need the cinder-side patch merged first, i think | 14:12 |
| rosmaita | so to be completely clear: these are really high priority patches | 14:13 |
| rosmaita | #link https://review.opendev.org/#/c/737707/ | 14:13 |
| whoami-rajat | oh the API one | 14:13 |
| whoami-rajat | ok | 14:13 |
| rosmaita | #link https://review.opendev.org/739223 | 14:13 |
| rosmaita | ok, next priority are other cinder features for victoria | 14:13 |
| rosmaita | use the blueprints: | 14:13 |
| rosmaita | #link https://blueprints.launchpad.net/cinder/victoria | 14:13 |
| rosmaita | but i will point out some really easy reviews (and they have been sitting quite a while ...) | 14:14 |
| rosmaita | support zstd compression: | 14:14 |
| rosmaita | #link https://review.opendev.org/726765 | 14:14 |
| rosmaita | extra-specs support for volume local cache: | 14:14 |
| rosmaita | #link https://review.opendev.org/#/c/700799/ | 14:14 |
| rosmaita | stop sending notifications to deprecated nonstandard publisher_id | 14:14 |
| rosmaita | #link https://review.opendev.org/747540 | 14:14 |
| rosmaita | hemna: would like you to take a look ^^ | 14:14 |
| *** Liang__ has joined #openstack-meeting-alt | 14:14 | |
| hemna | ok | 14:14 |
| rosmaita | remove deprecated rbd option for OSSN-0085 | 14:14 |
| rosmaita | #link https://review.opendev.org/#/c/747494/ | 14:14 |
| rosmaita | and there are the driver features | 14:14 |
| hemna | ah yes cool | 14:15 |
| rosmaita | ok, so to be completely clear: top priority is whoami-rajat's per-project-default-type stuff | 14:15 |
| rosmaita | because we must release cinderclient next week | 14:15 |
| rosmaita | so no possibility of FFE to help there | 14:16 |
| rosmaita | and a reminder to people with driver features | 14:16 |
| rosmaita | reviewing other people's patches will help you get yours reviewed faster | 14:16 |
| rosmaita | (we do pay attention) | 14:16 |
| hemna | lots of failures on the per project default type patch :( | 14:16 |
| *** ijw has joined #openstack-meeting-alt | 14:17 | |
| rosmaita | yeah, the gate has been really unstable | 14:17 |
| rosmaita | any other announcements? | 14:17 |
| rosmaita | (that's all from me) | 14:18 |
| rosmaita | #topic RBD connector fix | 14:18 |
| *** openstack changes topic to "RBD connector fix (Meeting topic: cinder)" | 14:18 | |
| rosmaita | there was some discussion about this today in the cinder channel | 14:18 |
| rosmaita | it may be possible to simplify the patch, but it requires more testing against various ceph versions | 14:19 |
| rosmaita | #link https://review.opendev.org/#/c/730376/ | 14:20 |
| e0ne | I tested only with octopus and nautilus | 14:20 |
| e0ne | the idea is to test against supported versions to check if [global] section works for every supported ceph release | 14:21 |
| rosmaita | so to be clear,the patch is backward compatible right now, and since the issue has only been reported for octopus, we should be OK | 14:21 |
| *** ijw has quit IRC | 14:21 | |
| rosmaita | but it may be possible to remove the conditional logic | 14:22 |
| rosmaita | so for ceph-inclined people, that's something to take a look at | 14:22 |
| rosmaita | e0ne: anything else? | 14:22 |
| e0ne | rosmaita: that's all for me | 14:22 |
| e0ne | I raised this topic only because release deadline | 14:22 |
| rosmaita | and quite appropriately, too | 14:23 |
| rosmaita | #topic service creation race condition | 14:23 |
| *** openstack changes topic to "service creation race condition (Meeting topic: cinder)" | 14:23 | |
| rosmaita | #link https://bugs.launchpad.net/cinder/+bug/1891330 | 14:23 |
| openstack | Launchpad bug 1891330 in Cinder "Duplicate Cinder services DB entries after upgrade" [Low,Triaged] | 14:23 |
| rosmaita | i mostly just want to raise some awareness | 14:23 |
| rosmaita | there is an abandoned patch (link in the bug) | 14:23 |
| rosmaita | but the major objection was that cinder didn't support A/A yet | 14:23 |
| rosmaita | which isn't the case any more | 14:23 |
| e0ne | I'm all for restoring proposed patch to add unique constraints into the DB | 14:24 |
| rosmaita | well, ordinarily, i would be too, but my experience this past week is making me a bit gun-shy | 14:24 |
| rosmaita | as we will discuss in a minute | 14:24 |
| rosmaita | but, it does look like an easy fix | 14:25 |
| rosmaita | #topic ussuri upgrade issue | 14:25 |
| *** openstack changes topic to "ussuri upgrade issue (Meeting topic: cinder)" | 14:25 | |
| rosmaita | this is the issue | 14:26 |
| rosmaita | #link https://bugs.launchpad.net/cinder/+bug/1893107 | 14:26 |
| openstack | Launchpad bug 1893107 in Cinder "Upgrade to Ussuri fails if deleted volumes exist prior to Train" [High,In progress] - Assigned to Mohammed Naser (mnaser) | 14:26 |
| rosmaita | well, i spent a bunch of time trying to figure out a way to fix this that was low-impact on operators | 14:27 |
| rosmaita | and i thought i had a solution | 14:27 |
| rosmaita | but, wound up going back to the original proposal | 14:27 |
| rosmaita | the issue in a nutshell is that the online migrations to set __DEFAULT__ as the volume type for all untyped volumes/snapshots in Train | 14:27 |
| rosmaita | did not take into account the soft-deleted volumes/snapshots | 14:28 |
| rosmaita | so when you try to upgrade to ussuri | 14:28 |
| rosmaita | and the db_sync puts a non-nullability constraint on the columns | 14:28 |
| rosmaita | the db_sync fails | 14:28 |
| rosmaita | this is complicated by the fact that someone discovered a way to delete the __DEFAULT__ volume type in train | 14:28 |
| rosmaita | so, we don't know for sure that it actually exists | 14:29 |
| rosmaita | but, the issue only applies to people who upgraded from Stein to Train without first purging the database | 14:29 |
| rosmaita | (which may actually be a lot of people) | 14:29 |
| rosmaita | anyway, the best thing i could come up with are these patches: | 14:30 |
| rosmaita | #link https://review.opendev.org/#/c/748481/ | 14:30 |
| whoami-rajat | (but if they don't have issue purging it then they can purge anytime and upgrade successfully) | 14:30 |
| rosmaita | #link https://review.opendev.org/#/c/748482/ | 14:30 |
| rosmaita | so, my first ask is for careful reviews of those patches | 14:31 |
| rosmaita | the code change is simple, and there are tests | 14:31 |
| e0ne | whoami-rajat: sometimes operators can't do purge because they use deleted volumes for billing | 14:31 |
| rosmaita | but there is manual operator intervention required when you don't do the purge | 14:31 |
| rosmaita | so PLEASE read the release notes carefully to see if i covered everything | 14:31 |
| whoami-rajat | e0ne, yep, that's why i said if they don't have issue/don't need that info | 14:31 |
| rosmaita | and if you are interested in all the crazy schemes i thought of that didn't work, you can read the sorry story here: | 14:32 |
| hemna | fwiw, we do purging | 14:32 |
| rosmaita | #link https://etherpad.opendev.org/p/ussuri-upgrade-issue | 14:32 |
| rosmaita | yes, so this shows a weakness in grenade testing of T->U upgrades | 14:33 |
| rosmaita | the problem only happens in S->T->U upgrades | 14:33 |
| rosmaita | but i am not going to propose a 3 stage grenade job | 14:34 |
| rosmaita | the reason i had this on the agenda was that i was going to propose using some of the "placeholder" db migrations | 14:35 |
| rosmaita | but it turned out that wasn't going to really help | 14:35 |
| rosmaita | so we are left with really long release notes explaining how to do it yourself | 14:35 |
| rosmaita | that's all from me, unless anyone has questions | 14:36 |
| whoami-rajat | excluding the part that they've renamed the __DEFAULT__ type, they just have to run migrations | 14:36 |
| rosmaita | #topic open discussion | 14:38 |
| *** openstack changes topic to "open discussion (Meeting topic: cinder)" | 14:38 | |
| kaisers | If i may, just a small note: | 14:38 |
| kaisers | Our CI (Quobyte) is one of the failing 3rdparty CIs. | 14:39 |
| kaisers | I've been trying to fix it in the last but that did not work out. | 14:39 |
| kaisers | I'm currently setting up a completely new CI stack | 14:39 |
| kaisers | So i hope i get this back online in the next days | 14:39 |
| kaisers | <done> | 14:39 |
| smcginnis | Thanks kaisers | 14:40 |
| rosmaita | ok, thanks for the update | 14:40 |
| whoami-rajat | i would like some feedback on the idea to backport https://review.opendev.org/#/c/741498/ | 14:40 |
| whoami-rajat | this doesn't change anything on a normal deployment just this allows the operators to delete the __DEFAULT__ type if they've | 14:40 |
| whoami-rajat | set a valid default_volume_type in the conf | 14:40 |
| rosmaita | on the plus side, that would make the behavior consistent from Train through Victoria | 14:41 |
| rosmaita | and it doesn't really impact the upgrade issue we just talked about | 14:41 |
| hemna | that patch makes setting the default_volume_type conf entry manditory | 14:41 |
| hemna | which might bust older deployments if you are backporting this | 14:42 |
| rosmaita | how? | 14:42 |
| whoami-rajat | yep, and it defaults to __DEFAULT__ | 14:42 |
| hemna | if that setting isn't in cinder.conf | 14:42 |
| hemna | what happens? | 14:42 |
| hemna | since it's manditory | 14:42 |
| hemna | cinder won't start? | 14:42 |
| whoami-rajat | ^^ | 14:42 |
| rosmaita | well, if they don't set it, they get __DEFAULT__ | 14:42 |
| rosmaita | if they do set it, they get whatever they set | 14:43 |
| hemna | so then it's not manditory | 14:43 |
| whoami-rajat | hemna, https://review.opendev.org/#/c/741498/16/cinder/common/config.py | 14:43 |
| eharney | i think mandatory here means it isn't set to empty/none ? | 14:43 |
| rosmaita | well, depends on what you mean | 14:43 |
| * hemna has a confusion | 14:43 | |
| rosmaita | it's required but it also has a default value | 14:43 |
| hemna | IMHO manditory means that you must set it in cinder.conf and have a valid value | 14:43 |
| hemna | if it has a default value then it's not manditory to set it | 14:43 |
| e0ne | hemna: +1 | 14:44 |
| hemna | it's confusing | 14:44 |
| whoami-rajat | to preserve the current behavior, we have defaulted it to __DEFAULT__ so there | 14:45 |
| whoami-rajat | 's no effort to set it manually | 14:45 |
| whoami-rajat | but if you're going to set it, it should be a valid type | 14:45 |
| whoami-rajat | also deleting the value set in default_volume_type is not allowed so we atleast have 1 type in deployment and don't allow untyped volumes | 14:46 |
| rosmaita | of all the confusing things in cinder, this is like the least confusing thing i can think of! :) | 14:46 |
| smcginnis | hemna: I had the same reaction. ;) | 14:47 |
| rosmaita | i think what it comes down to is that we want to encourage operators to set that config value | 14:48 |
| *** ijw has joined #openstack-meeting-alt | 14:48 | |
| rosmaita | but, they don't have to | 14:48 |
| rosmaita | you must have a default_volume_type correctly defined at all times, or you can't create volumes | 14:49 |
| rosmaita | whether you let us define it as __DEFAULT__ or whether you do it yourself | 14:49 |
| rosmaita | so that strikes me as "mandatory" in at least some aspect of the term | 14:49 |
| whoami-rajat | all these changes are a part of a bug saying operators/users get *confused* with the name __DEFAULT__ when they've a different default_volume_type set | 14:51 |
| hemna | I guess it's just the wording that makes it confusing is all. | 14:51 |
| smcginnis | The confusion is all based on the fact that we didn't do that right and should have kept __DEFAULT__ a hidden internal thing if someone didn't provide a type. | 14:52 |
| hemna | if it's manditory, then the deployer must set it. but he doesn't because there is a default. hence the confusing. | 14:52 |
| rosmaita | it's not too late to revise the release note to be more clear | 14:52 |
| *** eharney has left #openstack-meeting-alt | 14:52 | |
| rosmaita | https://review.opendev.org/#/c/741498/16/releasenotes/notes/allow-deleting-__DEFAULT__-type-d35dfb5d89760b9b.yaml | 14:53 |
| rosmaita | so, anyone interested in this issue, please read through ^^ | 14:53 |
| *** ijw has quit IRC | 14:53 | |
| rosmaita | we can get that clarified and then decide about the backports, squashing in the revised release note | 14:53 |
| whoami-rajat | the reported bug mentions the train and ussuri releases so that's also a consideration to backport it | 14:56 |
| rosmaita | coming up on 2 minutes left | 14:57 |
| rosmaita | ok, looks like we're done for today! | 14:58 |
| rosmaita | Please ... look at the release notes on the upgrade issue patches -- with Victoria release coming up, i think people will realize they need to upgrade to ussuri | 14:58 |
| rosmaita | https://review.opendev.org/#/c/748481/ | 14:59 |
| rosmaita | https://review.opendev.org/#/c/748482/ | 14:59 |
| rosmaita | ok, thanks everyone! let's clear out so horizon can start on time | 14:59 |
| rosmaita | happy reviewing! | 14:59 |
| rosmaita | #endmeeting | 14:59 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 14:59 | |
| openstack | Meeting ended Wed Sep 2 14:59:43 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-09-02-14.00.html | 14:59 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-09-02-14.00.txt | 14:59 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-09-02-14.00.log.html | 14:59 |
| *** belmoreira has joined #openstack-meeting-alt | 14:59 | |
| whoami-rajat | Thanks! | 15:00 |
| e0ne | #startmeeting horizon | 15:01 |
| openstack | Meeting started Wed Sep 2 15:01:00 2020 UTC and is due to finish in 60 minutes. The chair is e0ne. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| *** openstack changes topic to " (Meeting topic: horizon)" | 15:01 | |
| openstack | The meeting name has been set to 'horizon' | 15:01 |
| e0ne | hi everybody | 15:01 |
| tmazur | hi o/ | 15:01 |
| vishalmanchanda | hi | 15:01 |
| *** rosmaita has left #openstack-meeting-alt | 15:01 | |
| e0ne | ok, let's start | 15:04 |
| e0ne | I hope amotoki and Radomir will join us later | 15:04 |
| e0ne | #topic Notices | 15:04 |
| *** openstack changes topic to "Notices (Meeting topic: horizon)" | 15:04 | |
| tmazur | Radomir is on PTO | 15:04 |
| e0ne | tmazur: thanks for the information | 15:04 |
| e0ne | PTG planing | 15:05 |
| e0ne | please, don't forget to register for PTG if you're going to attend it | 15:05 |
| e0ne | #link https://www.openstack.org/ptg/ | 15:05 |
| e0ne | registration is free and fast enough | 15:06 |
| e0ne | also, there is our usual PTG planning etherpad | 15:06 |
| e0ne | #link https://etherpad.opendev.org/p/horizon-w-ptg | 15:06 |
| e0ne | feel free to add topics you would like to discuss | 15:07 |
| e0ne | you can find our timeslots in the etherpad | 15:07 |
| e0ne | forgot to mention, it will be a virtual PTG again | 15:07 |
| e0ne | #topic Release priorities | 15:11 |
| *** openstack changes topic to "Release priorities (Meeting topic: horizon)" | 15:11 | |
| e0ne | #link https://releases.openstack.org/victoria/schedule.html | 15:11 |
| e0ne | we're one week away from Victoria-3 milestone | 15:12 |
| e0ne | once we'll reach it, no more features will be allowed to merge | 15:12 |
| e0ne | of course, FFE (Feature Freeze Exception) mechanism is still working but I would like to have everything merged before the deadline | 15:13 |
| e0ne | you can find a list of proposed patches with features in our therpad https://etherpad.opendev.org/p/horizon-release-priorities | 15:15 |
| e0ne | s/therpad/etherpad | 15:15 |
| e0ne | feel free to add anything I missed | 15:15 |
| vishalmanchanda | Can I have a 2nd review on https://review.opendev.org/#/c/734161/(cinder-user-facing-messages) and https://review.opendev.org/#/c/708069/(refactor-error-messages) | 15:15 |
| vishalmanchanda | If I missed anything i can cover-up it in a follow-up patch. | 15:16 |
| e0ne | vishalmanchanda: I will review your patches tonight or tomorrow morning | 15:16 |
| vishalmanchanda | e0ne: thanks. | 15:17 |
| e0ne | It will be great to have them merged | 15:17 |
| e0ne | in this release :) | 15:17 |
| vishalmanchanda | yeah | 15:17 |
| e0ne | I hope, it will be a priority for all cores to review new features | 15:19 |
| e0ne | tmazur: your reviews are welcome too :) | 15:19 |
| tmazur | e0ne, as a former/hopefully future core, yep | 15:19 |
| e0ne | tmazur: :) | 15:19 |
| *** ijw has joined #openstack-meeting-alt | 15:19 | |
| e0ne | tmazur: I believe you know about angularized side more than all of us | 15:20 |
| tmazur | Speaking of Angular! I have a question on AngularJS unit tests. Should I wait for the open discussion? | 15:20 |
| e0ne | tmazur: please | 15:21 |
| tmazur | So first of all thanks for the merging this small patch on filters. I ran into an interesting fact while working on it. We do have 22 tests with "SPEC HAS NO EXPECTATIONS" which means that those tests are doing nothing right now. | 15:22 |
| e0ne | tmazur, vishalmanchanda: do you have something to add about our future relese? | 15:23 |
| tmazur | Oops, sorry. Please ignore until the open discussion | 15:23 |
| vishalmanchanda | e0ne: nothing from my side. | 15:23 |
| e0ne | vishalmanchanda: ok | 15:24 |
| tmazur | Nothing from my side about future release | 15:24 |
| *** ijw has quit IRC | 15:24 | |
| e0ne | I've got few more topics to discuss in our etherpad (tempest plugin and Chainese languages), but I would ike to to discuss it with Akihiro and Radomir too | 15:25 |
| e0ne | #topic Open Discussion | 15:25 |
| *** openstack changes topic to "Open Discussion (Meeting topic: horizon)" | 15:25 | |
| e0ne | tmazur: TBH, I don't surprised on it, we'd got some unit/integration tests which were broken from the beginning in the past :( | 15:26 |
| e0ne | tmazur: do you have a list of such tests? | 15:26 |
| tmazur | So, the tests. I've made a quick research and it has to do with unresolved promises. Also they cannot be fixed all in once, each case is pretty unique. | 15:27 |
| amotoki | hi, sorry for late. | 15:27 |
| e0ne | amotoki: welcome! | 15:27 |
| tmazur | I can make the list, it could be easily extracted from jasmine results | 15:28 |
| tmazur | It would be nice to make them work one by one, otherwise it's just confusing. | 15:29 |
| *** Liang__ has quit IRC | 15:29 | |
| e0ne | tmazur: +1 | 15:29 |
| vishalmanchanda | +1. | 15:29 |
| tmazur | I can start to work on it but I'm not sure about my current priorities. So apparently it hasn't to be scheduled to future release. | 15:30 |
| e0ne | tmazur: is any way to catch such tests on CI to prevent such issues in the future? | 15:31 |
| tmazur | e0ne: we should catch these "SPEC HAS NO EXPECTATIONS" warnings. | 15:31 |
| *** tosky has quit IRC | 15:31 | |
| e0ne | can we treat such warnings as errors on CI? | 15:32 |
| amotoki | in my understandng, it means no test actually. | 15:32 |
| e0ne | amotoki: +1 | 15:32 |
| tmazur | theoretically yes. I will tackle this as well | 15:32 |
| e0ne | tmazur: it will be awesome | 15:32 |
| tmazur | That means unresolved promise, once it is resolved, a test fails with an error. | 15:33 |
| tmazur | But in the current way, yes, no test at all | 15:34 |
| amotoki | tmazur: I don't see any "SPEC HAS NO EXPECTATIONS" in our npm test result. how did you get the warning? | 15:36 |
| tmazur | amotoki, I was using jasmine | 15:36 |
| amotoki | I see. so cannot we avoid it with our current test runner? | 15:38 |
| amotoki | (I don't know JS test suite very much) | 15:38 |
| tmazur | amotoki, I don't know yet. My plan is to fix current 22 tests which are doing nothing and then to tackle how we can avoid this situation in future | 15:39 |
| amotoki | tmazur: no problem. your plan sounds good. | 15:40 |
| tmazur | Also, another question. What is the current situation on angularization? I can see there were some efforts on instances page a while ago. I understand it's not a high priority right now, but what is the optimistic plan? | 15:42 |
| e0ne | remove angularjs? | 15:42 |
| tmazur | Essentially: do we still need it or not? | 15:42 |
| e0ne | it's a joke | 15:42 |
| tmazur | Haha, you got me :D | 15:43 |
| e0ne | honestly, I don't know | 15:43 |
| e0ne | from one side it would be good to to complete angularization | 15:43 |
| e0ne | from other side, I don't like a lot to work with old almost unsupported framework | 15:44 |
| tmazur | it's understandable. | 15:44 |
| amotoki | I see one priority which is to close gaps in AngularJS panels used by default | 15:45 |
| amotoki | like Instance Launch form. | 15:45 |
| e0ne | amotoki: that's what I was going to say | 15:45 |
| tmazur | and do not touch it anymore? :) | 15:45 |
| amotoki | for others it needs more discussion. | 15:46 |
| amotoki | Our current status comes from the lack of JS developers. | 15:46 |
| tmazur | I was just looking on ugly filtering system at instances page and was thinking how it would be easier to angularize it, only the filter or the whole page | 15:46 |
| e0ne | let's discuss it during the PTG | 15:48 |
| e0ne | it's important topic | 15:48 |
| tmazur | e0ne, Absolutely! | 15:48 |
| amotoki | agree | 15:48 |
| e0ne | it doesn't mean I don't want to discuss it now or on the next meeting | 15:48 |
| amotoki | one thing I would like to note is that there is no way to call Django form or page from angular pages | 15:48 |
| e0ne | I mean that during virtual face-to-face on the PTG it will be more productive for this particular case | 15:49 |
| amotoki | so angular migration tends to be slow because we cannot switch it until the whole migration is completed. | 15:49 |
| e0ne | amotoki: that's why I prefer to use modern frameworks like react or vue: they don't require to re-implement the whole page to start work | 15:50 |
| amotoki | hehe | 15:50 |
| e0ne | :) | 15:51 |
| *** ijw has joined #openstack-meeting-alt | 15:52 | |
| *** lpetrut has quit IRC | 15:53 | |
| tmazur | ok let's discuss it during the PTG | 15:53 |
| *** belmoreira has quit IRC | 15:53 | |
| e0ne | we've got 5 minutes left | 15:54 |
| e0ne | does anybody have anything to discuss? | 15:54 |
| tmazur | Nothing more from my side | 15:54 |
| amotoki | e0ne: I see you mentioned "tempest plugin and Chainese languages". | 15:54 |
| vishalmanchanda | I am working on this bug https://bugs.launchpad.net/horizon/+bug/1893305 and need some help. | 15:54 |
| openstack | Launchpad bug 1893305 in OpenStack Dashboard (Horizon) "Details href still displayed when details is empty and redirect is false." [Undecided,New] - Assigned to Vishal Manchanda (vishalmanchanda) | 15:54 |
| vishalmanchanda | but we can discuss it in horizon channel. | 15:55 |
| e0ne | amotoki: I moved them to the next meeting | 15:55 |
| amotoki | e0ne: okay. I just wonder what is the topic about tempest plugin. | 15:55 |
| e0ne | amotoki: QA team wants to merge our plugin (actually, only one test) into the tempest core | 15:56 |
| amotoki | e0ne: ah, got it. | 15:56 |
| *** ijw has quit IRC | 15:57 | |
| e0ne | amotoki: that's what they said to me, so I decided to discuss it with the whole horizon team | 15:57 |
| e0ne | we're almost out of time | 15:58 |
| e0ne | vishalmanchanda: we can move to our channel if you need to dicsuss something | 15:59 |
| e0ne | thanks everybody for your contributions! see you next week | 15:59 |
| e0ne | #endmeeting | 15:59 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 15:59 | |
| openstack | Meeting ended Wed Sep 2 15:59:35 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-09-02-15.01.html | 15:59 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-09-02-15.01.txt | 15:59 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-09-02-15.01.log.html | 15:59 |
| vishalmanchanda | bye | 15:59 |
| tmazur | Thanks everyone! | 15:59 |
| amotoki | o/ | 16:00 |
| *** ijw has joined #openstack-meeting-alt | 16:00 | |
| *** ijw has quit IRC | 16:01 | |
| *** ijw has joined #openstack-meeting-alt | 16:01 | |
| *** enriquetaso has quit IRC | 16:40 | |
| *** enriquetaso has joined #openstack-meeting-alt | 16:56 | |
| *** derekh has quit IRC | 17:08 | |
| *** ayoung has joined #openstack-meeting-alt | 17:37 | |
| *** e0ne has quit IRC | 18:29 | |
| *** vishalmanchanda has quit IRC | 18:43 | |
| *** zzzeek has quit IRC | 19:17 | |
| *** zzzeek has joined #openstack-meeting-alt | 19:18 | |
| *** zzzeek has quit IRC | 19:25 | |
| *** zzzeek has joined #openstack-meeting-alt | 19:27 | |
| *** ajitha has joined #openstack-meeting-alt | 19:39 | |
| *** ajitha has left #openstack-meeting-alt | 19:40 | |
| *** e0ne has joined #openstack-meeting-alt | 19:51 | |
| *** zzzeek has quit IRC | 19:53 | |
| *** zzzeek has joined #openstack-meeting-alt | 19:56 | |
| *** apetrich has quit IRC | 20:07 | |
| *** gyee has joined #openstack-meeting-alt | 20:17 | |
| *** ijw has quit IRC | 20:34 | |
| *** ijw has joined #openstack-meeting-alt | 20:35 | |
| *** jtomasek has quit IRC | 20:42 | |
| *** e0ne has quit IRC | 20:44 | |
| *** zzzeek has quit IRC | 20:50 | |
| *** zzzeek has joined #openstack-meeting-alt | 20:51 | |
| *** dustinc has joined #openstack-meeting-alt | 20:58 | |
| *** rcernin_ has joined #openstack-meeting-alt | 21:07 | |
| *** rcernin_ has quit IRC | 21:12 | |
| *** ijw has quit IRC | 21:14 | |
| *** ijw has joined #openstack-meeting-alt | 21:14 | |
| *** rcernin_ has joined #openstack-meeting-alt | 22:16 | |
| *** raildo_ has joined #openstack-meeting-alt | 22:30 | |
| *** raildo has quit IRC | 22:32 | |
| *** enriquetaso has quit IRC | 22:58 | |
| *** tmazur has quit IRC | 23:03 | |
| *** rcernin_ has quit IRC | 23:04 | |
| *** rcernin has joined #openstack-meeting-alt | 23:04 | |
| *** dustinc has quit IRC | 23:08 | |
| *** lifeless has quit IRC | 23:29 | |
| *** irclogbot_3 has quit IRC | 23:29 | |
| *** irclogbot_0 has joined #openstack-meeting-alt | 23:33 | |
| *** whoami-rajat has quit IRC | 23:35 | |
| *** kopecmartin has quit IRC | 23:40 | |
| *** kopecmartin has joined #openstack-meeting-alt | 23:40 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!