Friday, 2018-05-04

*** gyee has quit IRC00:02
*** markvoelker has quit IRC00:05
*** jistr has quit IRC00:07
*** jistr has joined #openstack-glance00:13
*** mingyu_ has joined #openstack-glance00:34
*** mingyu has quit IRC00:35
*** d0ugal has quit IRC00:36
*** edmondsw has joined #openstack-glance00:53
*** d0ugal has joined #openstack-glance01:03
*** edmondsw has quit IRC01:05
*** markvoelker has joined #openstack-glance01:05
*** dpawlik has joined #openstack-glance01:15
*** edmondsw has joined #openstack-glance01:18
*** dpawlik has quit IRC01:20
*** btully has joined #openstack-glance01:33
*** rosmaita has quit IRC01:35
*** btully has quit IRC01:38
*** markvoelker has quit IRC01:40
*** dalgaaf has quit IRC01:53
*** dalgaaf has joined #openstack-glance01:55
*** r-daneel has quit IRC02:06
*** edmondsw has quit IRC02:18
*** rcernin has quit IRC02:19
*** rcernin has joined #openstack-glance02:20
*** mingyu_ has quit IRC02:22
*** mingyu has joined #openstack-glance02:22
*** markvoelker has joined #openstack-glance02:37
*** germs has quit IRC02:39
*** germs has joined #openstack-glance02:40
openstackgerritwangxiyuan proposed openstack/glance master: CI check  https://review.openstack.org/56620602:56
*** markvoelker has quit IRC03:10
*** dpawlik has joined #openstack-glance03:16
*** dpawlik has quit IRC03:21
*** btully has joined #openstack-glance03:22
*** btully has quit IRC03:27
*** pdeore has joined #openstack-glance03:29
*** nicolasbock has quit IRC03:44
*** mingyu has quit IRC03:46
*** mingyu has joined #openstack-glance03:48
*** rcernin has quit IRC03:59
*** rcernin has joined #openstack-glance04:00
*** edmondsw has joined #openstack-glance04:00
*** germs has quit IRC04:02
*** germs has joined #openstack-glance04:02
*** germs has quit IRC04:02
*** germs has joined #openstack-glance04:02
*** gyankum has joined #openstack-glance04:04
*** edmondsw has quit IRC04:05
*** markvoelker has joined #openstack-glance04:08
*** r-daneel has joined #openstack-glance04:10
*** r-daneel_ has joined #openstack-glance04:35
*** r-daneel has quit IRC04:36
*** r-daneel_ is now known as r-daneel04:36
*** markvoelker has quit IRC04:41
*** Taseer has joined #openstack-glance04:42
*** bhagyashris has quit IRC04:43
*** bhagyashris has joined #openstack-glance04:43
*** abhishekk has joined #openstack-glance04:49
*** pas-ha has quit IRC05:15
*** hogepodge has quit IRC05:15
*** udesale has joined #openstack-glance05:15
*** hogepodge has joined #openstack-glance05:16
*** pas-ha has joined #openstack-glance05:16
*** dpawlik has joined #openstack-glance05:17
*** dpawlik has quit IRC05:21
*** markvoelker has joined #openstack-glance05:38
*** Taseer has quit IRC05:41
*** threestrands has quit IRC05:47
*** ratailor has joined #openstack-glance05:47
*** mosulica has joined #openstack-glance05:58
*** markvoelker has quit IRC06:11
*** mosulica has quit IRC06:12
*** tshefi has quit IRC06:13
*** pcaruana has joined #openstack-glance06:43
*** aniketh has joined #openstack-glance06:47
*** r-daneel has quit IRC06:49
*** tshefi has joined #openstack-glance06:55
*** tshefi has quit IRC07:00
*** btully has joined #openstack-glance07:01
*** rcernin has quit IRC07:05
*** btully has quit IRC07:05
*** markvoelker has joined #openstack-glance07:08
*** dpawlik has joined #openstack-glance07:10
*** dpawlik has quit IRC07:16
*** dpawlik has joined #openstack-glance07:19
*** tesseract has joined #openstack-glance07:20
*** alexchadin has joined #openstack-glance07:32
*** amoralej|off is now known as amoralej07:34
*** edmondsw has joined #openstack-glance07:36
*** edmondsw has quit IRC07:41
*** markvoelker has quit IRC07:42
*** slunkad has quit IRC07:45
*** kukacz has quit IRC07:46
*** slunkad has joined #openstack-glance07:49
*** kukacz_ has joined #openstack-glance07:55
*** kukacz_ has quit IRC08:01
*** kukacz_ has joined #openstack-glance08:01
*** kukacz_ has quit IRC08:07
*** kukacz_ has joined #openstack-glance08:11
*** kukacz_ has quit IRC08:14
*** kukacz_ has joined #openstack-glance08:14
*** mosulica has joined #openstack-glance08:15
*** stephenfin is now known as finucannot08:29
*** markvoelker has joined #openstack-glance08:39
*** germs has quit IRC08:43
*** germs has joined #openstack-glance08:44
*** germs has quit IRC08:44
*** germs has joined #openstack-glance08:44
*** lbragstad has quit IRC08:45
*** lbragstad has joined #openstack-glance08:45
*** btully has joined #openstack-glance08:50
*** btully has quit IRC08:54
*** markvoelker has quit IRC09:12
*** MattMan has quit IRC09:19
*** MattMan has joined #openstack-glance09:19
*** udesale_ has joined #openstack-glance09:29
*** udesale__ has joined #openstack-glance09:31
*** udesale has quit IRC09:32
*** dtantsur|afk is now known as dtantsur09:33
*** udesale_ has quit IRC09:34
*** pdeore has quit IRC09:37
*** pdeore has joined #openstack-glance09:37
*** pdeore has quit IRC09:38
*** pdeore has joined #openstack-glance09:40
openstackgerritAbhishek Kekane proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626209:42
abhishekkjokke_, you around?09:43
*** kholkina has joined #openstack-glance09:54
openstackgerritAbhishek Kekane proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626209:57
jokke_abhishekk: yup09:58
abhishekkjokke_, good time to discuss for multi-store specs?09:59
jokke_can you give me like 10 min before?10:02
abhishekkjokke_, yes10:04
jokke_cool I'll be right back10:04
abhishekkok10:07
*** markvoelker has joined #openstack-glance10:08
*** udesale_ has joined #openstack-glance10:13
*** udesale__ has quit IRC10:16
jokke_ok, sorry for that ... so what's on your mind?10:24
abhishekkso I have uploaded a new patch for the spec10:25
abhishekkhttps://review.openstack.org/#/c/562467/5/specs/rocky/approved/glance/multi-store.rst10:25
abhishekkwhere I have proposed to deprecate stores and default_store config option and introduce new options, enable_backends and default_multi_store10:26
abhishekkAlso recommended to add new store like, add, get, delete etc and old will be removed after the end of deprecation period10:28
abhishekk* new store api's10:28
jokke_Just a sec ... I'm reading the latest changes you made to the spec10:28
abhishekkyep10:29
jokke_thanks for the details there. Even I do understand what the problem is ;P10:29
*** r-daneel has joined #openstack-glance10:29
abhishekk:D10:29
*** mvk has quit IRC10:30
jokke_ok, so I do see one thing I'm not sure how to tackle yet, but lets go through first what you had on your mind ... so what exactly you wanted talk about?10:34
abhishekkI wanted to talk about should we use same default_store option and remove the choices option from it to avoid failure which I have mentioned?10:35
abhishekk2. I am keeping old code as it is which will be removed after deprecation period is over10:36
abhishekk3. Or still we should go in earlier way which I posted in PS 4?10:37
jokke_ok, so we definitely want to have some default store there to keep the user experience as seamless as possible10:37
jokke_for those users who does not care where their image lives, we should not expect them to specify the store as they haven't needed to do so far10:38
*** btully has joined #openstack-glance10:39
jokke_so there needs to be the operator preferred store if you wish where the images will land if the user does not specify which store they want to use10:39
jokke_and I think it needs to be mandatory to set to keep the user experience10:39
jokke_I hope that makes sense10:40
abhishekkSo if default_store is not set then it should be the first store from the enable_backends ?10:40
jokke_no I think it always needs to be set10:40
abhishekkSo if not set then error will be ok?10:41
jokke_because specially if the config option is dict we never know what the first one happens to be10:41
abhishekkjokke_, that is correct10:41
jokke_we want the operators to specify one store and that being consistent even if they happen to add more stores etc10:41
jokke_like they had done until now10:42
jokke_so it really doesn't change anything10:42
abhishekkthen current proposal sounds good? if not set by the operator then it will raise error?10:42
jokke_even if you had only one store enabled you still had to specify the default10:42
*** markvoelker has quit IRC10:42
jokke_I'm fine with that10:43
abhishekkin current case if operator doesn't sepcify the it takes default as file10:43
jokke_it does?10:43
abhishekkyes10:43
*** btully has quit IRC10:43
jokke_and what happens if file store is not in the enabled stores list?10:43
jokke_does it just barf?10:43
abhishekkthat I haven't checked ;)10:44
abhishekkhttps://github.com/openstack/glance_store/blob/master/glance_store/backend.py#L5810:44
jokke_yeah10:45
jokke_so it's mentioned there that the store needs to be enabled, but I'm not sure either how it behaves if it's not10:46
abhishekkI will check that behavior on monday during my time10:46
jokke_and the second question you had about the old code and deprecation, we will need to maintain the backwards compatibility for the glance_store API until the deprecation and removal10:47
jokke_which might be pain10:47
abhishekkSo for that, I think I will write totally new code and add deprecation comment to the code10:48
abhishekkold code I mean10:48
jokke_that likely is the easiest approach10:48
jokke_which brings us to my concern10:49
abhishekktell me10:49
jokke_for upgrade migration path. Say cloud get upgraded to the new code how in earth do we deal with the migrations of the locations from the old stores to the new ones?10:50
jokke_say operator wants to use 3 rdb stores instead of just one10:50
jokke_how we make the decision which of the 3 stores that image record actually belongs to10:50
jokke_same with if the operator decides to change the store-id/name of the store10:51
jokke_'cause we obviously need to facilitate this somehow in the db end10:51
abhishekk2nd one will be really a mess10:51
abhishekk1st case we can handle with easy way out10:52
abhishekkbut second one we should not allow and document the same?10:52
abhishekkFor 1st one, if existing user is using ceph backend say ceph1 and now he added two new backends ceph2 and ceph3 then he should mention ceph1 as default store so that he will be able to download his earlier images10:54
jokke_I'm not sure if not allowing it is just good enough ... I bet that will raise quite a bit of resistance ..10:54
abhishekkthen, we need to provide one utility tool to take care of that10:54
jokke_no I mean if existing user uses the rdb driver and they move to ceph1, ceph2 and ceph3 ... they all uses the same driver but different backend10:55
jokke_how do we deal about that in the locations records10:55
abhishekkthat what I am saying10:56
abhishekksee the existing backend is ceph with rbd driver which will now become ceph1 and operator will add two new ceph backends ceph2 and ceph310:57
jokke_I don't think it's realistic assumption to expect all the operators to stick with their old backend as default forever just because they have some legacy images10:57
jokke_there is no continuation path what so ever for deprecating that old store10:58
jokke_as it will get all the new images per default10:58
abhishekkwhat's the problem in mentioning old store as a default one?10:59
jokke_one option would be to have that utility code, like glance-manage doing the translation on the operator's request10:59
abhishekkbecause we are going to provide a facility to upload the image to specified backend10:59
jokke_well say the operator moves to ceph1 ceph2 ceph3 .... year down the line they want to deprecate ceph1 and change the default to ceph4 to bleed to ceph1 empty for removal on natural image lifecycle11:00
abhishekkyes, that will be one option to have a utility function in glance-manage which will update the location metadata for existing images11:00
abhishekkjokke_, understood11:01
jokke_they won't be able to set the default away from ceph1 as the code depends the old store being default11:01
abhishekkjokke_, got it11:01
jokke_the other option would be likely quite messy as well but we could walk through all the rdb stores when the image is requested and update the location record on access when we figure out in which store it actually lives in11:02
abhishekkSo we can provide two utility function, 1 for updating existing images and 2 for if operator wants to change the store identifier11:02
jokke_my main point is, we need to have some plan how to do this11:02
abhishekk2nd option will hit the performance11:03
jokke_it will delay the first access of the old style location a bit11:04
*** alexchadin has quit IRC11:04
abhishekkI will work on second approach which you have suggested11:05
jokke_but it would be easier migration for everyone and I think possibly more resilient in case some error is made11:05
abhishekkyep11:05
jokke_It just might be quite pain to do11:05
*** aniketh has quit IRC11:06
abhishekkyes11:06
jokke_so it was just an idea I wanted to throw out there as an option. My main point is the whole migration path from old to new is not taken into account in the spec at all11:06
jokke_so we need to have some plan documented and discussed there11:06
abhishekkI have hit this, but I thought mentioning the old store as default will solve the problem ;)11:07
abhishekkOk, so on monday I will add this in the specs11:08
jokke_thanks ... did you have something else on the specifics or is this sufficient for you to move forward and I'll just wait for the new revision next week?11:09
abhishekkjokke_, please add these concerns on the specs for the record11:09
abhishekkjokke_, I guess at the moment this is sufficient11:10
jokke_ok, will do11:10
abhishekkthank you for your time :D11:10
jokke_always11:10
jokke_now go and enjoy weekend with the family ;D11:11
jokke_it's getting late for you11:11
abhishekkjokke_, thank you :D11:11
abhishekkhave a nice weekend11:11
jokke_U211:11
*** abhishekk has quit IRC11:14
*** alexchadin has joined #openstack-glance11:17
*** udesale__ has joined #openstack-glance11:22
*** udesale__ has quit IRC11:22
*** udesale_ has quit IRC11:24
*** pcaruana has quit IRC11:27
*** markvoelker has joined #openstack-glance11:39
*** pcaruana has joined #openstack-glance11:39
*** nicolasbock has joined #openstack-glance11:42
*** linkmark has quit IRC11:51
*** rosmaita has joined #openstack-glance11:53
*** edmondsw has joined #openstack-glance11:53
*** pcaruana has quit IRC11:59
*** jistr is now known as jistr|mtg12:11
*** pcaruana has joined #openstack-glance12:12
*** markvoelker has quit IRC12:12
*** alexchadin has quit IRC12:15
*** dtantsur is now known as dtantsur|brb12:17
*** markvoelker has joined #openstack-glance12:24
*** btully has joined #openstack-glance12:27
*** jistr|mtg is now known as jistr12:31
*** btully has quit IRC12:31
*** aniketh has joined #openstack-glance12:39
*** slunkad has quit IRC12:45
*** udesale has joined #openstack-glance12:54
*** alexchadin has joined #openstack-glance12:55
*** ratailor has quit IRC12:59
*** dtantsur|brb is now known as dtantsur13:09
*** gyankum has quit IRC13:09
*** amoralej is now known as amoralej|lunch13:12
*** r-daneel has quit IRC13:16
*** mingyu has quit IRC13:26
*** slunkad has joined #openstack-glance13:47
*** r-daneel has joined #openstack-glance13:52
*** amoralej|lunch is now known as amoralej13:53
*** ratailor has joined #openstack-glance13:58
*** r-daneel has quit IRC13:59
*** r-daneel has joined #openstack-glance13:59
*** melwitt is now known as jgwentworth14:15
*** alexchadin has quit IRC14:19
*** rabel_ has joined #openstack-glance14:20
rabel_hi there. which permission needs to be set for a role in policy.json in order to allow system-wide member-update?14:20
openstackgerritMerged openstack/glance_store master: Fix wrong links in glance_store  https://review.openstack.org/55791914:32
rabel_seems to be hardcoded in authorization.py . so system-wide member-update is only for admins. am i correct?14:34
rosmaitarabel_: what do you mean by system-wide member update?14:39
*** germs has quit IRC14:39
*** germs has joined #openstack-glance14:40
*** germs has quit IRC14:40
*** germs has joined #openstack-glance14:40
rabel_rosmaita: glance member-update for member projects without having any role assigned in that project14:41
rabel_to create some kind of "image admin" role14:42
rabel_btw: image membership cannot be handled via horizon, right?14:43
rosmaitarabel_: you are correct that a user must either be an admin or in the project that's an image-member in order to change the member status ... the policy file only governs whether the user can make the member-update call or not.14:43
rosmaitarabel_: do not know about how/whether horizon does image sharing or not14:44
rabel_rosmaita: ok, thank you!14:45
*** r-daneel has quit IRC14:45
*** jistr is now known as jistr|tpb14:49
*** r-daneel has joined #openstack-glance14:52
*** dpawlik has quit IRC14:52
*** gyankum has joined #openstack-glance14:59
*** jistr|tpb is now known as jistr15:05
*** r-daneel has quit IRC15:06
*** mosulica has quit IRC15:06
*** r-daneel has joined #openstack-glance15:06
*** ratailor has quit IRC15:08
rosmaitasmcginnis, jokke_ : around?15:09
*** pcaruana has quit IRC15:13
smcginnisrosmaita: I am now for a little bit. What's up?15:20
rosmaitatwo quick doc reviews for you15:21
smcginnisOh good, those are easy ones. :)15:21
rosmaitahttps://review.openstack.org/#/c/565782/15:21
rosmaitahttps://review.openstack.org/#/c/358946/15:21
smcginnisLooking...15:21
rosmaitait's friday, don't want to give you anything too tough15:21
rosmaitathanks15:22
smcginnisWow, that's an old one.15:22
rosmaitayeah15:22
*** pooja_jadhav has quit IRC15:24
smcginnisEasy peasy15:25
*** nicolasbock has quit IRC15:26
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626215:33
*** rabel_ has left #openstack-glance15:34
openstackgerritMerged openstack/python-glanceclient master: Update property keys document  https://review.openstack.org/56578215:35
*** nicolasbock has joined #openstack-glance15:40
*** dpawlik has joined #openstack-glance15:57
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626215:59
*** gyee has joined #openstack-glance16:00
*** dpawlik has quit IRC16:02
*** germs has quit IRC16:08
*** kholkina has quit IRC16:08
*** EmilienM is now known as EvilienM16:09
*** r-daneel has quit IRC16:13
*** r-daneel has joined #openstack-glance16:13
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626216:17
*** dtantsur is now known as dtantsur|afk16:32
*** AlexeyAbashkin has joined #openstack-glance16:43
openstackgerritMerged openstack/glance master: [api-ref] "Show images" should be changed to "List images"  https://review.openstack.org/35894616:44
*** udesale has quit IRC16:50
*** mvk has joined #openstack-glance16:58
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626217:04
*** tesseract has quit IRC17:09
*** itlinux has joined #openstack-glance17:14
*** germs has joined #openstack-glance17:18
*** germs has quit IRC17:18
*** germs has joined #openstack-glance17:18
*** germs has quit IRC17:19
*** germs has joined #openstack-glance17:19
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626217:26
jokke_rosmaita: now here for a moment. How is it looking?17:53
*** pcaruana has joined #openstack-glance17:54
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626217:58
*** dpawlik has joined #openstack-glance17:58
*** dpawlik has quit IRC18:03
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626218:17
*** rosmaita has quit IRC18:22
*** AlexeyAbashkin has quit IRC18:30
*** gyankum has quit IRC18:49
*** imacdonn has quit IRC18:54
*** imacdonn has joined #openstack-glance18:55
*** rosmaita has joined #openstack-glance19:00
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM: Fix scrubber failure  https://review.openstack.org/56626219:05
*** pcaruana has quit IRC19:10
*** btully has joined #openstack-glance19:42
*** btully has quit IRC19:47
*** dpawlik has joined #openstack-glance19:59
*** dpawlik has quit IRC20:04
*** btully has joined #openstack-glance21:31
*** btully has quit IRC21:36
*** aniketh has quit IRC21:38
*** edmondsw has quit IRC21:47
*** itlinux has quit IRC21:47
*** edmondsw has joined #openstack-glance21:47
*** edmondsw has quit IRC21:52
*** r-daneel has quit IRC22:21
*** markvoelker has quit IRC22:29
*** rosmaita has quit IRC22:41
*** imacdonn has quit IRC22:42
*** linkmark has joined #openstack-glance22:52
*** mfedosin has quit IRC23:25

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!