Wednesday, 2019-01-30

*** imacdonn has quit IRC00:00
*** imacdonn has joined #openstack-glance00:00
*** Sravan has quit IRC00:33
*** Sravan has joined #openstack-glance00:34
*** Sravan has quit IRC00:41
*** Sravan has joined #openstack-glance00:43
*** Sravan has quit IRC00:51
*** Sravan has joined #openstack-glance00:54
*** Sravan has quit IRC01:00
*** Sravan has joined #openstack-glance01:01
*** Sravan has quit IRC01:08
*** Sravan has joined #openstack-glance01:08
*** Sravan has quit IRC01:21
*** Sravan has joined #openstack-glance01:26
*** Sravan has quit IRC01:39
*** Sravan has joined #openstack-glance01:40
*** tkajinam_ has joined #openstack-glance01:47
*** tkajinam has quit IRC01:50
*** _alastor_ has quit IRC01:52
*** cfriesen has quit IRC01:59
*** markvoelker has joined #openstack-glance02:37
*** Sravan has quit IRC02:39
*** markvoelker has quit IRC03:08
*** cfriesen has joined #openstack-glance04:41
*** Sravan has joined #openstack-glance04:42
*** abhishekk has joined #openstack-glance04:47
*** abhishekk has quit IRC05:13
*** Sravan has quit IRC05:20
*** Sravan has joined #openstack-glance05:24
*** udesale has joined #openstack-glance05:34
*** udesale has quit IRC05:46
*** abhishekk has joined #openstack-glance05:48
*** udesale has joined #openstack-glance05:48
*** udesale has quit IRC05:53
*** udesale has joined #openstack-glance05:54
*** pdeore has joined #openstack-glance06:02
*** pdeore has quit IRC06:44
*** abhishekk has quit IRC06:55
*** Luzi has joined #openstack-glance06:59
*** Sravan has quit IRC07:01
*** udesale has quit IRC07:02
*** udesale has joined #openstack-glance07:05
*** Sravan has joined #openstack-glance07:05
*** udesale has quit IRC07:06
*** udesale has joined #openstack-glance07:08
*** udesale has quit IRC07:09
*** udesale has joined #openstack-glance07:09
*** abhishekk has joined #openstack-glance07:09
*** Sravan has quit IRC07:09
*** mosulica has joined #openstack-glance07:19
*** mosulica has quit IRC07:25
*** pcaruana has joined #openstack-glance08:10
*** tkajinam_ has quit IRC08:15
*** markvoelker has joined #openstack-glance08:16
*** Sravan has joined #openstack-glance08:22
*** awalende has joined #openstack-glance08:24
*** Sravan has quit IRC08:27
*** awalende has quit IRC08:35
*** awalende has joined #openstack-glance08:35
*** awalende has quit IRC08:39
*** awalende has joined #openstack-glance08:40
*** markvoelker has quit IRC08:49
*** lpetrut has joined #openstack-glance09:17
*** Sravan has joined #openstack-glance09:24
*** Sravan has quit IRC09:29
*** cfriesen has quit IRC09:37
*** markvoelker has joined #openstack-glance09:47
*** awalende has quit IRC09:48
*** awalende has joined #openstack-glance09:55
*** markvoelker has quit IRC10:19
*** abhishekk has quit IRC10:25
*** abhishekk has joined #openstack-glance10:31
*** priteau has joined #openstack-glance10:46
*** Sravan has joined #openstack-glance10:49
*** Sravan has quit IRC10:54
*** markvoelker has joined #openstack-glance11:16
*** udesale has quit IRC11:26
*** mvkr has joined #openstack-glance11:29
*** markvoelker has quit IRC11:50
*** abhishekk has quit IRC12:30
*** markvoelker has joined #openstack-glance12:47
*** markvoelker has quit IRC13:13
*** abhishek has joined #openstack-glance13:40
*** pcaruana has quit IRC13:50
*** pcaruana has joined #openstack-glance13:57
*** jdillaman has joined #openstack-glance14:16
*** awalende has quit IRC14:39
*** awalende has joined #openstack-glance14:40
*** awalende has quit IRC14:44
*** cfriesen has joined #openstack-glance14:55
openstackgerritMerged openstack/glance master: Add test for data migration version  https://review.openstack.org/63251715:08
*** udesale has joined #openstack-glance15:08
*** Luzi has quit IRC15:18
rosmaitasmcginnis: when you have a few minutes, can you take a look at Tony and J-P's questions on https://review.openstack.org/#/c/633011/ ?  I'm not sure it matters whether 632517 (the CURRENT_VERSION test patch) is included in the tag or not--though i'm not sure i grasp all the implications15:21
rosmaitajokke_: ^^15:21
smcginnisrosmaita: Sure, I'll take a look.15:37
rosmaitathanks!15:37
*** pcaruana has quit IRC16:01
*** udesale has quit IRC16:02
*** _alastor_ has joined #openstack-glance16:03
*** _alastor_ has quit IRC16:12
openstackgerritPaul-Emile Element proposed openstack/glance master: Add flavor options to select emulated virtual tpm  https://review.openstack.org/63325616:14
*** pcaruana has joined #openstack-glance16:17
jokke_looking now16:24
jokke_rosmaita: hmm-m just when replying to that review, realized a problem in that test16:33
rosmaitaoh no16:33
jokke_So when we tag the first release from series, checking out that tag and running tests from it will always fail16:34
jokke_so our first beta will always fail testing16:35
rosmaitahmmm16:35
rosmaitai guess we really need to do the api-break thing instead, then16:35
jokke_meaning I really don't want to have that test in the tag :P16:35
rosmaitawell, it's not in the tag this time16:36
rosmaitabut will be next time16:36
jokke_and when we release the first from series in future, we probably should merge skip and unskip that test patches and tag the skip commit :P16:36
jokke_but that is just process dance16:37
rosmaitajeez ... this is getting pretty manual16:37
rosmaitawe should check with dhellmann, but maybe if we do the api-break in the same patch that changes the CURRENT_VERSION, that would be the way to go16:38
openstackgerritLucian Petrut proposed openstack/glance master: WIP: glance Windows support  https://review.openstack.org/63070516:40
openstackgerritLucian Petrut proposed openstack/glance master: WIP: Windows multiprocess wsgi  https://review.openstack.org/63070616:40
openstackgerritLucian Petrut proposed openstack/glance master: WIP: Allow glance tests to run on Windows  https://review.openstack.org/63400716:40
*** gyee has joined #openstack-glance16:42
*** abhishek has quit IRC16:42
jokke_What we could do is to move that test into it's own job. That way it will flag the gate on both cases but it will not break just running the tests when you do it locally16:45
*** pcaruana has quit IRC16:45
jokke_on both cases meaning trying to merge migration before version bump and not updating the migration after version bump16:46
rosmaitathat sounds like a good idea16:46
rosmaitai'll put up a patch for that later today16:47
rosmaitai guess we are ok for this beta release, since the change is not in the release? i mean, no reason to hold up the release patch?16:48
jokke_yep, like I said in my comment on the review, I prefer not having the test included to the release16:50
rosmaitaok, cool, so this is mainly for the next time16:50
*** lpetrut has quit IRC16:57
dhellmannrosmaita : why are your database migrations tied to the software version?17:06
rosmaitadhellmann: there is some bookkeeping that depends on a constant string that's the release name17:07
dhellmannwhat sort of bookkeeping?17:07
rosmaitai am not completely clear on that, tbh17:07
dhellmannit feels odd to be embedding names like that in the database17:08
*** priteau has quit IRC17:08
rosmaitawell, this doesn't wind up in the db, actually, but it has something to do with tracking migrations17:08
rosmaitawhich are named with the release name prefix17:09
dhellmannok. I don't really understand, but it feels like you're having to go to some amount of trouble to deal with it and that just made me question the approach in general17:10
rosmaitai agree, i need to take some time and look into why we did it this way ... it seemed to make sense at the time (ocata), and the people who worked on it left the project (osic)17:11
* dhellmann nods17:11
rosmaitayou are right that there should be a better way!17:11
jokke_dhellmann: it's basically the alembic migrations to be applied in correct order. The original design was based on the release name and alphabetical ordering of them17:27
jokke_so we're kind of stuck with that17:28
dhellmannjokke_ : alembic does all of that, though. why aren't you just telling it what order the scripts should be applied?17:28
dhellmannI mean, that's what it's *for*17:28
jokke_tbh, I have no idea. Like said I know as much as our implementation is depending on the version name as baseline there.17:35
*** Sravan has joined #openstack-glance17:54
*** Sravan has quit IRC17:55
*** pcaruana has joined #openstack-glance18:17
*** Sravan has joined #openstack-glance18:20
*** Sravan has quit IRC18:32
*** Sravan has joined #openstack-glance18:33
*** awalende has joined #openstack-glance18:40
*** awalende has quit IRC18:45
*** priteau has joined #openstack-glance18:55
*** Sravan has quit IRC18:56
*** Sravan has joined #openstack-glance18:58
*** Sravan has quit IRC19:03
*** Sravan has joined #openstack-glance19:05
*** Sravan has quit IRC19:09
*** Sravan has joined #openstack-glance19:09
*** rchurch has joined #openstack-glance19:22
*** Sravan has quit IRC19:40
*** Sravan has joined #openstack-glance19:42
*** priteau has quit IRC20:01
smcginnisjokke_, rosmaita: Would now be a good time to do that beta release?20:12
smcginnisI suppose I can always merge the follow on patch if necessary, but rather not give any surprises.20:12
*** Sravan has quit IRC20:16
*** Sravan has joined #openstack-glance20:19
*** Sravan has quit IRC20:26
*** Sravan has joined #openstack-glance20:39
*** _alastor_ has joined #openstack-glance20:44
*** Sravan has quit IRC20:47
*** Sravan has joined #openstack-glance20:47
jokke_sure20:48
rosmaitawas afk ... now would be ok20:51
rosmaitasmcginnis: ^^20:51
rosmaitai'll be around for a while20:51
smcginnisOK, I'll push the big red button.20:53
*** Sravan has quit IRC20:55
rosmaita:)20:59
*** Sravan has joined #openstack-glance21:01
*** rosmaita has quit IRC21:04
*** Sravan has quit IRC21:06
*** rosmaita has joined #openstack-glance21:07
smcginnisRelease is done.21:17
rosmaitasmcginnis: cool, i see the tag21:20
rosmaitaguess i could rebase something to see if it fails?21:20
openstackgerritBrian Rosmaita proposed openstack/glance master: DNM - test eventlet-0.22.1  https://review.openstack.org/57179721:21
rosmaitaold patch, merge conflict21:23
rosmaitadid a recheck on https://review.openstack.org/#/c/624983/3 , let's see what happens21:24
jokke_be quick, I just hit +2A on the migration patch21:35
rosmaitajokke_: good idea, the rebase i did should fail while the migration patch is going through the gate21:36
rosmaitawell, test_migration_version passes on the migration patch for py{27,35,36}21:50
*** openstackgerrit has quit IRC21:50
*** Sravan has joined #openstack-glance22:00
*** Sravan has quit IRC22:03
*** Sravan has joined #openstack-glance22:03
*** Sravan has quit IRC22:12
*** Sravan has joined #openstack-glance22:14
*** Sravan has quit IRC22:23
*** Sravan has joined #openstack-glance22:30
*** Sravan has quit IRC22:39
*** Sravan has joined #openstack-glance22:42
*** MattMan has quit IRC22:51
*** MattMan has joined #openstack-glance22:51
tonybSo ... I have a situation where I add an image with openstack image create ... --id $static_uuid $name22:58
tonybthen I delete it22:58
tonybwhen I try to re add it with the same command (including the same $static_uuid I get a conflict22:59
tonybI assume this means that the UUID is still in the database which kinda makes sense but is there a way (without manually editing the DB) to really purge deleted images?22:59
tonybFor the sake of transparency I *can* drop the static_uuid so this is eally a for my education rather then impacting anything real23:00
* tonyb probably shoudl have lead with that ;D23:00
*** tkajinam has joined #openstack-glance23:01
rosmaitatonyb: there is an ossn about that, gimme a sec to find it23:02
tonybrosmaita: Oh interesting23:02
rosmaitatonyb: https://wiki.openstack.org/wiki/OSSN/OSSN-007523:03
rosmaitai think we "fixed" this in rocky, gimme another sec23:03
tonybrosmaita: well I'm using rocky and seeing that behaviour23:03
rosmaitawell, that behavior is expected, that's why i said we "fixed" it23:04
tonybOh I see23:04
rosmaitahttps://docs.openstack.org/releasenotes/glance/rocky.html#security-issues23:04
tonybYeah I see this is the fixed behaviour23:05
rosmaitait will point you to this: https://docs.openstack.org/glance/latest/admin/db.html#purging-the-images-table23:05
rosmaitain a nutshell, what you are seeing is expected behavior, and there's a reason why you can't reuse deleted image ids23:06
tonybrosmaita: Yup.  Thanks.23:06
rosmaitanp, purge with care!23:06
* tonyb is now educated ;P #winning23:06
tonybrosmaita: In my case it's a single user development cloud so pfft23:07
rosmaitadelete away!23:07
tonybwe don't need no stinking securty ;P23:07
rosmaita:)23:07
*** Sravan has quit IRC23:08
*** Sravan has joined #openstack-glance23:12
*** Sravan has quit IRC23:15
*** Sravan has joined #openstack-glance23:39
*** Sravan has quit IRC23:44
*** Sravan has joined #openstack-glance23:50

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