openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/620577 | 00:12 |
---|---|---|
enriquetaso | xonsh looks cool, I'll try it | 00:15 |
*** tosky has quit IRC | 00:17 | |
*** martinkennelly has quit IRC | 00:32 | |
*** martinkennelly has joined #openstack-cinder | 00:36 | |
*** hamalq has quit IRC | 00:41 | |
*** martinkennelly has quit IRC | 00:46 | |
*** martinkennelly has joined #openstack-cinder | 00:47 | |
openstackgerrit | Sofia Enriquez proposed openstack/cinder master: [doc] remove outdated package python-keystone https://review.opendev.org/c/openstack/cinder/+/778119 | 00:51 |
*** martinkennelly has quit IRC | 00:55 | |
*** martinkennelly has joined #openstack-cinder | 00:56 | |
*** jv_ has quit IRC | 01:07 | |
*** n-saito has quit IRC | 01:17 | |
*** n-saito has joined #openstack-cinder | 01:18 | |
*** carloss has quit IRC | 01:42 | |
*** n-saito has quit IRC | 01:42 | |
*** n-saito has joined #openstack-cinder | 01:44 | |
*** zzzeek has quit IRC | 01:49 | |
*** zzzeek has joined #openstack-cinder | 01:51 | |
*** martinkennelly has quit IRC | 01:57 | |
rm_work | This bug is causing us some amount of consternation by the way, fix looks pretty simple if the review could get some eyes: https://review.opendev.org/c/openstack/python-cinderclient/+/776311 | 02:17 |
rm_work | thanks! | 02:17 |
*** rcernin has quit IRC | 02:34 | |
*** zenkuro has quit IRC | 02:36 | |
*** rcernin has joined #openstack-cinder | 02:47 | |
openstackgerrit | Lance Bragstad proposed openstack/cinder master: Implement project personas for volume action API https://review.opendev.org/c/openstack/cinder/+/760192 | 02:56 |
openstackgerrit | Xuan Yandong proposed openstack/os-brick master: Remove six https://review.opendev.org/c/openstack/os-brick/+/754598 | 02:56 |
*** swegener has quit IRC | 03:04 | |
openstackgerrit | Lance Bragstad proposed openstack/cinder master: Implement project personas for volume API https://review.opendev.org/c/openstack/cinder/+/760199 | 03:20 |
*** vishalmanchanda has joined #openstack-cinder | 03:44 | |
*** psachin has joined #openstack-cinder | 03:46 | |
*** zzzeek has quit IRC | 04:32 | |
*** zzzeek has joined #openstack-cinder | 04:33 | |
*** venkatakrishnath has joined #openstack-cinder | 04:38 | |
*** enriquetaso has quit IRC | 04:48 | |
*** enriquetaso has joined #openstack-cinder | 04:49 | |
*** udesale has joined #openstack-cinder | 05:08 | |
*** enriquetaso has quit IRC | 05:11 | |
*** whoami-rajat_ has joined #openstack-cinder | 05:25 | |
*** m75abrams has joined #openstack-cinder | 05:25 | |
*** brinzhang_ has quit IRC | 06:01 | |
*** brinzhang_ has joined #openstack-cinder | 06:01 | |
*** lbragstad_ has joined #openstack-cinder | 06:03 | |
*** lbragstad has quit IRC | 06:06 | |
*** zzzeek has quit IRC | 06:10 | |
*** zzzeek has joined #openstack-cinder | 06:11 | |
*** dsariel has joined #openstack-cinder | 06:12 | |
openstackgerrit | Xuan Yandong proposed openstack/os-brick master: Remove six https://review.opendev.org/c/openstack/os-brick/+/754598 | 06:21 |
*** whoami-rajat_ is now known as whoami-rajat | 06:54 | |
*** rcernin has quit IRC | 06:57 | |
*** m75abrams has quit IRC | 07:02 | |
*** felipe_rodrigues has quit IRC | 07:17 | |
*** jawad_axd has joined #openstack-cinder | 07:25 | |
*** lpetrut has joined #openstack-cinder | 07:27 | |
*** rpittau|afk is now known as rpittau | 08:22 | |
*** tosky has joined #openstack-cinder | 08:35 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Refactor: move project validation to api_utils https://review.opendev.org/c/openstack/cinder/+/752431 | 08:42 |
*** ociuhandu has joined #openstack-cinder | 08:44 | |
*** martinkennelly has joined #openstack-cinder | 08:50 | |
openstackgerrit | Xuan Yandong proposed openstack/os-brick master: Remove six https://review.opendev.org/c/openstack/os-brick/+/754598 | 09:03 |
*** GirishChilukuri has joined #openstack-cinder | 09:05 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/620577 | 09:05 |
*** brinzhang0 has joined #openstack-cinder | 09:15 | |
*** brinzhang_ has quit IRC | 09:18 | |
*** zenkuro has joined #openstack-cinder | 09:31 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Add project validation to volume type access https://review.opendev.org/c/openstack/cinder/+/777574 | 09:38 |
frickler | whoami-rajat: seems your fix for https://bugs.launchpad.net/cinder/+bug/1753885 was incomplete, need to insert a "filters.pop('all_tenants')" here https://review.opendev.org/c/openstack/cinder/+/641717/6/cinder/api/common.py#b430 | 09:43 |
openstack | Launchpad bug 1753885 in Cinder "invalid filter option "all_tenants" for non-aministrator user" [Critical,Fix released] - Assigned to Rajat Dhasmana (whoami-rajat) | 09:43 |
whoami-rajat | frickler: IIRC this method is only for validating the filters and not modifying them, we remove them at later stage, are you facing any error related to this? | 09:47 |
*** dsariel has quit IRC | 09:47 | |
*** dsariel has joined #openstack-cinder | 09:47 | |
*** ociuhandu has quit IRC | 09:50 | |
*** e0ne has joined #openstack-cinder | 10:02 | |
*** e0ne has quit IRC | 10:02 | |
*** zoharm has joined #openstack-cinder | 10:06 | |
*** ociuhandu has joined #openstack-cinder | 10:06 | |
frickler | whoami-rajat: yes, "cinder --os-volume-api-version 3.30 show v1" works, "cinder --os-volume-api-version 3.31 show v1" fails. cinder-api log shows that the all_tenants key is removed in the earlier version but not in the later one | 10:13 |
frickler | as a side effect, also a command like "openstack volume set" only works with an ID and not with a name when requiring 3.42 to allow resizing an attached volume | 10:15 |
*** udesale_ has joined #openstack-cinder | 10:15 | |
whoami-rajat | frickler: this shouldn't be related to my patch since it's not bumping microversion, let me check | 10:15 |
frickler | whoami-rajat: without your patch, api returns an error. with your patch, api returns an empty list, still not what is needed | 10:17 |
*** udesale has quit IRC | 10:19 | |
whoami-rajat | frickler: as i can see the checks are all over the place, i will push a patch to make it consistent (and working) | 10:29 |
whoami-rajat | thanks for reporting | 10:29 |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Add project validation to volume type access https://review.opendev.org/c/openstack/cinder/+/777574 | 10:48 |
*** m75abrams has joined #openstack-cinder | 10:51 | |
*** carloss has joined #openstack-cinder | 11:01 | |
openstackgerrit | Zohar Mamedov proposed openstack/os-brick master: NVMeOF connector driver connection information compatibility fix https://review.opendev.org/c/openstack/os-brick/+/777086 | 11:14 |
*** psachin has quit IRC | 11:22 | |
*** udesale_ has quit IRC | 11:40 | |
*** GirishChilukuri has quit IRC | 11:42 | |
*** udesale has joined #openstack-cinder | 11:43 | |
*** GirishChilukuri has joined #openstack-cinder | 11:45 | |
*** ociuhandu has quit IRC | 11:48 | |
GirishChilukuri | @core reviewers: Please review this code changes, Need +2 to merge the code changes. | 11:49 |
GirishChilukuri | https://review.opendev.org/c/openstack/cinder/+/773937 | 11:49 |
GirishChilukuri | https://review.opendev.org/c/openstack/cinder/+/753134 | 11:49 |
GirishChilukuri | https://review.opendev.org/c/openstack/cinder/+/771783 | 11:49 |
GirishChilukuri | https://review.opendev.org/c/openstack/cinder/+/765060 | 11:49 |
*** hoonetorg has quit IRC | 11:49 | |
*** ociuhandu has joined #openstack-cinder | 11:50 | |
*** ociuhandu has quit IRC | 11:50 | |
*** ociuhandu has joined #openstack-cinder | 11:52 | |
*** ociuhandu has quit IRC | 11:57 | |
*** Luzi has joined #openstack-cinder | 11:58 | |
*** hoonetorg has joined #openstack-cinder | 12:03 | |
*** ociuhandu has joined #openstack-cinder | 12:12 | |
*** ociuhandu has quit IRC | 12:17 | |
*** ociuhandu has joined #openstack-cinder | 12:18 | |
*** zzzeek has quit IRC | 12:20 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix automatic quota refresh for migrating volumes https://review.opendev.org/c/openstack/cinder/+/778182 | 12:22 |
*** ociuhandu has quit IRC | 12:22 | |
*** zzzeek has joined #openstack-cinder | 12:23 | |
*** GirishChilukuri has quit IRC | 12:25 | |
openstackgerrit | Merged openstack/os-brick stable/victoria: FC: Fix not flushing on detach https://review.opendev.org/c/openstack/os-brick/+/777092 | 12:32 |
*** ociuhandu has joined #openstack-cinder | 12:35 | |
*** enriquetaso has joined #openstack-cinder | 12:36 | |
*** ociuhandu has quit IRC | 12:44 | |
*** jv_ has joined #openstack-cinder | 13:05 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/620577 | 13:18 |
*** ociuhandu has joined #openstack-cinder | 13:27 | |
*** lbragstad_ is now known as lbragstad | 13:41 | |
*** dsariel has quit IRC | 13:52 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: mypy: generate stubs for imported libs https://review.opendev.org/c/openstack/cinder/+/775007 | 14:13 |
openstackgerrit | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 14:19 |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/620577 | 14:29 |
*** dsariel has joined #openstack-cinder | 14:30 | |
*** jawad_axd has quit IRC | 14:31 | |
*** jawad_axd has joined #openstack-cinder | 14:32 | |
*** Luzi has quit IRC | 14:36 | |
whoami-rajat | rosmaita: hey, these changes were not in the original tracker doc (not even added later) but this one seems important to get into victoria https://review.opendev.org/c/openstack/os-brick/+/776216 | 14:44 |
whoami-rajat | not sure about this one https://review.opendev.org/c/openstack/python-cinderclient/+/776688 | 14:45 |
*** atmark has joined #openstack-cinder | 14:55 | |
atmark | hello, is it possible to configure to have volumes backed by rbd for set of computes and volumes backed by local disk for another set of computes? | 14:57 |
*** jawad_axd has quit IRC | 15:15 | |
*** jawad_axd has joined #openstack-cinder | 15:16 | |
*** jawad_axd has quit IRC | 15:24 | |
*** zenkuro has quit IRC | 15:27 | |
openstackgerrit | Merged openstack/cinder master: Add ports filtering support to Dell EMC XtremIO driver https://review.opendev.org/c/openstack/cinder/+/775798 | 15:30 |
*** lpetrut has quit IRC | 15:34 | |
*** openstackgerrit has quit IRC | 15:35 | |
*** openstackgerrit has joined #openstack-cinder | 15:41 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix until_refresh config changes https://review.opendev.org/c/openstack/cinder/+/778210 | 15:41 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix until_refresh config changes https://review.opendev.org/c/openstack/cinder/+/778210 | 15:46 |
*** venkatakrishnath has quit IRC | 15:57 | |
openstackgerrit | Zohar Mamedov proposed openstack/os-brick master: NVMeOF connector driver connection information compatibility fix https://review.opendev.org/c/openstack/os-brick/+/777086 | 16:09 |
*** dsariel has quit IRC | 16:15 | |
*** dviroel has quit IRC | 16:34 | |
*** dsariel has joined #openstack-cinder | 16:35 | |
*** e0ne has joined #openstack-cinder | 16:37 | |
whoami-rajat | jungleboyj: smcginnis hey, can you take a look at this backport? it's good enhancement to have in victoria https://review.opendev.org/c/openstack/python-cinderclient/+/776688 | 16:42 |
hemna | what is the default timeout for our rpc api calls ? | 16:43 |
smcginnis | whoami-rajat: "backport" and "enhancement" usuallly don't go together. ;) But I'll take a look after this call. | 16:43 |
jungleboyj | smcginnis: Bwah ha ha. | 16:44 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Add warning message about slow volume backend https://review.opendev.org/c/openstack/cinder/+/766751 | 16:44 |
rosmaita | hemna: i think 60 sec | 16:45 |
hemna | ok | 16:45 |
jungleboyj | rosmaita: So you are +2 on this: https://review.opendev.org/c/openstack/python-cinderclient/+/776688 | 16:46 |
whoami-rajat | smcginnis: yeah, not sure what to call it since it can't be disguised as a bugfix :/ thanks! | 16:47 |
hemna | rosmaita ok I see the setting in oslo_messaging defaulting to 60 seconds | 16:47 |
hemna | https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/rpc/client.py#L39-L41 | 16:47 |
rosmaita | cool | 16:47 |
rosmaita | jungleboyj: yes, but i'm only 1 vote | 16:47 |
rosmaita | and good thing smcginnis took a look | 16:48 |
jungleboyj | Yeah, now that I read that ... not ok. | 16:50 |
whoami-rajat | yeah, it's not clear with the code but the v3 method is overriding the v2 one so we need to include --volume-id as a parameter in the v3 method as well | 16:52 |
jungleboyj | whoami-rajat: So, I can see that the fact that this wasn't implemented was kind of a gap. | 16:56 |
whoami-rajat | jungleboyj: since '--volume-id' was deprecated in v2 also and '--volume' was a recommended way of doing, i think it's safe to not include that option in v3 but for a backport, that might cause inconsistencies | 16:59 |
jungleboyj | Right. Making that kind of a change in a backport isn't allowed. | 16:59 |
whoami-rajat | yep, agreed | 17:00 |
smcginnis | Yeah, not allowed with stable policy to remove functionality on a stable release. And current release will need to be a major version bump. (hopefully that hasn't already gone out otherwise) | 17:00 |
*** m75abrams has quit IRC | 17:01 | |
smcginnis | Release note on the original patch also should have called that out. | 17:01 |
whoami-rajat | was thinking the same thing ^^ | 17:02 |
whoami-rajat | abishop: ^^^ | 17:03 |
*** udesale has quit IRC | 17:04 | |
*** zoharm has quit IRC | 17:04 | |
hemna | am getting an rpc timeout doing a backup, due to cinder's default behavior of creating a temporary clone of the volume being backed up, and that cloning operation taking longer than the default rpc timeout (which we have modified to 300 seconds) | 17:08 |
hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/rpcapi.py#L343-L352 | 17:09 |
hemna | that guy takes too long to return | 17:09 |
abishop | whoami-rajat: not much to add, other than "ack" | 17:09 |
hemna | some clone operations are painful for backends | 17:09 |
hemna | and take a long time depending on the size of the volume being cloned | 17:09 |
*** ociuhandu_ has joined #openstack-cinder | 17:09 | |
*** ociuhandu has quit IRC | 17:12 | |
smcginnis | Would love to get this in so I can automate the CI report generation: https://review.opendev.org/c/openstack/cinder/+/776458 | 17:12 |
*** ociuhandu_ has quit IRC | 17:13 | |
jungleboyj | +2 from me. :-) | 17:16 |
smcginnis | Thanks! | 17:16 |
*** lpetrut has joined #openstack-cinder | 17:28 | |
openstackgerrit | Merged openstack/os-brick stable/victoria: ScaleIO: More connection info backward compatibility https://review.opendev.org/c/openstack/os-brick/+/776216 | 17:28 |
openstackgerrit | Alan Bishop proposed openstack/os-brick stable/ussuri: ScaleIO: More connection info backward compatibility https://review.opendev.org/c/openstack/os-brick/+/778224 | 17:41 |
*** rpittau is now known as rpittau|afk | 17:42 | |
jbernard | hemna: for the backend in question, is a clone necessary to perform the backup operation? | 17:43 |
jbernard | hemna: perhaps that behaviour could be selectable for backends that struggle in the clone department | 17:44 |
*** tobias-urdin has quit IRC | 18:02 | |
openstackgerrit | Merged openstack/cinder master: PowerMax Driver - Port status check https://review.opendev.org/c/openstack/cinder/+/761643 | 18:04 |
*** lpetrut has quit IRC | 18:07 | |
*** hamalq has joined #openstack-cinder | 18:38 | |
openstackgerrit | Ratnakaram Rajesh proposed openstack/cinder master: Update code layout and missing Zadara features https://review.opendev.org/c/openstack/cinder/+/774463 | 19:01 |
*** rratnaka has joined #openstack-cinder | 19:03 | |
*** rratnaka has quit IRC | 19:11 | |
legochen | hey team, I enabled the - image_volume_cache_enabled = True . I can see the cinder/volume/manager.py load this attribute properly after inject the dump codes there. | 20:06 |
legochen | ax_cache_entries: 0 | 20:06 |
legochen | /opt/openstack/venv/cinder/lib/python3.6/site-packages/cinder/volume/manager.py:__init__:302 image_volume_cache_enabled: True, max_cache_size: 0, m | 20:06 |
legochen | ax_cache_entries: 0 | 20:06 |
legochen | but, don’t know why it doesn’t work in cinder/volume/flows/manager/create_volume.py | 20:08 |
legochen | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L1012 | 20:08 |
legochen | I also dump the variables’ values there - 2021-03-02 20:04:33,790 INFO [cinder.volume.flows.manager.create_volume] /opt/openstack/venv/cinder/lib/python3.6/site-packages/cinder/volume/flows/manager/create_volume.py:_create_from_image:1019 cloned: False , self.image_volume_cache: | 20:09 |
legochen | None , volume_is_encrypted: False | 20:09 |
legochen | you can see the self.image_volume_cache is set to None | 20:09 |
legochen | rosmaita ^^ could you please give me some guidance about this :) | 20:10 |
rosmaita | legochen: will take a look in a bit | 20:11 |
legochen | I also dump the value after this line. https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L389 | 20:12 |
legochen | the result is NONE as well. | 20:12 |
rosmaita | in the meantime, eric was wondering about that config problem you found yesterday ... did you define those config options in the [DEFAULT] section of the file? that might be why they weren't working ... he thinks they should be defined in the backend's part of the config file, or in with the [backend_defaults} | 20:13 |
legochen | this is my command to trigger this code - openstack volume create --type ScaleIO --size 5 --image rhel-7.9.11 bootable-img-1 | 20:13 |
legochen | you mean yesterday’s bug? I defined all them in [DEFAULT]. | 20:14 |
rosmaita | ok, that might be the problem, then | 20:15 |
legochen | rosmaita - https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.1/html/configuration_reference/cinder | 20:17 |
legochen | I refer to this doc and see both “image_conversion_address_space_limit” and “image_conversion_cpu_limit” defined in [DEFAULT] section. | 20:18 |
legochen | btw, is there a group “[backend_defaults]” can use? | 20:19 |
legochen | I’m okay to define “image_conversion_address_space_limit” and “image_conversion_cpu_limit” to my all backends and see if that could work. | 20:19 |
rosmaita | if you add a [backend_defaults] section, you can put options there that are shared by multiple backends | 20:20 |
legochen | rosmatia, about the today’s issue - image_volume_cache_enabled . I defined this attribute in [DEFAULT] and all my backends, but, it doesn’t work as expected. I think the CONF load is good. Just don’t know why the “_create_from_image” function thinks the cache is not enabled. | 20:22 |
legochen | okay, thanks for that. will try :) | 20:22 |
rosmaita | legochen: i think the image_volume_cache_enabled option is a per-backend option, so cinder won't find it in the [DEFAULT] section | 20:24 |
*** Underknowledge has quit IRC | 20:26 | |
atmark | hello, is it possible to configure to have volumes backed by rbd specific set of computes and volumes backed by local disk for another set of computes? | 20:26 |
*** Underknowledge has joined #openstack-cinder | 20:26 | |
legochen | rosmaita - https://docs.openstack.org/cinder/rocky/configuration/block-storage/samples/cinder.conf.html | 20:26 |
legochen | as the sample cinder.conf, it mentions image_volume_cache_enabled in [DEFAULT] section. | 20:27 |
legochen | rosmaita, I moved “image_conversion_address_space_limit” and “image_conversion_cpu_limit” to [backend_defaults] and all my backend sections…. howerever, it still doesn’t work | 20:30 |
rosmaita | legochen: thanks for trying that ... when you have a few minutes, please put a comment on the bug saying that you tried it | 20:31 |
rosmaita | (will save someone else some time) | 20:31 |
legochen | sure, let me do it just now | 20:31 |
rosmaita | thanks! | 20:31 |
rosmaita | legochen: so about the image_volume_cache_enabled setting ... it also shows up later in that sample config in the [backend_defaults] setting | 20:32 |
rosmaita | i think the setting in [DEFAULT] is leftover from when you could only have one backend with cinder | 20:33 |
rosmaita | (which was before i started working on it, so i'm not sure) | 20:33 |
rosmaita | it is definitely confusing, though | 20:33 |
rosmaita | atmark: what time zone are you in? | 20:36 |
legochen | rosmaita, looks like after I moved image_volume_cache_enabled to [backend_defaults], it works fine now | 20:39 |
rosmaita | \o/ | 20:39 |
rosmaita | legochen: you might want to file a bug about that, because we need to fix the docs or the sample config generation or something ... it's definitely confusing | 20:41 |
legochen | sure, will do. let me validate more about that first. | 20:42 |
rosmaita | thanks! | 20:43 |
atmark | rosmaita: EST | 20:46 |
rosmaita | atmark: the agenda for tomorrow's cinder meeting at 1400-1500 UTC is pretty light, why don't you put it on the agenda and you can ask at the meeting? | 20:47 |
rosmaita | you may be able to do something with availability zones, but i am really not sure | 20:47 |
rosmaita | https://etherpad.opendev.org/p/cinder-wallaby-meetings | 20:47 |
*** e0ne has quit IRC | 20:55 | |
*** dviroel has joined #openstack-cinder | 20:55 | |
*** abhishekk has quit IRC | 21:06 | |
*** bhagyashri|rover has quit IRC | 21:06 | |
*** abhishekk has joined #openstack-cinder | 21:06 | |
*** bhagyashris has joined #openstack-cinder | 21:07 | |
*** dsariel has quit IRC | 21:07 | |
openstackgerrit | Merged openstack/cinder master: Fix CI_WIKI_NAME entries https://review.opendev.org/c/openstack/cinder/+/776458 | 21:17 |
openstackgerrit | Merged openstack/cinder stable/ussuri: Require oslo.serialization 3.1.2 https://review.opendev.org/c/openstack/cinder/+/774680 | 21:17 |
openstackgerrit | Merged openstack/cinder stable/ussuri: [SVF]:Reduce slowness by caching pool information https://review.opendev.org/c/openstack/cinder/+/767478 | 21:18 |
*** hoonetorg has quit IRC | 21:21 | |
legochen | romaita, one more question about the output from openstack volume show <volume-id>. I’d like to know where to configure the format of “os-vol-host-attr:host”. | 21:40 |
legochen | in my use case, it shows - | 21:41 |
legochen | | os-vol-host-attr:host | my-host1.test.com@vxflexos-ssd#pd1:pd1-ssd | 21:41 |
legochen | another volume shows: | 21:41 |
legochen | | os-vol-host-attr:host | my-host2.test.com@vxflexos-ssd#pd1:pd1-ssd | 21:41 |
legochen | looks like the format is <hostname>@<backend_name>#<pool_name> | 21:42 |
legochen | I’m curious is this defined by backend driver or can customize by admins? | 21:42 |
*** hoonetorg has joined #openstack-cinder | 21:42 | |
legochen | as I have multiple cinder api servers behind LB. | 21:43 |
openstackgerrit | Merged openstack/cinder stable/ussuri: RBD: Pass bytes type for mon_command inbuf https://review.opendev.org/c/openstack/cinder/+/777545 | 21:43 |
legochen | so, the cinder API requests will dispatch to different api services. | 21:43 |
legochen | I’m thinking can I let the host portion of “os-vol-host-attr:host “ could be my LB CNAME instead of “hostname” of server who handling the request | 21:45 |
*** brinzhang0 has quit IRC | 21:47 | |
*** rcernin has joined #openstack-cinder | 22:33 | |
*** enriquetaso has quit IRC | 22:39 | |
*** vishalmanchanda has quit IRC | 22:54 | |
*** enriquetaso has joined #openstack-cinder | 23:10 | |
*** zzzeek has quit IRC | 23:13 | |
*** zzzeek has joined #openstack-cinder | 23:17 | |
*** carloss has quit IRC | 23:40 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!