| *** boxiang has quit IRC | 00:48 | |
| *** altlogbot_1 has quit IRC | 00:50 | |
| *** altlogbot_1 has joined #openstack-cinder | 00:53 | |
| *** altlogbot_1 has quit IRC | 00:54 | |
| *** altlogbot_3 has joined #openstack-cinder | 00:55 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:04 | |
| *** baojg has quit IRC | 01:08 | |
| openstackgerrit | renminmin proposed openstack/cinder master: Fix rbd mirror retype bug https://review.opendev.org/657985 | 01:23 |
|---|---|---|
| *** baojg has joined #openstack-cinder | 01:29 | |
| *** ircuser-1 has quit IRC | 01:32 | |
| *** brinzhang has joined #openstack-cinder | 01:49 | |
| *** v12aml has quit IRC | 02:10 | |
| *** whoami-rajat has joined #openstack-cinder | 03:05 | |
| *** psachin has joined #openstack-cinder | 03:19 | |
| *** udesale has joined #openstack-cinder | 03:56 | |
| *** david-lyle is now known as dklyle | 03:58 | |
| *** dklyle has quit IRC | 04:03 | |
| *** itlinux has joined #openstack-cinder | 04:10 | |
| *** Luzi has joined #openstack-cinder | 04:42 | |
| *** dpawlik has joined #openstack-cinder | 04:44 | |
| *** e0ne has joined #openstack-cinder | 05:16 | |
| *** boxiang has joined #openstack-cinder | 05:18 | |
| *** lpetrut has joined #openstack-cinder | 05:39 | |
| *** udesale has quit IRC | 05:45 | |
| *** udesale has joined #openstack-cinder | 05:45 | |
| *** udesale has quit IRC | 05:49 | |
| *** udesale has joined #openstack-cinder | 05:49 | |
| *** lpetrut has quit IRC | 05:55 | |
| *** lpetrut has joined #openstack-cinder | 06:06 | |
| *** sapd1_x has joined #openstack-cinder | 06:11 | |
| *** pcaruana has joined #openstack-cinder | 06:16 | |
| *** boxiang has quit IRC | 06:17 | |
| *** boxiang has joined #openstack-cinder | 06:17 | |
| *** lpetrut has quit IRC | 06:22 | |
| *** baojg has quit IRC | 06:28 | |
| *** e0ne has quit IRC | 06:29 | |
| openstackgerrit | Merged openstack/cinder master: move 3par exception https://review.opendev.org/658189 | 06:38 |
| *** udesale has quit IRC | 06:47 | |
| *** udesale has joined #openstack-cinder | 06:47 | |
| *** vishalmanchanda has joined #openstack-cinder | 06:57 | |
| *** rcernin has quit IRC | 07:06 | |
| *** tesseract has joined #openstack-cinder | 07:17 | |
| *** itlinux has quit IRC | 07:35 | |
| *** helenafm has joined #openstack-cinder | 07:40 | |
| *** sahid has joined #openstack-cinder | 07:50 | |
| *** tkajinam has quit IRC | 08:31 | |
| *** e0ne has joined #openstack-cinder | 08:32 | |
| *** lixiaoy1 has quit IRC | 08:54 | |
| *** sapd1_x has quit IRC | 08:57 | |
| *** sapd1_x has joined #openstack-cinder | 09:24 | |
| *** wxy-xiyuan has joined #openstack-cinder | 09:39 | |
| *** dpawlik has quit IRC | 09:41 | |
| *** dpawlik has joined #openstack-cinder | 09:41 | |
| *** kambiz has quit IRC | 09:44 | |
| *** lpetrut has joined #openstack-cinder | 10:12 | |
| *** udesale has quit IRC | 11:17 | |
| *** m75abrams has joined #openstack-cinder | 11:20 | |
| openstackgerrit | renminmin proposed openstack/cinder master: Fix rbd mirror retype bug https://review.opendev.org/657985 | 11:39 |
| *** dviroel has joined #openstack-cinder | 11:39 | |
| openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax driver - Rapid TDEV Deallocation https://review.opendev.org/658793 | 11:42 |
| *** boxiang has quit IRC | 11:51 | |
| *** boxiang has joined #openstack-cinder | 11:51 | |
| *** boxiang has quit IRC | 11:54 | |
| *** boxiang has joined #openstack-cinder | 11:57 | |
| *** carloss has joined #openstack-cinder | 11:58 | |
| *** rosmaita has joined #openstack-cinder | 12:04 | |
| *** lennyb has quit IRC | 12:12 | |
| *** baojg has joined #openstack-cinder | 12:28 | |
| *** psachin has quit IRC | 12:46 | |
| *** baojg has quit IRC | 12:49 | |
| *** pcaruana|afk| has joined #openstack-cinder | 12:55 | |
| *** pcaruana has quit IRC | 12:57 | |
| *** rchurch has joined #openstack-cinder | 12:58 | |
| *** dirk has quit IRC | 12:58 | |
| openstackgerrit | Merged openstack/cinder stable/stein: Make sure we install cinder requirements during the correct tox phase https://review.opendev.org/658297 | 12:58 |
| *** wxy-xiyuan has quit IRC | 12:59 | |
| *** jbernard_ has joined #openstack-cinder | 12:59 | |
| *** rchurch_ has quit IRC | 12:59 | |
| *** xyang_ has joined #openstack-cinder | 12:59 | |
| *** carloss has quit IRC | 12:59 | |
| *** jdillaman has joined #openstack-cinder | 13:00 | |
| *** jbernard has quit IRC | 13:00 | |
| *** xyang has quit IRC | 13:00 | |
| *** xyang_ is now known as xyang | 13:00 | |
| *** dirk has joined #openstack-cinder | 13:00 | |
| *** carloss has joined #openstack-cinder | 13:02 | |
| openstackgerrit | Brin Zhang proposed openstack/cinder master: Handle collections.abc deprecations https://review.opendev.org/658808 | 13:03 |
| *** brinzhang has quit IRC | 13:09 | |
| *** kambiz has joined #openstack-cinder | 13:09 | |
| *** Luzi has quit IRC | 13:10 | |
| openstackgerrit | Merged openstack/cinder master: Add python37 job https://review.opendev.org/655612 | 13:10 |
| *** udesale has joined #openstack-cinder | 13:10 | |
| *** enriquetaso has joined #openstack-cinder | 13:12 | |
| *** wxy-xiyuan has joined #openstack-cinder | 13:14 | |
| *** vishalmanchanda has quit IRC | 13:16 | |
| *** udesale has quit IRC | 13:18 | |
| *** lseki has joined #openstack-cinder | 13:18 | |
| *** udesale has joined #openstack-cinder | 13:18 | |
| *** mchlumsky has joined #openstack-cinder | 13:23 | |
| *** dpawlik has quit IRC | 13:35 | |
| *** gmann has joined #openstack-cinder | 14:06 | |
| *** gmann is now known as gmann_pto | 14:07 | |
| *** gnufied has joined #openstack-cinder | 14:10 | |
| *** baojg has joined #openstack-cinder | 14:11 | |
| *** jbernard_ is now known as jbernard | 14:13 | |
| openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax driver - Rapid TDEV Deallocation https://review.opendev.org/658830 | 14:23 |
| *** mriedem has joined #openstack-cinder | 14:25 | |
| *** baojg has quit IRC | 14:32 | |
| *** abishop has joined #openstack-cinder | 14:50 | |
| *** pcaruana|afk| has quit IRC | 14:55 | |
| *** __hemna has quit IRC | 14:57 | |
| *** ircuser-1 has joined #openstack-cinder | 15:15 | |
| *** itlinux has joined #openstack-cinder | 15:37 | |
| *** pcaruana|afk| has joined #openstack-cinder | 15:44 | |
| *** mvkr has joined #openstack-cinder | 15:46 | |
| *** __hemna has joined #openstack-cinder | 15:57 | |
| *** __hemna has quit IRC | 15:58 | |
| *** hemna has joined #openstack-cinder | 15:59 | |
| hemna | morning | 16:00 |
| *** helenafm has quit IRC | 16:03 | |
| mriedem | so i thought i knew how this worked in cinder but i might have forgotten. if i boot from volume i've got an in-use volume. then if i'm going to migrate the server, nova will create an "empty" volume attachment for the dest host - this is a volume attachment without a connector, and the volume status changes to "attaching", | 16:08 |
| mriedem | now when nova updates the dest host attachment with the connector, it should make the volume as in-use again right? i'm seeing a failure like this instead: "Invalid input received: Invalid volume: Volume b2aba195-6570-40c4-82bb-46d3557fceeb status must be available or downloading to reserve, but the current status is attaching." | 16:08 |
| mriedem | maybe the issue is i haven't deleted the source host attachment yet? | 16:09 |
| smcginnis | mriedem: Looking at blame in the code, doesn't appear we've ever allowed that with the new attach APIs. | 16:12 |
| mriedem | ah yeah i think that's the problem. looking at how nova does a migration, we create the empty dest host attachment first which reserves the volume, then we delete the source host attachment, and once we migrate to the dest host, we update the dest host attachment with the connector and finally complete it to mark the volume as in-use | 16:12 |
| smcginnis | We sepcifically only check for available or downloading and also allow in-use only if multiattach is set. | 16:12 |
| mriedem | yeah...shazbot, my cross-cell stuff has hit a snag for volume-backed servers then... | 16:13 |
| mriedem | i'm trying to sniff out if a dest host is going to work in another cell by creating an export there first (updating the attachment using the dest host connector), but i haven't deleted the existing source host attachment for the volume, which is different from how normal resize works | 16:14 |
| smcginnis | Yeah, that could be tricky. | 16:14 |
| smcginnis | We could add "attaching", but I think we would need to enforce that the update is for the same host that started the attachment or something to prevent a race of someone else coming in and attaching it. | 16:15 |
| mriedem | i'd have to (1) create the dest host empty attachment, (2) delete the source host attachment, (3) update the dest host attachment with the dest host connector, and failing that, re-connect the source host attachment | 16:15 |
| smcginnis | Which I think defeats the purpose of having that "reserved" state. | 16:15 |
| smcginnis | mriedem: That sounds reasonable. | 16:15 |
| mriedem | reasonable, but pretty sucky | 16:16 |
| mriedem | i.e. lots of things to go wrong there | 16:16 |
| mriedem | but without creating the export on the dest host we don't really have a good pre-check that that host will work | 16:17 |
| mriedem | right? | 16:17 |
| smcginnis | I think you are right. | 16:18 |
| smcginnis | We would have to add a new API call to validate that beforehand. | 16:18 |
| *** tesseract has quit IRC | 16:20 | |
| *** whoami-rajat has quit IRC | 16:31 | |
| *** whoami-rajat has joined #openstack-cinder | 16:36 | |
| *** sahid has quit IRC | 16:37 | |
| *** e0ne has quit IRC | 16:42 | |
| *** tejdeep has joined #openstack-cinder | 16:51 | |
| *** tejdeep has quit IRC | 16:53 | |
| *** imacdonn has joined #openstack-cinder | 16:57 | |
| *** udesale has quit IRC | 16:59 | |
| *** tejdeep has joined #openstack-cinder | 17:07 | |
| *** mchlumsky has quit IRC | 17:23 | |
| openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Bump bandit version https://review.opendev.org/658873 | 17:26 |
| *** lpetrut has quit IRC | 17:27 | |
| *** mchlumsky has joined #openstack-cinder | 17:29 | |
| *** jroll has quit IRC | 17:34 | |
| *** jroll has joined #openstack-cinder | 17:35 | |
| *** itlinux has quit IRC | 17:50 | |
| *** itlinux has joined #openstack-cinder | 17:53 | |
| *** sapd1_x has quit IRC | 17:56 | |
| *** dklyle has joined #openstack-cinder | 17:59 | |
| openstackgerrit | John Cates proposed openstack/cinder master: Change _unmap_vdisk_from_host to unmap all connected hosts if connector is unspecified https://review.opendev.org/658884 | 18:07 |
| *** dklyle has quit IRC | 18:08 | |
| *** enriquetaso has quit IRC | 18:09 | |
| *** enriquetaso has joined #openstack-cinder | 18:11 | |
| *** e0ne has joined #openstack-cinder | 18:17 | |
| *** tejdeep has quit IRC | 18:17 | |
| *** dklyle has joined #openstack-cinder | 18:23 | |
| *** dklyle has quit IRC | 18:34 | |
| *** tejdeep has joined #openstack-cinder | 18:39 | |
| *** e0ne has quit IRC | 18:54 | |
| *** dpawlik has joined #openstack-cinder | 19:03 | |
| *** m75abrams has quit IRC | 19:15 | |
| *** gmann_pto has quit IRC | 19:33 | |
| *** e0ne has joined #openstack-cinder | 20:14 | |
| openstackgerrit | Dirk Mueller proposed openstack/cinder stable/rocky: Make sure we install cinder requirements during the correct tox phase https://review.opendev.org/658913 | 20:21 |
| *** e0ne has quit IRC | 20:26 | |
| hemna | simondodsley: https://review.opendev.org/#/c/658096/ | 20:30 |
| *** itlinux has quit IRC | 20:32 | |
| openstackgerrit | Dirk Mueller proposed openstack/cinder master: Use testenv deps instead of repeating the dependencies https://review.opendev.org/658914 | 20:33 |
| openstackgerrit | Dirk Mueller proposed openstack/cinder stable/queens: Make sure we install cinder requirements during the correct tox phase https://review.opendev.org/658915 | 20:34 |
| *** enriquetaso has quit IRC | 20:35 | |
| openstackgerrit | Dirk Mueller proposed openstack/cinder stable/pike: Make sure we install cinder requirements during the correct tox phase https://review.opendev.org/658916 | 20:35 |
| *** dpawlik has quit IRC | 20:35 | |
| dirk | smcginnis: looks like there were more missing deps in tox.ini (see https://review.opendev.org/658914) | 20:35 |
| smcginnis | dirk: Was just looking at that. I don't think so, I think this just adds more to some targets that don't need them. | 20:38 |
| dirk | smcginnis: reason for the constraint arg being injected (incorrectly imho) intot he deps option is the lower-constraints stuff | 20:39 |
| dirk | I was just trying to find a setenv or similar trick for this | 20:39 |
| dirk | smcginnis: any idea why -U is missing in the install command? that looks pretty broken | 20:40 |
| smcginnis | dirk: Why does that affect l-c? | 20:40 |
| smcginnis | -U is likely another oversight. | 20:40 |
| *** pcaruana|afk| has quit IRC | 20:46 | |
| dirk | smcginnis: is your comment a -1? e.g. the releasenotes target is already installing all cinder deps (instead of just pulling in reno..) | 20:47 |
| smcginnis | dirk: I think that could get cleaned up, and api-ref and bandit could probably be left alone if we just move the constraints arg into the install_command like most other projects have it. | 20:48 |
| smcginnis | That seems "more correct". | 20:48 |
| dirk | smcginnis: imho reno and linters should not be part of test-requirements (test-requirements should imho ~ match what is needed for unit/functional testing) | 20:48 |
| smcginnis | I'm assuming they were probably added to test-requirements because they are not needed to actually run the code. | 20:49 |
| dirk | right, but they're also not needed for tests | 20:51 |
| *** openstackgerrit has quit IRC | 21:09 | |
| *** dklyle has joined #openstack-cinder | 21:13 | |
| _alastor_ | jungleboyj: Another day, another weird set of error from bare-metal tempest | 21:19 |
| _alastor_ | jungleboyj: May 13 14:14:15 tlx3cp cinder-volume[29059]: ERROR oslo_messaging.rpc.server ImageNotAuthorized: Not authorized for image e1e6b8f0-416c-4263-8a81-cbd1f84efba4. | 21:19 |
| _alastor_ | jungleboyj: I haven't changed the devstack setup, just the box where stack.sh was run | 21:20 |
| _alastor_ | jungleboyj: It's like devstack decided this time not to give Cinder any Glance permissions at all | 21:21 |
| *** dklyle has quit IRC | 21:24 | |
| *** mchlumsky has quit IRC | 21:38 | |
| *** whoami-rajat has quit IRC | 21:41 | |
| *** openstackgerrit has joined #openstack-cinder | 21:46 | |
| openstackgerrit | Dirk Mueller proposed openstack/cinder master: Use testenv deps instead of repeating the dependencies https://review.opendev.org/658914 | 21:46 |
| *** mriedem is now known as mriedem_afk | 21:53 | |
| *** lixiaoy1 has joined #openstack-cinder | 22:48 | |
| *** tkajinam has joined #openstack-cinder | 22:56 | |
| *** rcernin has joined #openstack-cinder | 23:04 | |
| *** rcernin has quit IRC | 23:07 | |
| *** rcernin has joined #openstack-cinder | 23:07 | |
| *** carloss has quit IRC | 23:17 | |
| *** dklyle has joined #openstack-cinder | 23:18 | |
| *** mriedem_afk has quit IRC | 23:31 | |
| *** hemna has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!