| *** dviroel|afk is now known as dviroel|out | 00:46 | |
| opendevreview | DataCore SANsymphony CI proposed openstack/cinder master: Add Datacore Software volume driver https://review.opendev.org/c/openstack/cinder/+/831154 | 05:06 |
|---|---|---|
| opendevreview | DataCore SANsymphony CI proposed openstack/cinder master: Fix Datacore driver NameError https://review.opendev.org/c/openstack/cinder/+/831167 | 05:06 |
| opendevreview | DataCore SANsymphony CI proposed openstack/cinder master: Fix pep8/flake8 violations in Datacore volume driver https://review.opendev.org/c/openstack/cinder/+/832750 | 05:06 |
| opendevreview | DataCore SANsymphony CI proposed openstack/cinder master: Add unit test and documentation for DataCore volume driver https://review.opendev.org/c/openstack/cinder/+/832751 | 05:06 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/791921 | 08:01 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-tempest-plugin master: Add multiattach tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/791921 | 09:33 |
| *** brinzhang_ is now known as brinzhang | 10:39 | |
| stephenfin | eharney: If so, we can probably just bump it even higher. Nova has OS_TEST_TIMEOUT=160 (seconds), so with TIMEOUT_SCALING_FACTOR=4 that's over 10 minutes. I could just set TIMEOUT_SCALING_FACTOR=10 and do the same | 11:16 |
| stephenfin | eharney: That doesn't answer the question of why it's so damn slow for MySQL, but given the issue occurs in nova too, I suspect the answer is either MySQL is slow for this stuff or alembic is | 11:17 |
| stephenfin | (neither of which we can really fix) | 11:17 |
| *** dviroel|out is now known as dviroel | 11:19 | |
| yuval | reviewed by me:https://review.opendev.org/c/openstack/cinder/+/832750, https://review.opendev.org/c/openstack/cinder/+/830086 | 13:50 |
| yuval | cores can you please review: https://review.opendev.org/c/openstack/cinder/+/830538 | 13:50 |
| rosmaita | yuval: thanks for doing reviews! | 13:54 |
| abishop | rosmaita: do you know when cinder is scheduled to cut its first yoga release? | 13:54 |
| rosmaita | abishop: tomorrow | 13:54 |
| abishop | well that worked out nicely :D | 13:54 |
| rosmaita | depends on what you mean by 'nicely' :D | 13:55 |
| abishop | an answer that I like... I'm holding a tripleo patch related to cinder endpoints until we have a release that includes the code | 13:56 |
| abishop | the code that removes the need for project_id | 13:56 |
| rosmaita | ah, ok | 13:56 |
| abishop | didn't know how long I needed to continue to hold the patch | 13:56 |
| rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:57 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^ | 13:57 |
| whoami-rajat | thanks! | 13:57 |
| jungleboyj | o/ | 13:57 |
| geguileo | rosmaita: thanks | 13:58 |
| rosmaita | it's the least i can do! | 13:58 |
| enriquetaso | thanks! | 13:58 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 14:02 | |
| *** hemna7 is now known as hemna | 14:28 | |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Pure: Fix replication in A/A https://review.opendev.org/c/openstack/cinder/+/829664 | 14:38 |
| geguileo | simondodsley: ^ | 14:38 |
| *** hemna5 is now known as hemna | 14:47 | |
| dc_arunkv | core, I made a mistake by committing some logic changes as part of pep8 fix | 14:53 |
| dc_arunkv | (https://review.opendev.org/c/openstack/cinder/+/832750) | 14:53 |
| dc_arunkv | . Is it ok if I fix the commit message to include those details or should I revert those logic changes. Thanks. | 14:53 |
| rosmaita | dc_arunkv: it would be better to split them up | 14:59 |
| opendevreview | Merged openstack/cinder master: db: Increase timeout for migration tests https://review.opendev.org/c/openstack/cinder/+/832696 | 15:00 |
| enriquetaso | #startmeeting cinder_bs | 15:01 |
| opendevmeet | Meeting started Wed Mar 9 15:01:21 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:01 |
| enriquetaso | Welcome to Cinder Bug Meeting | 15:01 |
| enriquetaso | Cinder has 2 bugs reported this week. | 15:01 |
| enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027625.html | 15:01 |
| enriquetaso | #topic [powerstore driver]: no volume caching after 35 volumes. | 15:02 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1962824 | 15:02 |
| enriquetaso | The driver needs to add a "SnapshotLimitReached" to invalidate cache on the limit. Looks valid and needs a patch. | 15:02 |
| enriquetaso | Moving on. | 15:03 |
| enriquetaso | #topic Don't require control-location for creating encrypted volume types. | 15:03 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1964145 | 15:03 |
| enriquetaso | Summary: We shouldn't require "control-location" on encrypted type creation -- just default to "front-end" if one is not supplied. | 15:03 |
| enriquetaso | The full discussion and work is going on the patch. | 15:03 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/677862 | 15:03 |
| eharney | this doesn't have a patch proposed for it | 15:03 |
| eharney | but, basically, we require specifying control-location when creating an encrypted type, even though everyone always wants "front-end", so we should do that by default | 15:04 |
| eharney | (and probably drop support for back-end IMO but that's a whole additional project) | 15:04 |
| enriquetaso | OK, I've misunderstood the bp | 15:05 |
| enriquetaso | thanks eharney for clarifying | 15:06 |
| dc_arunkv | Thanks rosmaita, I will do it. | 15:07 |
| rosmaita | that's a widening change, so won't require a new microversion | 15:07 |
| rosmaita | dc_arunkv: ty | 15:07 |
| rosmaita | the "widening change" was about the control-location | 15:07 |
| enriquetaso | if we would like to drop support for back-end, should we create a new bp or use that one eharney ? | 15:07 |
| eharney | that bug is just about setting a default control-location behavior | 15:08 |
| eharney | i don't know if we need a bp | 15:08 |
| enriquetaso | sorry, not a bp, a new bug.. | 15:08 |
| eharney | i'm not sure if any drivers attempt to implement backend encryption any more, at one point i think one did | 15:08 |
| eharney | i'll have to look around at that | 15:09 |
| enriquetaso | #action: drop support for back-end ? | 15:09 |
| enriquetaso | thanks | 15:09 |
| enriquetaso | Moving on.. | 15:10 |
| enriquetaso | #topic Open Discussion | 15:10 |
| rosmaita | i can't believe i reviewed that patch 2 years ago | 15:10 |
| enriquetaso | OK | 15:11 |
| enriquetaso | thanks for attending! | 15:12 |
| enriquetaso | oh, i'll be on PTO, so no meeting next week! | 15:13 |
| rosmaita | ok, enjoy your PTO | 15:13 |
| enriquetaso | #endmeeting | 15:14 |
| opendevmeet | Meeting ended Wed Mar 9 15:14:03 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:14 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-03-09-15.01.html | 15:14 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-03-09-15.01.txt | 15:14 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-03-09-15.01.log.html | 15:14 |
| *** dviroel is now known as dviroel|lunch | 15:14 | |
| enriquetaso | thanks | 15:14 |
| opendevreview | Merged openstack/cinder master: fix 'huawei dorado v6' FC initiator can be added to host https://review.opendev.org/c/openstack/cinder/+/819778 | 16:18 |
| opendevreview | Merged openstack/cinder master: RBD driver: clean up _trash_purge exception handling https://review.opendev.org/c/openstack/cinder/+/818830 | 16:18 |
| opendevreview | Merged openstack/cinder master: mypy: RBD driver https://review.opendev.org/c/openstack/cinder/+/774288 | 16:18 |
| *** dviroel|lunch is now known as dviroel | 16:42 | |
| opendevreview | Antoine Eiche proposed openstack/python-cinderclient stable/ussuri: Add user and project domain name parameters in HTTPClient https://review.opendev.org/c/openstack/python-cinderclient/+/832830 | 16:43 |
| jamesbenson | Morning all, I'm failing on cinder refstack tests with a `tempest.lib.exceptions.UnexpectedResponseCode: Unexpected response code received` on about 12 cinder tests (and about 9 network tests). They all have the same error. Any idea's? | 16:56 |
| ManishLaddha_ | Hello everyone, can you please review a minor document change: https://review.opendev.org/c/openstack/cinder/+/832499, this change is planned for Yoga from SVf driver | 17:01 |
| rosmaita | jamesbenson: do you have a link to any logs? or at least a list of the failing tests | 17:05 |
| jamesbenson | let me know if this works: https://storage.googleapis.com/gitlab-gprd-artifacts/40/df/40df4f49880eb3f6a36fa30ca9946e13b6aa912d26c9d4e98afeb5cbce8fa58b/2022_03_09/2180025910/2374161924/job.log?response-content-type=text%2Fplain%3B%20charset%3Dutf-8&response-content-disposition=inline&GoogleAccessId=gitlab-object-storage-prd@gitlab-production.iam.gserviceaccount.com&Signature=FnqSSDx9s1DbdCsYK62juudj0N | 17:09 |
| jamesbenson | YWfApqAW3HN%2BiFpnzf8BgFG7SLB%2BhKzIun%0A2nnmeXCHp%2BipTxyy%2FMR0olua%2B%2FcODT3NQ%2BG1QwPKRAh7wp89QFgvGqv0TozK%0AXXIGLNrx3m%2B0lAc97FyrnJP9Id%2Fb%2BY%2FhcHJXi5%2FyaWAnjSOStYus5359jjqw%0AFPlJWWYX%2Fran37CjcZzG0c5L6XnJYFI44Qy1tDEUO292%2FvBFn9aEBX5g3jZk%0AE2XKURsUzeyI1YGVFxOcUardvK9hm87gzlv6hKsbGLPX0F2U1k5rOUpJyUjs%0AuGVtg0lXTul56Vmjy0MLmDrV1FhoDJrgI0OrqeSAsA%3D%3D&Expires=1646846345 | 17:09 |
| jamesbenson | sorry for the multiline link there... gitlab is where we do our testing | 17:09 |
| rosmaita | ok, link works | 17:11 |
| jamesbenson | FWIW: tempest.api.compute.servers.test_multiple_create.MultipleCreateTestJSON always fails when run with other tests for us, so that one can be ignored. | 17:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate remotefs https://review.opendev.org/c/openstack/cinder/+/733623 | 17:14 |
| rosmaita | jamesbenson: at a quick glance, looks like the unexpected code you are getting is a 504 (Gateway Timeout) ... most of the failures are during setup and teardown, so i suspect some kind of networking problem | 17:19 |
| jamesbenson | @rosmaita: hmm, okay. These hosts are all virsh VM's on the same host, so I feel like it shouldn't be network. But I definitely don't claim to be a virsh expert. Not sure if you know more about that. Could it be something with TLS not working properly? We have TLS enabled on internal/external/backend. | 17:24 |
| jamesbenson | rosmaita: We have been running these tests for a while and only relatively recently have we been having these errors. | 17:24 |
| rosmaita | jamesbenson: you're going to have to dig into the logs a bit to see where the timeout is happening | 17:27 |
| jamesbenson | alrighty, thanks :-) | 17:29 |
| rosmaita | not much help, i know, but good luck! | 17:29 |
| jamesbenson | One other piece of info. | 17:32 |
| jamesbenson | Those failures are on a 3-node Xena deploy. We do nightly tests also on an all-in-one Wallaby, those seem to always pass. | 17:32 |
| gmann | whoami-rajat: rosmaita can you please check this patch, adding irrelevant-files for newly added grenade-skip-level job https://review.opendev.org/c/openstack/cinder/+/831438 | 17:41 |
| whoami-rajat | looking | 17:42 |
| whoami-rajat | gmann, I'm unable to find much info regarding what this job does (even in the dependent patch) | 17:43 |
| gmann | whoami-rajat: ah sorry, this is patch added the job definition and depends-on added to run for cinder gate https://review.opendev.org/c/openstack/grenade/+/826101 | 17:44 |
| gmann | so it already run on cinder gate as non voting and my patch 831438 just add the irrelevant-files so that it would not run on doc/test only changes | 17:45 |
| whoami-rajat | ack, so it is testing kind of a fast forward upgrade skipping one release in between | 17:46 |
| gmann | yeah but not exactly the FFU - https://governance.openstack.org/tc/resolutions/20220210-release-cadence-adjustment.html#details | 18:00 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: set no_implicit_optional https://review.opendev.org/c/openstack/cinder/+/782260 | 18:44 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: quota.py https://review.opendev.org/c/openstack/cinder/+/788366 | 19:08 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: ceph backup driver https://review.opendev.org/c/openstack/cinder/+/788368 | 19:33 |
| opendevreview | Merged openstack/cinder master: db: Preserve API signatures https://review.opendev.org/c/openstack/cinder/+/830099 | 20:02 |
| *** hemna6 is now known as hemna | 20:29 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Use functools with require_context wrappers https://review.opendev.org/c/openstack/cinder/+/832854 | 20:35 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Use functools with require_context wrappers https://review.opendev.org/c/openstack/cinder/+/832854 | 20:38 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Use functools with require_context wrappers https://review.opendev.org/c/openstack/cinder/+/832854 | 20:45 |
| rosmaita | yes, somebody forgot to run pep8 | 20:45 |
| eharney | rosmaita: put some pep in your step(s) | 20:46 |
| rosmaita | :P | 20:46 |
| *** dviroel is now known as dviroel|out | 21:29 | |
| opendevreview | Merged openstack/cinder master: Hitachi: Add Target Port Assignment for VSP Driver https://review.opendev.org/c/openstack/cinder/+/828060 | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!