| *** openstack has joined #openstack-cinder | 09:56 | |
| *** ChanServ sets mode: +o openstack | 09:56 | |
| *** xinliang has quit IRC | 10:03 | |
| *** andrebeltrami has joined #openstack-cinder | 11:24 | |
| *** sapd1 has quit IRC | 11:30 | |
| *** udesale_ has joined #openstack-cinder | 11:34 | |
| *** udesale has quit IRC | 11:36 | |
| *** ociuhandu has quit IRC | 11:41 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder stable/rocky: PowerMax Driver - Legacy volume not found https://review.opendev.org/715007 | 11:52 |
|---|---|---|
| *** vishalmanchanda has quit IRC | 12:11 | |
| *** tkajinam has quit IRC | 12:12 | |
| *** ociuhandu has joined #openstack-cinder | 12:19 | |
| *** sapd1 has joined #openstack-cinder | 12:33 | |
| *** ociuhandu has quit IRC | 12:34 | |
| *** ociuhandu has joined #openstack-cinder | 12:35 | |
| *** lpetrut has quit IRC | 12:54 | |
| *** lpetrut has joined #openstack-cinder | 13:01 | |
| *** e0ne_ has joined #openstack-cinder | 13:04 | |
| *** e0ne has quit IRC | 13:05 | |
| *** vishalmanchanda has joined #openstack-cinder | 13:15 | |
| rosmaita | smcginnis: did you not +A https://review.opendev.org/#/c/656271/14 on purpose, or did we hit a race condition there? | 13:20 |
| smcginnis | rosmaita: Race. :) | 13:20 |
| rosmaita | ok, just checking | 13:21 |
| smcginnis | I just hit the big blue button. | 13:21 |
| rosmaita | cool, ty | 13:21 |
| smcginnis | We still have e0ne_'s patch: https://review.opendev.org/656888 | 13:21 |
| smcginnis | If we're going to do that in ussuri, I think we should do it very soon. | 13:22 |
| smcginnis | Otherwise explicitly hold it until v. | 13:22 |
| * e0ne_ would like to have this one https://review.opendev.org/#/c/713649/ for Ussuri too | 13:22 | |
| smcginnis | Looking. | 13:22 |
| e0ne_ | smcginnis: thank you, Sean! | 13:24 |
| e0ne_ | geguileo: could you please take a look on this ^^ patch once more? | 13:28 |
| *** pcaruana has quit IRC | 13:28 | |
| *** tkajinam has joined #openstack-cinder | 13:33 | |
| geguileo | e0ne_: mmm, there are no unit tests... | 13:36 |
| e0ne_ | geguileo: oops | 13:36 |
| smcginnis | Good point. | 13:37 |
| geguileo | e0ne_: I was going to merge it when I saw it... :-( | 13:37 |
| smcginnis | e0ne_: Want to update this patch? Or follow up with a patch later to add test coverage? | 13:38 |
| smcginnis | Would probably be good to include them here. | 13:38 |
| e0ne_ | geguileo, smcginnis: I'll add tests once I have a time to implement them. I can't say that it will be today or tomorrow | 13:38 |
| *** bcm has quit IRC | 13:50 | |
| *** simondodsley has quit IRC | 13:50 | |
| *** ykarel is now known as ykarel|away | 13:50 | |
| *** walshh_ has quit IRC | 13:50 | |
| *** walshh_ has joined #openstack-cinder | 13:50 | |
| *** bcm has joined #openstack-cinder | 13:51 | |
| *** simondodsley has joined #openstack-cinder | 13:51 | |
| *** ociuhandu has quit IRC | 14:03 | |
| *** ociuhandu has joined #openstack-cinder | 14:04 | |
| openstackgerrit | Xuchu Jiang proposed openstack/cinder master: [DS8000] Update cinder driver configuration guide https://review.opendev.org/719743 | 14:07 |
| Xuchu | e0ne_: available for a talk about https://review.opendev.org/#/c/698630/ ? | 14:16 |
| *** pcaruana has joined #openstack-cinder | 14:16 | |
| *** tkajinam has quit IRC | 14:17 | |
| *** tkajinam has joined #openstack-cinder | 14:18 | |
| *** udesale_ has quit IRC | 14:20 | |
| *** sapd1 has quit IRC | 14:24 | |
| *** lpetrut has quit IRC | 14:24 | |
| *** dosaboy_ is now known as dosaboy | 15:04 | |
| *** ociuhandu has quit IRC | 15:19 | |
| *** ociuhandu has joined #openstack-cinder | 15:21 | |
| *** ociuhandu has quit IRC | 15:25 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Handle py38 unit test changes https://review.opendev.org/720008 | 15:31 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Drop py2 collections.abc compat code https://review.opendev.org/720017 | 15:40 |
| *** raunak12 has joined #openstack-cinder | 15:41 | |
| openstackgerrit | Merged openstack/cinder master: [Unity] Retype volume support https://review.opendev.org/656271 | 15:49 |
| *** ociuhandu has joined #openstack-cinder | 15:56 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Add Python3 victoria unit tests https://review.opendev.org/719081 | 16:11 |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Update master for stable/ussuri https://review.opendev.org/718562 | 16:11 |
| *** enriquetaso has joined #openstack-cinder | 16:17 | |
| *** tkajinam has quit IRC | 16:20 | |
| *** yaawang_ has joined #openstack-cinder | 16:32 | |
| *** yaawang has quit IRC | 16:33 | |
| *** psachin has quit IRC | 16:34 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Update upgrade check removed drivers for ussuri https://review.opendev.org/720031 | 16:56 |
| *** vishalmanchanda has quit IRC | 17:04 | |
| jungleboyj | rosmaita: They just sent notes on what the Virtual PTG is going to look like on the mailing list. Take a look. Might be worth discussing at our weekly meeting tomorrow so we can be on top of getting our requests in place. | 17:13 |
| *** e0ne_ has quit IRC | 17:29 | |
| *** e0ne has joined #openstack-cinder | 17:32 | |
| *** ociuhandu has quit IRC | 17:40 | |
| *** ociuhandu has joined #openstack-cinder | 17:41 | |
| *** e0ne has quit IRC | 17:42 | |
| *** ociuhandu has quit IRC | 17:46 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Prevent creation of ssh_known_hosts file in UT run https://review.opendev.org/720043 | 17:52 |
| *** dpawlik has quit IRC | 18:02 | |
| rosmaita | jungleboyj: ack | 18:04 |
| openstackgerrit | Rafael Weingärtner proposed openstack/cinder master: Add mv to return volume type name and ID with volumes https://review.opendev.org/666886 | 18:11 |
| *** ociuhandu has joined #openstack-cinder | 18:14 | |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: WIP: Retype test added https://review.opendev.org/715566 | 18:16 |
| *** martinkennelly has quit IRC | 18:29 | |
| openstackgerrit | Merged openstack/cinder master: Add test coverage for manage_existing API revert https://review.opendev.org/691527 | 18:41 |
| *** ociuhandu has quit IRC | 18:50 | |
| openstackgerrit | Brian Rosmaita proposed openstack/cinder master: Update upgrade check removed drivers for ussuri https://review.opendev.org/720031 | 18:50 |
| *** ociuhandu has joined #openstack-cinder | 18:51 | |
| *** e0ne has joined #openstack-cinder | 18:52 | |
| *** ociuhandu has quit IRC | 18:55 | |
| *** e0ne has quit IRC | 19:02 | |
| *** Lucas_Gray has joined #openstack-cinder | 19:11 | |
| *** ociuhandu has joined #openstack-cinder | 19:16 | |
| *** Lucas_Gray has quit IRC | 19:21 | |
| *** Lucas_Gray has joined #openstack-cinder | 19:22 | |
| *** e0ne has joined #openstack-cinder | 19:22 | |
| walshh_ | Hi all, is anyone available to look at https://review.opendev.org/#/c/718475/. It has clean CI, thank you | 19:27 |
| *** e0ne has quit IRC | 19:40 | |
| *** Lucas_Gray has quit IRC | 19:41 | |
| *** Lucas_Gray has joined #openstack-cinder | 19:45 | |
| *** tobberydberg_ has quit IRC | 20:17 | |
| *** tobberydberg has joined #openstack-cinder | 20:22 | |
| *** tobberydberg has quit IRC | 20:30 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Add Python3 victoria unit tests https://review.opendev.org/719080 | 20:36 |
| *** tobberydberg has joined #openstack-cinder | 20:36 | |
| *** tobberydberg has quit IRC | 20:37 | |
| *** tobberydberg has joined #openstack-cinder | 20:42 | |
| *** tobberydberg has quit IRC | 20:43 | |
| *** Lucas_Gray has quit IRC | 20:44 | |
| hemna_ | Is there no way to rename a volume type ? | 20:45 |
| hemna_ | other than hacking the db | 20:45 |
| *** tobberydberg has joined #openstack-cinder | 20:45 | |
| *** tobberydberg has quit IRC | 20:45 | |
| *** ociuhandu has quit IRC | 20:45 | |
| *** tobberydberg has joined #openstack-cinder | 20:46 | |
| *** tobberydberg has quit IRC | 20:46 | |
| *** ociuhandu has joined #openstack-cinder | 20:46 | |
| *** tobberydberg has joined #openstack-cinder | 20:46 | |
| *** tobberydberg has quit IRC | 20:47 | |
| hemna_ | looks like the openstack client can do it | 20:47 |
| *** tobberydberg has joined #openstack-cinder | 20:47 | |
| *** tobberydberg has quit IRC | 20:47 | |
| *** tobberydberg has joined #openstack-cinder | 20:51 | |
| *** ociuhandu has quit IRC | 20:52 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Reduce deprecation warnings https://review.opendev.org/720084 | 20:53 |
| smcginnis | hemna_: Our api-ref only mentions updating extra specs. | 20:54 |
| *** tobberydberg has quit IRC | 20:55 | |
| smcginnis | hemna_: Theoretically, you should be able to create a duplicate one with a new name and retype, but that's not ideal. | 20:55 |
| hemna_ | openstack volume type set --name my_new_name <existing volume type uuid> | 20:55 |
| hemna_ | that worked actually | 20:55 |
| smcginnis | So maybe our api-ref just doesn't mention that name is also an option. | 20:56 |
| hemna_ | first command I've ever seen work in openstack client and not in cinder client | 20:57 |
| hemna_ | heh | 20:57 |
| smcginnis | Oh, actually, it does say name is an option: https://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded=create-or-update-extra-specs-for-volume-type-detail,update-extra-specification-for-volume-type-detail,update-a-volume-type-detail#update-a-volume-type | 20:57 |
| smcginnis | So bug in python-cinderclient. I guess that's better than it being a bug in cinder API. | 20:57 |
| hemna_ | heh yah | 20:57 |
| hemna_ | I checked the volume info as well after changing the name | 20:58 |
| hemna_ | cinder show <id> | 20:58 |
| hemna_ | volume_type name was updated | 20:58 |
| hemna_ | which I thought it should be since it should simply be an id reference | 20:58 |
| *** Lucas_Gray has joined #openstack-cinder | 20:58 | |
| hemna_ | I have thousands of volumes on a specific volume type | 20:59 |
| hemna_ | and kinda need to rename it | 20:59 |
| hemna_ | didn't want to have to do a retype | 20:59 |
| smcginnis | Looks like the client has it in the class at least: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volume_types.py#L149 | 20:59 |
| hemna_ | ok maybe I'm just old | 21:00 |
| hemna_ | cinder type-update --name foo <id> should work | 21:00 |
| * hemna_ is lame | 21:00 | |
| smcginnis | Yeah, just found https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/shell.py#L786 | 21:00 |
| hemna_ | nevermind me...back to the crack pipe | 21:01 |
| smcginnis | :) | 21:01 |
| *** tobberydberg has joined #openstack-cinder | 21:01 | |
| *** ociuhandu has joined #openstack-cinder | 21:02 | |
| *** tobberydberg has quit IRC | 21:02 | |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: Add test for check dependencies between incr backups https://review.opendev.org/652771 | 21:02 |
| *** tobberydberg has joined #openstack-cinder | 21:03 | |
| *** ociuhandu has quit IRC | 21:06 | |
| *** Lucas_Gray has quit IRC | 21:06 | |
| *** KeithMnemonic has joined #openstack-cinder | 21:08 | |
| *** Lucas_Gray has joined #openstack-cinder | 21:09 | |
| *** Lucas_Gray has quit IRC | 21:15 | |
| *** KeithMnemonic has quit IRC | 21:16 | |
| *** KeithMnemonic has joined #openstack-cinder | 21:17 | |
| *** ociuhandu has joined #openstack-cinder | 21:20 | |
| *** Lucas_Gray has joined #openstack-cinder | 21:20 | |
| *** Lucas_Gray has quit IRC | 21:32 | |
| *** Lucas_Gray has joined #openstack-cinder | 21:42 | |
| *** Lucas_Gray has quit IRC | 21:52 | |
| *** ociuhandu has quit IRC | 21:55 | |
| *** ociuhandu has joined #openstack-cinder | 21:56 | |
| *** ociuhandu has quit IRC | 22:01 | |
| *** ociuhandu has joined #openstack-cinder | 22:46 | |
| *** tkajinam has joined #openstack-cinder | 22:46 | |
| *** ociuhandu has quit IRC | 23:01 | |
| openstackgerrit | Merged openstack/cinder stable/train: Mark Nimble Storage Driver supported https://review.opendev.org/718758 | 23:11 |
| *** ircuser-1 has joined #openstack-cinder | 23:19 | |
| *** tosky has quit IRC | 23:20 | |
| *** ociuhandu has joined #openstack-cinder | 23:35 | |
| *** ociuhandu has quit IRC | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!