Tuesday, 2018-07-31

*** abhishekk-mbl has quit IRC00:13
*** edmondsw has joined #openstack-glance00:42
*** edmondsw has quit IRC00:48
*** mvk_ has quit IRC01:10
*** mvk_ has joined #openstack-glance01:11
*** gyee has quit IRC01:40
*** threestrands has quit IRC02:10
*** gcb has joined #openstack-glance02:33
*** links has joined #openstack-glance03:15
*** itlinux_ has joined #openstack-glance03:21
*** btully has joined #openstack-glance03:23
*** threestrands has joined #openstack-glance03:26
*** threestrands has quit IRC03:26
*** threestrands has joined #openstack-glance03:26
*** btully has quit IRC03:27
*** tshefi has joined #openstack-glance03:28
*** rosmaita has quit IRC03:40
*** dalgaaf has quit IRC03:40
*** dalgaaf has joined #openstack-glance03:41
*** andreykurilin has quit IRC04:07
*** jmccrory has quit IRC04:09
*** kgz has quit IRC04:09
*** jmccrory has joined #openstack-glance04:13
*** andreykurilin has joined #openstack-glance04:13
*** ileixe has quit IRC04:19
*** kgz has joined #openstack-glance04:21
*** gcb has quit IRC04:22
*** abhishekk has joined #openstack-glance04:26
*** pdeore has joined #openstack-glance04:28
*** pdeore has quit IRC04:28
*** pooja_jadhav has joined #openstack-glance04:30
*** udesale has joined #openstack-glance04:50
*** btully has joined #openstack-glance05:12
*** btully has quit IRC05:16
*** ratailor has joined #openstack-glance05:35
*** gcb has joined #openstack-glance05:36
*** mosulica has joined #openstack-glance05:38
*** akki has quit IRC05:44
*** kholkina has quit IRC06:03
*** pcaruana has joined #openstack-glance06:37
*** aniketh has joined #openstack-glance06:51
*** Luzi has joined #openstack-glance07:04
*** peereb has joined #openstack-glance07:10
*** links has quit IRC07:13
*** rcernin has quit IRC07:22
*** tesseract has joined #openstack-glance07:42
*** AlexeyAbashkin has joined #openstack-glance07:42
*** threestrands has quit IRC07:45
*** udesale has quit IRC07:49
*** udesale has joined #openstack-glance07:49
*** threestrands has joined #openstack-glance08:00
*** threestrands has quit IRC08:03
*** udesale has quit IRC08:13
*** links has joined #openstack-glance08:13
*** udesale has joined #openstack-glance08:16
*** priteau has quit IRC08:28
*** priteau has joined #openstack-glance08:28
*** abhishekk has quit IRC08:48
*** ratailor has quit IRC09:20
*** ratailor has joined #openstack-glance09:24
*** abhishekk-mbl has joined #openstack-glance09:25
abhishekk-mbljokke_: hi, fixed your comments on multi-store patches09:35
*** abhishekk has joined #openstack-glance09:49
*** ratailor has quit IRC09:51
*** ratailor has joined #openstack-glance09:52
*** ratailor has quit IRC09:56
*** ratailor has joined #openstack-glance09:57
*** mvk_ has quit IRC10:16
*** btully has joined #openstack-glance10:38
*** btully has quit IRC10:43
*** mvk_ has joined #openstack-glance10:47
abhishekkjokke_, hi around?11:00
*** abhishekk has quit IRC11:04
jokke_abhishekk-mbl: yeah ... I saw11:12
*** abhishekk has joined #openstack-glance11:56
*** rosmaita has joined #openstack-glance12:06
*** MattMan has quit IRC12:10
*** MattMan has joined #openstack-glance12:10
*** edmondsw has joined #openstack-glance12:24
*** ratailor has quit IRC12:32
openstackgerritAbhishek Kekane proposed openstack/glance_store master: Wrap interface function for multihash correctly  https://review.openstack.org/58709812:50
rosmaitai was just about to do that! thanks!12:50
abhishekkrosmaita, sorry if you were already working on that12:50
rosmaitano, was literally just checking out a new repo to work in12:51
abhishekkI thought we could save time12:51
abhishekk:D12:51
rosmaitayes indeed12:51
rosmaitai haven't tested multihash with multiple backend yet, but i have tested it with filesystem and swift backends, seems to be working12:51
abhishekkrosmaita, have you confirmed tests are passing in your local environment for multihash?12:52
rosmaitayes, i had to hack the glance_store into the .tox/.../site-packages/glance_store/backend.py in each of the test environments, and everything passed12:53
abhishekkgreat12:53
rosmaitaprobably a better way to do it, but i couldn't think of it12:53
abhishekkrosmaita, I am pretty sure it will work with multi-backend as well12:53
rosmaitame too, same basic code paths12:54
abhishekkyes12:54
abhishekkLater night I will try to pull your patch and will rebase it over mine and test for confirmation12:54
rosmaitagreat12:55
openstackgerritAbhishek Kekane proposed openstack/glance master: Add multi-store support  https://review.openstack.org/57458212:55
openstackgerritAbhishek Kekane proposed openstack/glance master: Unit/Functional tests for multi store support  https://review.openstack.org/57750212:55
rosmaitaguess i better get reviewing!12:55
rosmaitai'm going to do a fresh checkout of my glance patch into a clean environment, hack in the glance_store patch, and re-run all the tests just to make sure there was no cruft hanging around12:58
abhishekksounds good13:02
*** links has quit IRC13:09
*** ygl has joined #openstack-glance13:11
yglhi all13:13
yglcan someone help with glance image listing13:14
ygla non admin user is unable to see the list of all the shared images13:14
rosmaitaright, you can only see the images shared with you that you have accepted13:15
rosmaitaor, the images with visibility=shared that you own13:15
yglrosmaita: but I have shared some images with all other users, but they cant see them all when they list them13:16
rosmaitahave they 'accepted' the images?13:16
yglits mitaka openstack13:17
yglwhen they launch the heat templates which use those images, they are able to do so13:17
yglbut cant see them when they list them13:17
rosmaitaygl: you might want to read through this doc: http://specs.openstack.org/openstack/glance-specs/specs/api/v2/sharing-image-api-v2.html13:17
rosmaitaygl: you can use an image as soon as it is shared with you, but you cannot see it in your image-list response until you 'accept' it13:18
rosmaitait's an anti-spam feature13:18
rosmaitaso what you are seeing is normal13:18
yglrosmaita: how the users can accept it ?13:18
rosmaitaeither Image API v2, glanceclient, or openstack client13:19
yglrosmaita: how can they accept it to see them in the list13:19
rosmaitaygl that doc explains everything, let me know if you have questions after reading through13:20
yglrosmaita: ok thanks for ur time13:20
openstackgerritAbhishek Kekane proposed openstack/glance master: Doc:Multiple stores support  https://review.openstack.org/57607513:22
*** abhishekk has quit IRC13:23
yglrosmaita: one question though. is this applicable to mitaka also ?13:25
rosmaitaygl: i'm pretty sure it is, image sharing has been around for a while13:26
*** links has joined #openstack-glance13:26
yglrosmaita: how can a non admin user list the pending images using the openstack command line or glance command line ?13:29
yglrosmaita: i see only rest api commands in that link to list the pending images ? how to use openstack commandline to list them ?13:30
rosmaitaygl: i don't know about the openstack client, you'll have to look at their docs13:34
rosmaitaygl: for glanceclient, you do 'glance image-list --member-status pending'13:35
yglrosmaita: ok let me try13:35
rosmaitado 'glance help image-list' to get more info13:35
rosmaitaygl the accept etc commands are member-update13:36
*** Luzi has quit IRC13:36
yglrosmaita: i dont see this option in the openstack commandline13:39
rosmaitaygl i'm pretty sure there's something in there, i forget what they call it13:41
jokke_rosmaita: abhishekk-mbl: the store patch looks good ... happy to +2 as soon as zuul agrees13:41
jokke_smcginnis: I'll keep eye on that, but as it's collab between abhishekk-mbl and rosmaita we need to review it, so will ping you once it'd be ready to go13:42
yglrosmaita: i searched but cant find such options with the openstack commandline13:43
jokke_ygl: rosmaita: I'm pretty sure one can't list pending images. The whole point is that the shared image needs to be communicated out of band to the recipient to avoid _any_ level of spamming of it13:44
jokke_admin and owner can see the users list on individual images 'though13:45
*** ygl has quit IRC13:53
smcginnisjokke_: Feel free to point me at any patches that need attention. I've been tied up in other things unfortunately, so haven't been able to go out and proactively review as much as I'd hoped.13:54
jokke_smcginnis: that's fine ... we just need to get this _store patch in and release point release from store to make the multihash working13:57
rosmaitajokke_ smcginnis abhishekk-mbl : i just did a fresh checkout of the glance multihash patch, applied the glance_store patch to the tox virtualenvs, and all the tests pass13:57
smcginnisrosmaita: Great!13:57
*** abhishekk-mbl has quit IRC14:15
rosmaitajokke_ you can do an image-list filter to see pending images, it's like opening your 'spam' folder to see what's in it; the anti-spam feature just keeps them out of your default image-list14:17
rosmaitaunless you were specifically talking about openstackclient, i don't know what they do14:17
jokke_oh you can ... I thought it was not possible ... cool14:18
rosmaitayes, it is a really well-designed feature :)14:19
rosmaitajokke_ want me to put up a quick releasenote for glance_store just mentioning the bugfix?14:28
*** itlinux_ has quit IRC14:38
*** links has quit IRC14:41
*** abhishekk-mbl has joined #openstack-glance14:45
abhishekk-mblrosmaita: great14:46
*** imacdonn has quit IRC14:53
*** imacdonn has joined #openstack-glance14:54
jokke_rosmaita: can do14:56
rosmaitayou can or you want me to?14:56
jokke_smcginnis: when you can https://review.openstack.org/#/c/58709814:57
jokke_rosmaita: go ahead14:57
rosmaitaok14:57
jokke_you can likely explain it better ;)14:57
rosmaitai'll try14:58
rosmaitayour releasenotes have been very good14:58
*** links has joined #openstack-glance15:00
*** wxy| has joined #openstack-glance15:00
*** mosulica has quit IRC15:01
*** peereb has quit IRC15:06
*** pcaruana has quit IRC15:10
*** links has quit IRC15:12
rosmaitasmcginnis https://review.openstack.org/#/c/587098/3 when you have a few minutes15:19
smcginnisrosmaita, jokke_: Has anyone brought up a requirements FFE exception with the requirements team to make sure we can actually release this?15:21
rosmaitasmcginnis don't think so15:21
smcginnisGuess we'll find out. :)15:22
rosmaitaluckily glance is the only consumer of that library at this point15:23
*** udesale has quit IRC15:23
smcginnisrosmaita: That should make it a lot easier to get a FFE.15:23
jokke_smcginnis: No and I don't think this is a new feature. This is bug fix for the store feature released that happened to miss those wrappers on some needed functions. It's well isolated and I think that's well within the stable scope for backporting15:31
smcginnisI believe we will still need an FFE since we are in requirements freeze.15:32
jokke_that's what I said to rosmaita as well on the first PS ... it's not really a new feature it's a fix so that the new feature released is actually consumable15:32
jokke_ok ... then what's the process to get that solved?15:33
smcginnisjokke_: There's some detail here: http://lists.openstack.org/pipermail/openstack-dev/2018-July/132477.html15:34
*** abhishekk has joined #openstack-glance15:36
abhishekkrosmaita, you around?15:39
rosmaitahello15:40
abhishekkI guess once multi-backend patch is merged the we need another patch so that multihash should work with multi-backend15:40
abhishekkBecause I am calling add method from glance in case of multi-backend and once your patch gets merged we need to call add_with_multihash15:41
abhishekkhttps://review.openstack.org/#/c/574582/10/glance/location.py@45715:42
abhishekk^^15:42
jokke_smcginnis: mail sent15:43
abhishekkor as this is experimental can we do it in S cycle?15:48
jokke_abhishekk: I think we need to spin that in15:49
*** links has joined #openstack-glance15:49
jokke_it shouldn't be big change 'though15:49
abhishekkyes, just need one condition15:49
abhishekkbut only problem is I will not be around after tomorrow :(15:49
jokke_abhishekk: we'll get it sorted, please just make a comment of it into the review so we know where to look at15:50
abhishekkjokke_, I will15:50
abhishekkif we have a store release tagged today, I guess tomorrow I will rebase my patch on multihash and do that change15:51
jokke_yeah, that should work15:51
*** links has quit IRC15:51
*** links has joined #openstack-glance15:52
abhishekkjokke_, I have added a comment on the patch15:52
abhishekkjust in case if things doesn't worked as planned15:53
*** links has quit IRC15:55
*** links has joined #openstack-glance15:55
*** links has quit IRC15:58
*** links has joined #openstack-glance15:58
*** r-daneel has joined #openstack-glance16:00
jokke_cheers16:01
*** links has quit IRC16:01
*** links has joined #openstack-glance16:01
abhishekkrosmaita, jokke_, smcginnis just confirmed multihash is working with multi-backend16:03
abhishekk\o/16:03
rosmaitaexcellent!16:04
*** btully has joined #openstack-glance16:05
abhishekkI have rebased patch locally, only two files are going in conflicts, location.py and test_schemas_resource.py (nominal conflict)16:05
jokke_abhishekk: that sounds really good16:07
smcginnisNice!16:07
jokke_The light at the end of tunnel is not the headlights of oncoming train at least :D16:08
rosmaitajokke_ you certainly have a way with metaphors16:08
smcginnishaha16:08
abhishekk:D16:08
abhishekkrosmaita, do you have the list of the files which you copied under .tox directories from glance_store?16:09
*** btully has quit IRC16:09
rosmaitaabhishekk this may be what you are looking for: http://paste.openstack.org/show/726942/16:09
*** links has quit IRC16:10
abhishekkrosmaita, excellent16:10
rosmaitajokke_ smcginnis abhishekk : release note for glance_store stable/rocky: https://review.openstack.org/#/c/587513/16:13
smcginnisLooking...16:13
rosmaitalooks like we're still an hour away from merging the patch into glance_store master16:14
jokke_rosmaita: looks good ... now we just need to get that master merging and backport the fix16:14
jokke_rosmaita: go ahead and backport it so we get at least the check rolling16:14
*** r-daneel has quit IRC16:21
*** links has joined #openstack-glance16:21
rosmaitaok16:23
rosmaitahttps://review.openstack.org/#/c/587516/16:24
*** links has quit IRC16:24
*** links has joined #openstack-glance16:25
jokke_rosmaita: you forgot the -x from the cherrypick ;)16:25
rosmaitai just did it from gerrit16:25
rosmaitaguess no -x because it hasn't merged to master yet16:25
*** wxy| has quit IRC16:26
*** links has quit IRC16:27
*** links has joined #openstack-glance16:28
*** aniketh has quit IRC16:29
rosmaitahttps://review.openstack.org/#/c/587516/216:30
*** links has quit IRC16:30
*** links has joined #openstack-glance16:31
jokke_yes, that ticks all the boxes :)16:32
rosmaita:)16:32
jokke_now we just need to get zuul finishing it's work16:32
abhishekkwhich is very slow16:33
*** links has quit IRC16:33
abhishekkI will have my dinner and be back16:33
*** links has joined #openstack-glance16:33
*** itlinux has joined #openstack-glance16:35
*** links has quit IRC16:35
*** links has joined #openstack-glance16:36
*** links has quit IRC16:38
*** links has joined #openstack-glance16:39
*** r-daneel has joined #openstack-glance16:41
*** links has quit IRC16:41
*** links has joined #openstack-glance16:42
*** links has quit IRC16:44
*** links has joined #openstack-glance16:45
*** AlexeyAbashkin has quit IRC16:47
*** links has quit IRC16:48
*** links has joined #openstack-glance16:48
*** links has quit IRC16:51
*** links has joined #openstack-glance16:51
*** links has quit IRC16:54
*** links has joined #openstack-glance16:54
*** links has quit IRC16:56
*** links has joined #openstack-glance16:57
*** links has quit IRC17:00
*** r-daneel_ has joined #openstack-glance17:01
*** links has joined #openstack-glance17:01
openstackgerritMerged openstack/glance_store master: Wrap interface function for multihash correctly  https://review.openstack.org/58709817:01
*** r-daneel has quit IRC17:03
*** r-daneel_ is now known as r-daneel17:03
*** tesseract has quit IRC17:03
*** links has quit IRC17:03
*** links has joined #openstack-glance17:04
*** links has quit IRC17:06
*** links has joined #openstack-glance17:07
*** gyee has joined #openstack-glance17:07
*** links has quit IRC17:09
*** links has joined #openstack-glance17:10
*** links has quit IRC17:12
*** links has joined #openstack-glance17:12
*** links has quit IRC17:15
*** links has joined #openstack-glance17:15
*** links has quit IRC17:18
*** links has joined #openstack-glance17:18
*** links has quit IRC17:21
*** mvk_ has quit IRC17:21
*** links has joined #openstack-glance17:21
*** links has quit IRC17:24
*** links has joined #openstack-glance17:24
*** links has quit IRC17:27
*** links has joined #openstack-glance17:27
*** links has quit IRC17:30
*** links has joined #openstack-glance17:30
*** links has quit IRC17:33
*** links has joined #openstack-glance17:33
jokke_brb17:35
*** links has quit IRC17:36
rosmaitaack17:36
*** links has joined #openstack-glance17:37
*** links has quit IRC17:39
*** links has joined #openstack-glance17:40
*** links has quit IRC17:42
*** links has joined #openstack-glance17:42
*** links has quit IRC17:45
*** links has joined #openstack-glance17:46
abhishekkhas someone pushed glance_store release patch?17:49
abhishekkor it will be done once stable patch gets merged?17:49
abhishekkrosmaita, ^^17:49
rosmaitai haven't pushed it yet17:49
abhishekkack17:50
rosmaitagot it ready, i can put it up without the hash so it's all set17:50
*** priteau has quit IRC17:50
abhishekkok,17:50
rosmaitai'll do that and then it will be easy for whoever's around when everything lands to get it set17:50
abhishekkBetween I have rebased all my patches on top of multihash17:51
abhishekkjust not pushed it17:51
rosmaitawow!17:51
rosmaitagood work17:51
abhishekk:D17:51
jokke_b17:52
*** links has quit IRC17:52
*** btully has joined #openstack-glance17:54
jokke_knowing that we haven't got anything between the master and that stable yet and master merged cleanly, I'll just +2W the stable so it goes to gate as soon as the check is ready17:54
abhishekksounds good17:54
rosmaitahere's the release patch, just needs the hash: https://review.openstack.org/#/c/587570/117:55
*** btully has quit IRC17:58
*** mvk_ has joined #openstack-glance18:01
*** r-daneel has quit IRC18:02
*** r-daneel has joined #openstack-glance18:20
*** itlinux has quit IRC18:24
abhishekkjokke_, could you +a releasenotes patch as well?18:34
abhishekkhttps://review.openstack.org/#/c/587513/18:34
rosmaitai will be offline for a bit, will check back later18:37
abhishekkack18:38
*** itlinux has joined #openstack-glance19:02
*** rosmaita has quit IRC19:03
*** abhishekk has quit IRC19:44
*** abhishekk has joined #openstack-glance20:32
abhishekkjokke_, smcginnis release patch for glance store20:33
abhishekkhttps://review.openstack.org/#/c/587570/20:33
abhishekk^^^20:33
abhishekkkindly verify the hash20:33
smcginnisabhishekk: Looks correct to me.20:35
abhishekksmcginnis, thank you20:35
abhishekksmcginnis, to whom should I approach for approving release patch20:45
smcginnisabhishekk: You don't need to do anything other than submitting the patch.20:47
abhishekksmcginnis, clear :)20:47
*** edmondsw has quit IRC21:00
*** edmondsw has joined #openstack-glance21:01
*** edmondsw has quit IRC21:01
*** edmondsw has joined #openstack-glance21:01
jokke_smcginnis: actually we have bigger problem than we thought21:14
smcginnisjokke_: Should I stop that release?21:14
jokke_looking Matt's response to my ffe mail. we haven't bumped the minimum for ages and 0.22.0 will definitely not work with Rocky21:15
jokke_smcginnis: no, that is still needed21:15
smcginnisAh, raising the minimum might be an issue. Most will install with the latest, but it's at least highly inaccurate with that lower limit.21:16
jokke_yeah and we know for a fact that glance will break21:17
smcginnisjokke_: I think you can update lower-constraints.txt in glance to the newer version.21:19
abhishekki am going offline now, will be back in morning21:29
*** abhishekk has quit IRC21:30
*** btully has joined #openstack-glance21:31
*** btully has quit IRC21:35
*** itlinux has quit IRC22:09
*** edmondsw has quit IRC22:23
*** edmondsw has joined #openstack-glance22:23
*** rcernin has joined #openstack-glance22:26
*** edmondsw has quit IRC22:30
*** edmondsw has joined #openstack-glance22:31
*** ThomasWhite has quit IRC23:06
*** edmondsw has quit IRC23:11
*** btully has joined #openstack-glance23:20
*** threestrands has joined #openstack-glance23:24
*** threestrands has joined #openstack-glance23:24
*** btully has quit IRC23:24
*** _KaszpiR_ has quit IRC23:39
*** _KaszpiR_ has joined #openstack-glance23:50
*** abhishekk-mbl has quit IRC23:55
*** r-daneel has quit IRC23:58

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