Wednesday, 2017-02-01

« Tuesday, 2017-01-31 Index Thursday, 2017-02-02 »
*** ducttape_ has joined #openstack-glance00:01
*** dims_ has joined #openstack-glance00:10
*** dims has quit IRC00:11
*** jdurgin has quit IRC00:34
*** zhurong has joined #openstack-glance00:41
*** zhurong has quit IRC00:44
*** jose-phillips has quit IRC00:45
*** jdurgin has joined #openstack-glance00:48
*** ducttape_ has quit IRC00:55
*** ducttape_ has joined #openstack-glance01:06
*** TravT has quit IRC01:26
*** ducttape_ has quit IRC01:39
*** ducttape_ has joined #openstack-glance01:54
*** ducttape_ has quit IRC02:13
*** ducttape_ has joined #openstack-glance02:20
*** ducttape_ has quit IRC02:20
*** ducttape_ has joined #openstack-glance02:20
openstackgerritBrian Rosmaita proposed openstack/glance: Glare-ectomy  https://review.openstack.org/42753503:11
hemanthmlol, Glare-ectomy03:32
hemanthmgood one rosmaita03:32
*** mtanino has quit IRC03:43
*** nicolasbock has quit IRC03:46
*** flwang1 has quit IRC03:53
*** ducttape_ has quit IRC04:05
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299304:25
*** ducttape_ has joined #openstack-glance04:28
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299304:29
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract migrations for CI  https://review.openstack.org/42477404:35
openstackgerritMerged openstack/glance: Refine migration query added with CI change  https://review.openstack.org/42659904:36
*** udesale has joined #openstack-glance04:37
*** ducttape_ has quit IRC04:38
openstackgerritHemanth Makkapati proposed openstack/glance: Testing changes required with Alembic move  https://review.openstack.org/39740904:46
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract migrations for CI  https://review.openstack.org/42477404:46
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299304:46
openstackgerritHemanth Makkapati proposed openstack/glance: Port Glance Migrations to Alembic  https://review.openstack.org/38295804:46
*** ducttape_ has joined #openstack-glance04:47
openstackgerritMerged openstack/glance: Update deprecated show_multiple_locations helptext  https://review.openstack.org/42628305:13
*** ducttape_ has quit IRC05:13
*** pdeore has joined #openstack-glance05:22
*** ducttape_ has joined #openstack-glance06:27
*** rcernin has joined #openstack-glance06:29
*** ducttape_ has quit IRC06:31
*** groen692 has joined #openstack-glance06:46
*** fnordahl has quit IRC06:57
*** fnordahl has joined #openstack-glance06:57
*** mosulica has joined #openstack-glance07:03
*** tesseract has joined #openstack-glance07:18
*** pcaruana has joined #openstack-glance07:19
*** fnordahl has quit IRC07:27
*** fnordahl has joined #openstack-glance07:27
*** mosulica has quit IRC07:41
*** tshefi has joined #openstack-glance07:44
*** adisky_ has joined #openstack-glance07:54
*** ducttape_ has joined #openstack-glance07:57
*** ducttape_ has quit IRC08:02
*** bkopilov has quit IRC08:14
*** bkopilov has joined #openstack-glance08:24
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-glance09:00
*** e0ne has joined #openstack-glance09:23
*** ducttape_ has joined #openstack-glance09:28
*** ducttape_ has quit IRC09:32
*** mfedosin has joined #openstack-glance09:33
*** e0ne has quit IRC09:56
*** udesale has quit IRC10:04
*** dalgaaf has quit IRC10:48
*** dalgaaf has joined #openstack-glance10:49
*** mvk has quit IRC10:49
openstackgerritDanny Al-Gaaf proposed openstack/glance: Adds purge command to glancemanage man page  https://review.openstack.org/42763410:50
*** ducttape_ has joined #openstack-glance10:58
*** ducttape_ has quit IRC11:03
*** nicolasbock has joined #openstack-glance11:04
*** groen692 has quit IRC11:09
*** groen692 has joined #openstack-glance11:13
*** mvk has joined #openstack-glance11:19
*** groen693 has joined #openstack-glance11:20
*** groen692 has quit IRC11:21
*** groen693 is now known as groen69211:21
*** e0ne has joined #openstack-glance11:22
*** jamielennox is now known as jamielennox|away11:25
*** groen692 has quit IRC11:41
*** pdeore has quit IRC11:52
*** groen692 has joined #openstack-glance11:54
*** cdelatte has joined #openstack-glance12:11
*** catintheroof has joined #openstack-glance12:28
*** ducttape_ has joined #openstack-glance12:29
*** ducttape_ has quit IRC12:33
*** mvk has quit IRC12:47
*** mtanino has joined #openstack-glance12:56
*** ducttape_ has joined #openstack-glance12:57
*** mvk has joined #openstack-glance13:00
*** cdelatte has quit IRC13:05
*** flwang has quit IRC13:05
*** bapalm has quit IRC13:07
*** cdelatte has joined #openstack-glance13:07
*** bapalm has joined #openstack-glance13:12
*** mtanino has quit IRC13:13
*** alex_bash has quit IRC13:21
*** abashmak_ has joined #openstack-glance13:21
*** flwang has joined #openstack-glance13:22
*** bapalm has quit IRC13:24
*** hemanthm has quit IRC13:34
*** hemanthm has joined #openstack-glance13:34
openstackgerritBrian Rosmaita proposed openstack/glance: Glare-ectomy  https://review.openstack.org/42753513:35
*** ducttape_ has quit IRC13:42
openstackgerritBrian Rosmaita proposed openstack/glance_store: Replace image.stat() with image.size() call  https://review.openstack.org/37846013:54
*** ducttape_ has joined #openstack-glance14:09
sigmavirushemanthm: how's the rolling upgrades work oing?14:19
hemanthmsigmavirus: all the code is up14:19
hemanthmPatches #1 and #2, have/had a +2, need another one14:20
hemanthmThese are the patches that introduce Alembic, not rollingupgrades per se14:20
hemanthmPatches #3 and #4 are the rolling upgrade bits14:20
hemanthmThose ones never had a +2 yet14:21
hemanthmAnd, gate isn't behaving .. the neutron temptest job to be precise14:21
sigmaviruswhich one is patch #2? https://review.openstack.org/397409 ?14:21
hemanthmsigmavirus: aye14:21
sigmavirusThat is missing a +2 =P14:21
*** ducttape_ has quit IRC14:22
sigmavirusokay, just checking in14:22
sigmavirusI'd love to see those approved by tonight14:22
sigmavirusThat way we have a little time tomorrow to get them in with rechecks etc.14:22
hemanthmsigmavirus: It had one yday but we merged something else which conflicted with the 2nd patch14:22
sigmavirusgot it14:22
hemanthmit's just a rebase14:22
*** ducttape_ has joined #openstack-glance14:22
*** ducttape_ has quit IRC14:23
*** cdelatte has quit IRC14:25
*** ducttape_ has joined #openstack-glance14:28
*** ducttape_ has quit IRC14:31
*** cdelatte has joined #openstack-glance14:53
*** sigmavirus has quit IRC15:14
*** udesale has joined #openstack-glance15:14
*** sigmavirus has joined #openstack-glance15:23
*** ChanServ sets mode: +o sigmavirus15:23
*** ducttape_ has joined #openstack-glance15:34
*** ducttape_ has quit IRC15:34
*** ducttape_ has joined #openstack-glance15:35
*** BobBall has joined #openstack-glance15:38
*** tshefi has quit IRC15:49
*** rcernin has quit IRC15:51
*** mtanino has joined #openstack-glance15:54
*** ezoszed has quit IRC16:01
*** udesale has quit IRC16:03
*** _ducttape_ has joined #openstack-glance16:09
*** ducttape_ has quit IRC16:13
*** thefallenlad has joined #openstack-glance16:23
stevelle^ that +2 restored since the rebase16:38
hemanthmthanks stevelle16:39
stevellealembic port is still queued in the gate16:40
stevellelooks like it could be a while before any of the jobs even start16:41
hemanthmjust started16:42
hemanthmsomeone heard you :)16:42
stevelleoh hey, I just needed to grouse about it16:42
hemanthmhopefully the neutron temptest cooperatrs16:43
BobBallHave there been any changes to authentication or image upload in the last day?  The XenServer CI started failing all changes around 13 hours ago with a 404 reported from Glance while attempting to update the image metadata at http://192.168.33.1:9292/v2/images/0ff255ea-8082-48f4-a17f-021179fc3707/file16:53
*** tesseract has quit IRC16:56
*** e0ne has quit IRC17:08
*** burgerk has quit IRC17:33
*** _ducttape_ has quit IRC17:54
*** ducttape_ has joined #openstack-glance17:54
*** ducttape_ has quit IRC17:54
*** ducttape_ has joined #openstack-glance17:55
*** ducttape_ has quit IRC17:55
*** ducttape_ has joined #openstack-glance17:56
*** aarefiev is now known as aarefiev_afk17:57
rosmaitaBobBall: not aware of anything along those lines, can you give me a link to a failure to look at?17:59
*** thefallenlad has quit IRC18:05
*** harlowja_ has joined #openstack-glance18:05
*** harlowja has quit IRC18:06
BobBallrosmaita: http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/36/427536/1/check/dsvm-tempest-neutron-network/875a384/ - One suggestion that's just come up is whether the recent requirements bump in oslo.context needs us to set X-Project-Id (we are currently setting X-Tenant-Id)18:07
BobBallok, so that looks like a red herring...18:19
BobBallbut it does look from the g-api logs and keystone logs that the tenant g-api is using was deleted shortly before use...18:21
*** pcaruana has quit IRC18:28
rosmaitaBobBall: weird. looking now18:30
*** dharinic is now known as dharinic|lunch18:50
*** mvk has quit IRC18:57
openstackgerritMerged openstack/glance: Port Glance Migrations to Alembic  https://review.openstack.org/38295819:00
*** ducttape_ has quit IRC19:02
*** ducttape_ has joined #openstack-glance19:03
*** adisky_ has quit IRC19:09
*** harlowja_ has quit IRC19:19
*** e0ne has joined #openstack-glance19:30
*** mvk has joined #openstack-glance19:34
*** dharinic|lunch is now known as dharinic19:37
*** mtanino has quit IRC19:51
*** e0ne has quit IRC19:55
*** mtanino has joined #openstack-glance19:59
*** e0ne has joined #openstack-glance19:59
rosmaitaBobBall: i dont' see anything helpful in those logs ... sorry20:01
*** mtanino has quit IRC20:09
*** mtanino has joined #openstack-glance20:12
*** e0ne has quit IRC20:17
*** e0ne has joined #openstack-glance20:21
*** e0ne has quit IRC20:28
*** jamielennox|away is now known as jamielennox20:30
*** e0ne has joined #openstack-glance20:32
*** groen692 has quit IRC20:35
*** e0ne has quit IRC20:37
*** harlowja has joined #openstack-glance20:39
*** e0ne has joined #openstack-glance20:41
*** e0ne has quit IRC20:47
*** e0ne has joined #openstack-glance20:51
*** e0ne has quit IRC20:55
*** _ducttape_ has joined #openstack-glance20:58
*** e0ne has joined #openstack-glance21:00
*** ducttape_ has quit IRC21:02
*** jose-phillips has joined #openstack-glance21:05
*** _ducttape_ has quit IRC21:37
*** catintheroof has quit IRC21:37
*** ducttape_ has joined #openstack-glance21:37
*** catintheroof has joined #openstack-glance21:37
nikhilrosmaita: stevelle hemanthm : commented back, I can quick review the next PS. thanks!21:39
stevellegood small changes, should be a fast revisit21:41
hemanthmok, working  on a new PS21:41
hemanthmnikhil: the cleanup part is already mentioned here https://review.openstack.org/#/c/397409/16/tox.ini21:45
hemanthmDo you want me to mention it again?21:45
hemanthmOr am I misunderstanding you?21:45
nikhilhemanthm: commented21:49
*** e0ne has quit IRC21:53
*** catintheroof has quit IRC21:57
nikhilrosmaita: flwang : fyi http://lists.openstack.org/pipermail/openstack-operators/2017-January/012543.html21:58
*** catintheroof has joined #openstack-glance21:58
rosmaitanikhil: ty, flwang replies later in that thread22:03
nikhilrosmaita: oh cool22:04
openstackgerritDanny Al-Gaaf proposed openstack/glance: Adds purge command to glancemanage man page  https://review.openstack.org/42763422:06
*** ducttape_ has quit IRC22:09
openstackgerritHemanth Makkapati proposed openstack/glance: Refactor tests to use Alembic to run migrations  https://review.openstack.org/39740922:09
*** ducttape_ has joined #openstack-glance22:10
hemanthmrosmaita, nikhil, stevelle: posted another PS ^22:12
rosmaitahemanthm: ty, will look in a minute22:12
stevellehemanthm: that does obligate at least a rebase of the rest of the set22:15
stevelle?22:15
*** ducttape_ has quit IRC22:15
nikhilhemanthm: I thought you'd update the commit too!22:15
*** adrian_otto has joined #openstack-glance22:15
nikhilCM22:15
hemanthmYeah, I was expecting a merge-conflict to show up on the 3rd one. But, it didn't22:15
hemanthmnikhil: I did22:16
*** mfedosin has quit IRC22:16
hemanthmUnless it's not the update you were expecting22:16
nikhilhemanthm: meh, it's okay.22:17
hemanthmstevelle: Want me to push up a new PS for the 3rd patch?22:17
nikhilhemanthm: I got my answer. hopefully the next person in line to read the CM would.22:18
stevellehemanthm: up to you but I think you will need to rebase at least so same risk WRT gate instability and restoring the +1 Verified22:18
stevellethe polish on the docs lines would be nice22:18
dharinichemanthm: Just a quick reminder that if u r updating the third patch, you edit the test too.22:19
hemanthmdharinic: sure, thanks for the reminder22:19
hemanthmguess I'll post a new PS22:19
dharinictiny changes, should be quick. :)22:19
hemanthmit's not the changes I'm worried about ..22:20
dharinicahh.22:20
*** adrian_otto has left #openstack-glance22:20
nikhilhemanthm: I have nits on the next22:21
hemanthmsure, I'm working a new PS anyway22:23
hemanthmI'll hold until you post your comments nikhil22:24
*** catintheroof has quit IRC22:35
stevellerosmaita nikhil dharinic any reason not to start merging patch 2 (Refactor tests to use Alembic to run migrations)?22:35
rosmaitastevelle: just saw your quick +/-A on that22:36
stevellerealized I should go ahead and check22:36
rosmaitai'll do it, might as well go through the gate only once22:36
stevellein your hands22:36
stevellegonna grab some food while setting up some test boxes for 3 and 422:37
rosmaitagreat22:37
stevelleFWIW I am am not seeing a reason we can't get all of them in today22:37
stevellefairly confident there is no race condition on a rolling upgrade22:38
dharinicno rosmaita. no specific reason. I thought you would be +w'ing EOD22:38
dharinicyes, agree with stevelle22:38
rosmaitacool22:39
rosmaitai'll put up a release notes patch for this22:40
dharinicawesome22:40
*** ducttape_ has joined #openstack-glance22:47
*** ducttape_ has quit IRC22:49
*** _ducttape_ has joined #openstack-glance22:49
nikhilhemanthm: rosmaita feel free to ping for +As22:52
*** haplo37 has quit IRC22:52
*** _ducttape_ has quit IRC22:53
hemanthmrosmaita, nikhil: replied to your question on #322:57
*** haplo37 has joined #openstack-glance23:05
rosmaitahemanthm: ty23:14
hemanthmrosmaita, nikhil: I was able to get rid of LATEST_EXPAND and LATEST_CONTRACT23:20
hemanthmeasy enough change23:20
hemanthmThe other ones need a lit bit more thought23:20
rosmaitaok, i think we've agreed it's acceptable technical debt for now23:20
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299323:21
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299323:21
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299323:22
openstackgerritHemanth Makkapati proposed openstack/glance: Add expand/migrate/contract commands to glance-manage CLI  https://review.openstack.org/39299323:23
hemanthmstevelle, dharinic, rosmaita, nikhil: this is the change introduced https://review.openstack.org/#/c/392993/28..3223:24
dharinicThank hemanthm. Looking23:25
hemanthmstevelle, rosmaita: do you have any feedback on the 4th patchset. I may push another PS23:37
hemanthmsorry, 4th patch I mean23:37
rosmaitahemanthm: nothing ATM23:38
hemanthmack23:38
openstackgerritBrian Rosmaita proposed openstack/glance: Alembic migrations/rolling upgrades release note  https://review.openstack.org/42797123:43
hemanthmrosmaita: updating http://docs.openstack.org/developer/glance/db.html would be a separate patch I'm guessing?23:46
hemanthmAnd that can go later as well?23:46
rosmaitawell that's db.rst in your 3rd patch23:47
dharinicwe need to keep track of all the follow ups with respect to rolling upgrades23:47
hemanthmah, right23:47
rosmaitabut we can follow up with some improvements, i figure23:47
hemanthmyeah23:47
rosmaitathe http:// is always a link to master23:48
hemanthmWe should probably talk about the new database versions23:48
hemanthmcool23:48
hemanthmthe db versions are not numbers any more23:48
hemanthmit's a string of the form <release>_<seq number>23:49
hemanthm<release>_<expand/contract><seq number>23:49
hemanthmdharinic: yeah, I'm keep track of the followups23:51
*** ducttape_ has joined #openstack-glance23:51
hemanthm*keeping23:51
dharinichemanthm: Great. We can work on all of those in pike,23:57
« Tuesday, 2017-01-31 Index Thursday, 2017-02-02 »

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