Monday, 2017-09-18

openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Serve keys from canonical project name  https://review.openstack.org/50480700:03
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add support for result data in child jobs  https://review.openstack.org/50480800:03
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: add abstract job attribute  https://review.openstack.org/50480900:03
*** mat128 has joined #openstack-infra00:05
dmsimardEven a recheck doesn't trigger anything, I suspect there's probably some errors somewhere in logs.. maybe it's the depends-on(s) ?00:06
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478800:06
dmsimardYup, as soon as I removed the depends-on, it queued00:06
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add multinode integration jobs and integration tests for known_hosts  https://review.openstack.org/50478700:07
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478800:07
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge  https://review.openstack.org/50478900:08
*** mat128 has quit IRC00:08
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge  https://review.openstack.org/50478900:08
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478800:09
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge  https://review.openstack.org/50478900:09
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add multinode integration jobs and integration tests for known_hosts  https://review.openstack.org/50478700:09
*** thorst has joined #openstack-infra00:13
*** thorst has quit IRC00:13
dmsimardI basically had a tree of 3 patches with a depends-on on the same patch http://paste.openstack.org/raw/621297/00:16
dmsimardTo make it work I had to remove the depends-on on the two 'child' patches00:16
dmsimardSounds like Zuul could at least tell me how horribly wrong I was00:16
*** sshnaidm has joined #openstack-infra00:43
*** LindaWang has joined #openstack-infra00:55
fungidmsimard: any chance there was a circular dep? if so, that's basically feature parity with zuul v2 for now00:59
*** yamahata has joined #openstack-infra01:05
*** thorst has joined #openstack-infra01:08
*** thorst has quit IRC01:08
*** cody-somerville has quit IRC01:16
*** cshastri has joined #openstack-infra01:23
*** cuongnv has joined #openstack-infra01:33
*** liujiong has joined #openstack-infra01:36
*** bobh has joined #openstack-infra01:51
*** mwarad has joined #openstack-infra01:54
*** mat128 has joined #openstack-infra01:54
*** dave-mccowan has joined #openstack-infra01:55
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Actually sort mount-point list  https://review.openstack.org/50481901:55
*** dave-mcc_ has joined #openstack-infra01:58
*** mat128 has quit IRC02:00
*** dave-mccowan has quit IRC02:00
*** Daisy_ has joined #openstack-infra02:08
*** Daisy_ has quit IRC02:08
*** thorst has joined #openstack-infra02:09
*** gouthamr has quit IRC02:10
*** jmorgan1_ has quit IRC02:11
*** jmorgan1 has joined #openstack-infra02:11
*** thorst has quit IRC02:14
*** hongbin has joined #openstack-infra02:14
*** cody-somerville has joined #openstack-infra02:20
openstackgerritIan Wienand proposed openstack/diskimage-builder master: [WIP] Use olso to parse size strings  https://review.openstack.org/50357402:22
*** cody-somerville has quit IRC02:27
*** gongysh has joined #openstack-infra02:35
*** yamahata has quit IRC02:37
*** larainema has quit IRC02:38
*** dave-mcc_ has quit IRC02:39
*** cody-somerville has joined #openstack-infra02:51
*** ramishra has joined #openstack-infra02:52
*** cody-somerville has quit IRC02:57
*** yamahata has joined #openstack-infra02:59
*** gouthamr has joined #openstack-infra03:01
*** tiswanso has joined #openstack-infra03:01
*** tiswanso has quit IRC03:01
*** tiswanso has joined #openstack-infra03:02
*** jesusaur has quit IRC03:08
*** baoli has quit IRC03:08
*** thorst has joined #openstack-infra03:10
*** thorst has quit IRC03:14
*** gouthamr has quit IRC03:17
*** cody-somerville has joined #openstack-infra03:17
*** cody-somerville has joined #openstack-infra03:17
*** bobh has quit IRC03:19
*** bobh has joined #openstack-infra03:22
openstackgerritzhangyangyang proposed openstack-infra/log_processor master: Remove py26 support  https://review.openstack.org/50483103:22
*** hongbin has quit IRC03:25
*** hongbin has joined #openstack-infra03:25
*** bobh has quit IRC03:26
*** jesusaur has joined #openstack-infra03:27
*** tiswanso has quit IRC03:33
openstackgerritzhangyangyang proposed openstack-infra/nodepool master: Remove py26 support  https://review.openstack.org/50483603:37
openstackgerritClark Boylan proposed openstack-infra/system-config master: Upgrade review.openstack.org to Gerrit 2.13  https://review.openstack.org/50483803:41
clarkbfungi: I think getting ^ proposed was the last thing I needed to do before running through the upgrade steps tomorrow. Note that the new db schema uses the existing trove user and passwd for access if you end up wanting to create that before I wake03:44
clarkbI will see everyone in the morning03:44
*** hongbin has quit IRC03:45
*** ihrachys has joined #openstack-infra03:45
*** yamahata has quit IRC03:53
*** yamahata has joined #openstack-infra03:55
*** gildub_ has joined #openstack-infra03:55
*** ihrachys has quit IRC03:56
openstackgerritzhangyangyang proposed openstack-infra/os-loganalyze master: Remove py26 support  https://review.openstack.org/50484203:57
*** mwarad has quit IRC03:58
openstackgerritzhangyangyang proposed openstack-dev/pbr master: Remove py26 support  https://review.openstack.org/50484304:01
SpamapSanybody around who can tell me what infra's zuulv3 has for [webapp]status_url ?04:10
* SpamapS has sort of given up trying to find how @status_url gets set :-P04:10
*** thorst has joined #openstack-infra04:11
*** psachin has joined #openstack-infra04:15
*** thorst has quit IRC04:15
*** gongysh has quit IRC04:16
*** yamahata has quit IRC04:17
*** ykarel has joined #openstack-infra04:18
*** ykarel has quit IRC04:23
*** Sree has joined #openstack-infra04:29
*** gongysh has joined #openstack-infra04:34
*** yamahata has joined #openstack-infra04:37
*** Sree has quit IRC04:39
*** Sree has joined #openstack-infra04:40
*** Sree has quit IRC04:44
*** links has joined #openstack-infra04:46
openstackgerritAnkit Kumar proposed openstack-infra/project-config master: Add experimental py35 gate for ironic-inspector  https://review.openstack.org/46269504:53
*** gildub_ has quit IRC04:59
*** dhajare has joined #openstack-infra05:03
*** larainema has joined #openstack-infra05:06
*** thorst has joined #openstack-infra05:11
*** cody-somerville has quit IRC05:13
*** thorst has quit IRC05:16
*** jaosorior has joined #openstack-infra05:22
*** jbadiapa has joined #openstack-infra05:23
*** bobh has joined #openstack-infra05:23
*** Sree has joined #openstack-infra05:25
*** bobh has quit IRC05:27
*** david-lyle has quit IRC05:30
*** Dinesh_Bhor has joined #openstack-infra05:33
openstackgerritJuan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: Only inject cloud-init in CentOS 7.3  https://review.openstack.org/50485005:36
*** gongysh has quit IRC05:39
*** udesale has joined #openstack-infra05:39
*** jtomasek has joined #openstack-infra05:48
openstackgerritMatthew Thode proposed openstack/diskimage-builder master: Update Gentoo element for element changes  https://review.openstack.org/50384405:50
*** khappone has joined #openstack-infra05:53
*** dhajare has quit IRC05:57
*** udesale has quit IRC06:01
*** AJaeger has quit IRC06:03
*** martinkopec has joined #openstack-infra06:06
*** AJaeger has joined #openstack-infra06:07
openstackgerritJuan Antonio Osorio Robles proposed openstack-infra/tripleo-ci master: Only inject cloud-init in CentOS 7.3  https://review.openstack.org/50485006:09
*** yolanda has joined #openstack-infra06:11
*** thorst has joined #openstack-infra06:12
*** rwsu has joined #openstack-infra06:14
*** sshnaidm has quit IRC06:17
*** thorst has quit IRC06:17
*** bhavik1 has joined #openstack-infra06:19
*** dhajare has joined #openstack-infra06:23
*** bobh has joined #openstack-infra06:24
*** florianf has joined #openstack-infra06:25
*** sshnaidm has joined #openstack-infra06:25
*** jtomasek has quit IRC06:26
*** makowals has joined #openstack-infra06:27
*** bobh has quit IRC06:29
openstackgerritMatthew Thode proposed openstack/diskimage-builder master: Update Gentoo element for element changes  https://review.openstack.org/50384406:30
*** jtomasek has joined #openstack-infra06:37
*** Sree has quit IRC06:44
*** aeng has quit IRC06:45
*** hashar has joined #openstack-infra06:46
*** shardy_afk is now known as shardy06:52
*** Sree has joined #openstack-infra06:53
*** Sree has quit IRC06:58
*** threestrands has quit IRC06:58
*** ykarel has joined #openstack-infra06:59
*** eumel8 has joined #openstack-infra07:02
*** ccamacho has joined #openstack-infra07:03
*** pgadiya has joined #openstack-infra07:04
*** jpena|off is now known as jpena07:05
*** thorst has joined #openstack-infra07:13
*** rcernin has joined #openstack-infra07:13
*** sshnaidm has quit IRC07:15
*** Sree has joined #openstack-infra07:16
*** Sree has quit IRC07:16
*** Sree has joined #openstack-infra07:17
*** bhavik1 has quit IRC07:17
*** thorst has quit IRC07:18
*** alexchadin has joined #openstack-infra07:19
*** pcaruana has joined #openstack-infra07:21
*** Sree has quit IRC07:21
*** tesseract has joined #openstack-infra07:22
apuimedoyolanda: could you please review https://review.openstack.org/#/c/504374/ ?07:22
openstackgerritThomas Bechtold proposed openstack-infra/irc-meetings master: Switch chair for RPM packaging  https://review.openstack.org/50486507:26
*** danpawlik has joined #openstack-infra07:27
*** sshnaidm has joined #openstack-infra07:27
*** shardy is now known as shardy_afk07:28
*** szaher has joined #openstack-infra07:28
fricklerinfra-root: is apps.o.o still a thing? no A record, only AAAA, and seems not to respond there either07:34
*** sshnaidm has quit IRC07:36
AJaegerfrickler: I think that was taken out of commission, let's check...07:39
fricklerAJaeger: someone forgetting to remove the AAAA would seem rather plausible07:41
AJaegerfrickler: https://wiki.openstack.org/wiki/Infrastructure_Status - search for apps07:42
AJaegerfrickler: let's tell fungi about it ^07:43
fricklerAJaeger: agree, interesting page, don't think I've seen that before *bookmark*07:44
AJaegerfrickler: http://lists.openstack.org/pipermail/openstack-operators/2017-July/013965.html07:44
AJaegerfrickler: that page records all the #status log/alert etc messages we send07:45
*** ykarel is now known as ykarel|lunch07:45
fricklerAJaeger: cool, I've always wondered where those might get logged :)07:46
AJaegerfrickler: and tweeted - see also the box at  https://docs.openstack.org/infra/manual/07:47
*** gongysh has joined #openstack-infra07:48
fricklerAJaeger: a box titled "Important" on top of a page, who would ever read such a thing? ;)07:48
*** ralonsoh has joined #openstack-infra07:52
*** shardy_afk is now known as shardy07:53
openstackgerritRUIJIE YUAN proposed openstack-infra/project-config master: Add new repo for Senlin tempest plugin  https://review.openstack.org/50487007:54
*** dizquierdo has joined #openstack-infra07:54
AJaegerfrickler: should we use a smaller font? White on white? ;)07:55
openstackgerritAparna proposed openstack/diskimage-builder master: Change to install a package in 'proliant-tools'  https://review.openstack.org/50156808:01
*** LindaWang has quit IRC08:05
*** shardy is now known as shardy_afk08:05
*** LindaWang has joined #openstack-infra08:06
*** dizquierdo has quit IRC08:07
*** LindaWang has quit IRC08:07
*** LindaWang has joined #openstack-infra08:09
*** LindaWang has quit IRC08:09
*** LindaWang has joined #openstack-infra08:11
*** egonzalez has joined #openstack-infra08:12
*** thorst has joined #openstack-infra08:14
*** akscram1 has quit IRC08:15
*** tinwood has quit IRC08:15
*** electrofelix has joined #openstack-infra08:15
*** akscram1 has joined #openstack-infra08:16
*** tinwood has joined #openstack-infra08:18
*** thorst has quit IRC08:19
*** jpich has joined #openstack-infra08:19
*** bobh has joined #openstack-infra08:26
*** slaweq has joined #openstack-infra08:27
*** bobh has quit IRC08:30
*** ykarel|lunch is now known as ykarel08:35
*** pgadiya has quit IRC08:35
*** shardy_afk is now known as shardy08:40
*** mwarad has joined #openstack-infra08:42
*** efoley has joined #openstack-infra08:42
odyssey4meWe have a patch that seems to not want to enter into the gate queue for some reason - I wonder if someone could take a peek to figure out why: https://review.openstack.org/50417008:43
AJaegerodyssey4me: does it use depends-on? Or is on top of one that uses depends-on?08:44
odyssey4meAJaeger nope - just a single patch, ported back.08:44
AJaegerodyssey4me: toggle the +A - or add another +A. WE had some downtime over the weekend and the even might not have been recorded properly.08:45
odyssey4meAJaeger I rechecked it this morning.08:45
AJaegerodyssey4me: recheck will not help in those cases, Zuul reacts on *Changes*08:45
AJaegerodyssey4me: Just add another +A08:45
odyssey4meAJaeger you also rechecked it after the CI failure ;)08:45
odyssey4meok, added another workflow08:46
AJaegerodyssey4me: I did - in the hope that this would work out ;)08:46
odyssey4meok, that did it08:46
odyssey4methanks!08:46
AJaegernext time: As a core, a +A will be quicker than a recheck ;)08:46
odyssey4meunderstood08:47
AJaegerglad to see the change in the gate now...08:48
*** dtantsur|afk is now known as dtantsur08:56
*** _ari_ has quit IRC09:07
*** weshay has quit IRC09:07
*** _ari_ has joined #openstack-infra09:07
*** ssbarnea has quit IRC09:08
*** weshay has joined #openstack-infra09:10
*** slaweq has quit IRC09:10
*** ssbarnea has joined #openstack-infra09:10
*** slaweq has joined #openstack-infra09:11
*** Douhet has quit IRC09:11
*** thorst has joined #openstack-infra09:15
*** thorst has quit IRC09:19
*** Sree has joined #openstack-infra09:24
*** bobh has joined #openstack-infra09:26
*** Sree has quit IRC09:29
*** bobh has quit IRC09:31
*** slaweq has quit IRC09:44
*** tosky has joined #openstack-infra09:46
*** slaweq has joined #openstack-infra09:47
*** otherwiseguy has quit IRC09:48
*** pabelanger has quit IRC09:48
*** dtantsur is now known as dtantsur|afk09:49
*** dgonzalez has left #openstack-infra09:49
*** LindaWang has quit IRC09:49
*** _ari_ has quit IRC09:49
*** ssbarnea has quit IRC09:50
*** ssbarnea has joined #openstack-infra09:50
*** _ari_ has joined #openstack-infra09:50
*** otherwiseguy has joined #openstack-infra09:51
*** pabelanger has joined #openstack-infra09:51
*** ociuhandu has quit IRC09:52
*** ociuhandu has joined #openstack-infra09:54
*** mwarad has quit IRC09:57
openstackgerritChandan Kumar proposed openstack-infra/project-config master: Use zaqar-tempest-plugin for zaqar jobs  https://review.openstack.org/50490109:59
*** cuongnv has quit IRC10:03
*** rcarrillocruz has quit IRC10:04
*** cshastri has quit IRC10:05
*** Hal has joined #openstack-infra10:09
*** Hal is now known as Guest8221010:09
*** rcarrillocruz has joined #openstack-infra10:10
*** efoley is now known as elfoley10:13
*** elfoley is now known as efoley10:14
*** thorst has joined #openstack-infra10:16
*** alexchadin has quit IRC10:18
*** alexchadin has joined #openstack-infra10:19
*** alexchadin has quit IRC10:19
*** alexchadin has joined #openstack-infra10:19
*** alexchadin has quit IRC10:20
*** alexchadin has joined #openstack-infra10:20
*** thorst has quit IRC10:20
*** alexchadin has quit IRC10:20
*** alexchadin has joined #openstack-infra10:21
*** nicolasbock has joined #openstack-infra10:22
*** tosky is now known as tosky_10:23
*** tosky_ is now known as tosky10:24
*** alexchadin has quit IRC10:25
*** bobh has joined #openstack-infra10:27
*** egonzalez has quit IRC10:29
*** bobh has quit IRC10:32
*** wolverineav has joined #openstack-infra10:33
*** wolverineav has quit IRC10:33
*** wolverineav has joined #openstack-infra10:34
*** alex_xu has quit IRC10:35
*** alex_xu has joined #openstack-infra10:36
*** Qiming has quit IRC10:40
*** amotoki has quit IRC10:41
*** Qiming has joined #openstack-infra10:41
*** amotoki has joined #openstack-infra10:43
*** dizquierdo has joined #openstack-infra10:43
*** rhallisey has joined #openstack-infra10:43
*** ociuhandu has quit IRC10:44
*** egonzalez has joined #openstack-infra10:45
*** e0ne has joined #openstack-infra10:48
*** Douhet has joined #openstack-infra10:49
*** jkilpatr has quit IRC10:49
*** mat128 has joined #openstack-infra10:53
*** ffledgling has joined #openstack-infra10:54
*** psachin has quit IRC10:54
ffledglingHello, is this right place for questions about jenkins-job-builder?10:54
*** links has quit IRC10:55
*** e0ne has quit IRC11:04
*** liujiong has quit IRC11:06
*** e0ne has joined #openstack-infra11:08
*** LindaWang has joined #openstack-infra11:08
*** ociuhandu has joined #openstack-infra11:08
*** jkilpatr has joined #openstack-infra11:09
*** sdague_ has joined #openstack-infra11:11
*** gongysh has quit IRC11:15
*** gongysh has joined #openstack-infra11:17
*** alexchadin has joined #openstack-infra11:22
*** alexchadin has quit IRC11:24
*** egonzalez has quit IRC11:26
*** bobh has joined #openstack-infra11:28
*** sdague has joined #openstack-infra11:31
*** mat128 has quit IRC11:32
*** alexchadin has joined #openstack-infra11:32
*** mat128 has joined #openstack-infra11:33
*** bobh has quit IRC11:33
AJaegerffledgling: yes, in general - but might be wrong time. Just ask and if nobody answers, ask again at US morning time. Please stay in the channel, people might read your question and answer later11:36
ffledglingAJaeger: got it11:37
ffledglingI was wondering if there's a standard way to convert handwritten XML Jenkins configs to jjb job configs (or at least get a rough cut of some sort to get started with). I see a few OSS projects, that I'm going to try, but I was wondering if there's a standard/accepting way of doing it11:38
AJaegerffledgling: no idea - hope others can help...11:39
*** ldnunes has joined #openstack-infra11:42
*** jamesdenton has joined #openstack-infra11:46
*** mat128 has quit IRC11:46
*** dminca has joined #openstack-infra11:48
openstackgerritJustin Kilpatrick proposed openstack-infra/project-config master: Add devstack job for Cyborg  https://review.openstack.org/50414111:48
dmincahi guys11:48
*** sambetts_ is now known as sambetts11:53
*** arxcruz has quit IRC12:01
fungiAJaeger: frickler: yes, i thought i had deleted all the dns records but i must have missed the aaaa. the service was decommissioned months ago after the community agreed it was better to help support other application registries emergent in the ecosystem rather than create our own to compete with them12:02
fungii'll make a note to clean up that rr in a bit12:02
*** thorst has joined #openstack-infra12:03
fungi#status notice Gerrit will be offline for the upgrade to 2.13 starting at 15:00 UTC (in roughly 3 hours) and is expected to probably be down/unusable for 8+ hours while an offline reindex is performed: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html12:04
openstackstatusfungi: sending notice12:04
*** mattmceuen has quit IRC12:05
-openstackstatus- NOTICE: Gerrit will be offline for the upgrade to 2.13 starting at 15:00 UTC (in roughly 3 hours) and is expected to probably be down/unusable for 8+ hours while an offline reindex is performed: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html12:05
*** sdague has quit IRC12:06
*** dave-mccowan has joined #openstack-infra12:06
*** trown|ptg is now known as trown12:07
*** egonzalez has joined #openstack-infra12:07
*** jpena is now known as jpena|lunch12:07
*** dave-mcc_ has joined #openstack-infra12:08
fungiffledgling: AJaeger: there's also an #openstack-jjb channel specifically for jenkins job builder discussions, which seems to be relatively active (i lurk in there but don't tend to get involved unless someone asks me infra-specific questions)12:11
*** dave-mccowan has quit IRC12:11
AJaegerfungi: thanks, wasn't aware of that one...12:14
*** jcoufal has joined #openstack-infra12:16
*** rlandy has joined #openstack-infra12:17
*** tiswanso has joined #openstack-infra12:17
*** sdague_ is now known as sdague12:18
*** sshnaidm has joined #openstack-infra12:20
smcginnisSo the upgrade outage is expected to be pretty much the whole US work day?12:21
fungismcginnis: yep12:21
smcginnisfungi: Guess this will be a good day to get my basement cloud up to pike then. ;)12:22
fungiexpectation was monday after the ptg would be slow, but not so slow that people won't be able to spot subtle breakage we miss when it comes back online12:22
smcginnisYeah, I would bet there are a few folks still travelling. At least a lot of the non-US folks.12:23
fungiwe may be lucky and finish faster, but previous reindexes have taken quite a long time to complete12:23
smcginnisBest not to be too optimistic for upgrades.12:24
fungialso, one way to look at it is that the infra team is encouraging geographic diversity by not taking the service offline during EMEA/APAC workdays ;)12:26
smcginnisHah12:27
AJaegerfungi, that's an interesting twist! ;)12:28
*** tpsilva has joined #openstack-infra12:29
*** arxcruz has joined #openstack-infra12:35
*** markmcd has quit IRC12:37
openstackgerritJustin Kilpatrick proposed openstack-infra/project-config master: Add devstack job for Cyborg  https://review.openstack.org/50414112:41
*** markmcd has joined #openstack-infra12:46
*** dminca has quit IRC12:48
ffledglingfungi: thanks, let me re-ask there...12:49
openstackgerritMerged openstack-infra/project-config master: Remove #openstack-neutron-ovn.  https://review.openstack.org/50172212:54
*** tiswanso has quit IRC12:54
*** markmcd has quit IRC12:54
*** tiswanso has joined #openstack-infra12:55
*** Goneri has joined #openstack-infra12:56
*** markmcd has joined #openstack-infra12:57
*** bobh has joined #openstack-infra12:59
*** dave-mcc_ has quit IRC12:59
*** dprince has joined #openstack-infra13:00
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Only uninstall rdo-release if it was installed in the first place  https://review.openstack.org/50493613:02
*** LindaWang has quit IRC13:07
*** dminca has joined #openstack-infra13:09
*** Sree has joined #openstack-infra13:09
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Set repo setup release in playbook  https://review.openstack.org/50493913:10
*** hrybacki|ptg is now known as hrybacki13:10
*** dminca has quit IRC13:13
*** xyang1 has joined #openstack-infra13:18
*** kgiusti has joined #openstack-infra13:18
*** jpena|lunch is now known as jpena13:18
*** esberglu has joined #openstack-infra13:19
openstackgerritMerged openstack-infra/project-config master: Add new repo for Neutron tempest plugin  https://review.openstack.org/50222413:19
*** baoli has joined #openstack-infra13:19
*** sshnaidm has quit IRC13:19
openstackgerritMerged openstack-infra/project-config master: Publish Freezer docs  https://review.openstack.org/50432913:23
openstackgerritMerged openstack-infra/project-config master: Really fix kuryr-kubernetes Octavia job  https://review.openstack.org/50437413:25
openstackgerritMerged openstack-infra/project-config master: networking-midonet: Move legacy job from experimental to check  https://review.openstack.org/50424013:26
openstackgerritMerged openstack-infra/project-config master: Update gerritbot branch alerts in #openstack-swift  https://review.openstack.org/50412713:26
openstackgerritMerged openstack-infra/project-config master: Remove pypy jobs from python-barbicanclient  https://review.openstack.org/50423013:26
*** alexchadin has quit IRC13:27
*** alexchadin has joined #openstack-infra13:28
*** ihrachys has joined #openstack-infra13:29
*** dminca has joined #openstack-infra13:29
*** makowals has quit IRC13:30
*** martinkopec has quit IRC13:30
*** dave-mccowan has joined #openstack-infra13:31
fungihuh, rackspace says elasticsearch02 was the target of a denial of service attack yesterday... i guess they're referring to this spike in packet rate between 16:50 and 17:10 utc yesterday: http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=1665&rra_id=all13:32
*** makowals has joined #openstack-infra13:32
*** dminca has quit IRC13:33
fungi13.45kpps average at peak, though that's just one 5-minute sample so no clue how much higher it really went13:33
openstackgerritMerged openstack-infra/project-config master: Remove ironic nv job from tempest check pipeline  https://review.openstack.org/50406613:34
openstackgerritMerged openstack-infra/project-config master: Add old python-barbicanclient gate logic back for the stable branches  https://review.openstack.org/50411813:34
funginot seeing any similar spikes (inbound nor outbound) on any of the other es cluster members not the kibana server, so looks likely to be a random target13:34
openstackgerritMerged openstack-infra/project-config master: Update gate job to stop cloning old gnocchi location.  https://review.openstack.org/50408313:34
fungiand not a very useful target at that13:34
openstackgerritMerged openstack-infra/project-config master: Add devstack and tempest to the real section  https://review.openstack.org/50415813:34
openstackgerritMerged openstack-infra/project-config master: Propose nova-lxd job using OpenStack-Ansible  https://review.openstack.org/48218513:34
openstackgerritMerged openstack-infra/project-config master: Add test job for zun with split tempest plugin  https://review.openstack.org/50382513:34
openstackgerritMerged openstack-infra/project-config master: neutron-vpnaas: Move the coverage job from post to check  https://review.openstack.org/50317613:34
openstackgerritMerged openstack-infra/project-config master: Barbican: Make dogtag job voting and add it to gate  https://review.openstack.org/50384613:34
openstackgerritMerged openstack-infra/project-config master: Add new project networking-lagopus  https://review.openstack.org/50173013:34
*** dminca has joined #openstack-infra13:34
fungi#status notice Gerrit will be offline for the upgrade to 2.13 starting at 15:00 UTC (in roughly 1.5 hours) and is expected to probably be down/unusable for 8+ hours while an offline reindex is performed: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html13:35
openstackstatusfungi: sending notice13:35
-openstackstatus- NOTICE: Gerrit will be offline for the upgrade to 2.13 starting at 15:00 UTC (in roughly 1.5 hours) and is expected to probably be down/unusable for 8+ hours while an offline reindex is performed: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html13:36
AJaegerfungi, do you want to send an updated email with new subject like the IRC messages?13:37
fungiAJaeger: clarkb sent a reminder on friday. do you think we need more?13:38
dmsimardfungi: gl with the upgrade13:38
fungithanks dmsimard!13:38
fungi#status log updatepuppetmaster cron job on puppetmaster.openstack.org has been disabled in preparation for the upcoming gerrit upgrade maintenance13:38
openstackstatusfungi: finished logging13:38
*** alexchadin has quit IRC13:39
dmsimardfungi: re: dmsimard: any chance there was a circular dep? if so, that's basically feature parity with zuul v2 for now13:39
AJaegerfungi: we should sent one today IMHO. Shall I?13:39
*** mriedem has joined #openstack-infra13:39
fungiAJaeger: sure, that would be a huge help--thanks!13:40
*** mat128 has joined #openstack-infra13:40
dmsimardfungi: I don't know, it looked like this: http://paste.openstack.org/raw/621297/13:40
fungii'm currently working on getting the additional accountPatchReviewDb created for 2.13 now13:40
dmsimardfungi: so you had 3 patches from openstack-zuul-jobs (based on top of each other) depending on a patch from zuul-jobs13:40
*** dminca has quit IRC13:40
fungidmsimard: yep, that constitutes a loop13:40
*** dhajare has quit IRC13:41
fungidmsimard: so known/expected behavior13:41
dmsimardfungi: fair, could zuul bubble up an exception like it currently does on things like syntax errors ? :)13:41
*** mat128 has quit IRC13:42
fungidmsimard: i agree, that would be nice. i'm not sure what safety considerations there might be (i know the current behavior was chosen in order to avoid zuul doing anything dangerous when it identified a sequencing cycle)13:42
*** makowals has quit IRC13:43
AJaegerfungi: sent13:43
fungidmsimard: the normal way to sequence them, as a patch author, would have been to make just the bottom git parent in the series depends-on the one from the other repo13:43
fungithen zuul would have been able to clearly identify a serial order13:44
*** srobert has joined #openstack-infra13:44
dmsimardfungi: right, I wasn't sure if the parent patch's depends-on would have pulled in the dep for the two child commits so I did it that way first and then removed them when I saw it wasn't working13:45
*** mat128 has joined #openstack-infra13:46
fungiyeah, zuul is smart enough to work out that cross-repo deps and git deps are transitive with one another13:46
fungi#status log accountPatchReviewDb database created and gerrit2 account granted access in Review-MySQL trove instance, in preparation for upcoming gerrit upgrade maintenance13:48
openstackstatusfungi: finished logging13:48
*** felipemonteiro__ has joined #openstack-infra13:51
*** felipemonteiro_ has joined #openstack-infra13:52
clarkbgood morning. Looks like prep is going well. I am about 15-20 minutes away from proper keyboard13:54
*** dminca has joined #openstack-infra13:55
fungii think we're all set. i'm planning to do two more notices around 14:00 and 14:3013:55
*** felipemonteiro__ has quit IRC13:56
*** erlon has joined #openstack-infra13:57
*** makowals has joined #openstack-infra13:59
*** mat128 has quit IRC14:02
*** dminca has quit IRC14:04
*** Swami has joined #openstack-infra14:04
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: WIP: Honor cloud quotas before launching nodes  https://review.openstack.org/50383814:06
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Make max-servers optional  https://review.openstack.org/50428214:06
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support cores limit per pool  https://review.openstack.org/50428314:06
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Don't fail on quota exceeded  https://review.openstack.org/50305114:06
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support ram limit per pool  https://review.openstack.org/50428414:06
*** armax has joined #openstack-infra14:07
*** trown is now known as trown|brb14:09
*** trown|brb is now known as trown14:12
clarkbok at proper keyboard now with tea14:16
*** gongysh has joined #openstack-infra14:17
*** armax has quit IRC14:18
* mordred waves at clarkb and fungi14:18
*** marst_ has joined #openstack-infra14:19
*** marst_ has quit IRC14:19
clarkbit is raining here. When I went to denver it was proper summer weather, when I got back it was to the worst air quality in the country due to forest fires and today it is fall14:20
jeblairo/14:20
fungiin case anyone needs the pad url: https://etherpad.openstack.org/p/gerrit-2.13.-upgrade-steps14:21
mnasergood luck :D14:21
fungiwe should be ready up to "stop zuul" now14:21
fungithanks mnaser!14:21
*** jtomasek has quit IRC14:22
jeblairmy gertty is synced14:23
AJaegerclarkb: raining? That's great for the fires, isn't it?14:24
*** mat128 has joined #openstack-infra14:25
clarkbAJaeger: ya, though there is now concern of flooding and landslides in the fire area14:25
clarkbI am ready for the rain its been hot and dry for too long14:26
jeblairalso if the ash runs into reservoirs it's bad too.14:26
AJaegernothing perfect ;(14:26
clarkbfungi: looks like you set the charactern encoding and collation on the databaase as well. Thanks for getting everything prepped14:27
mnaser(except openstack)14:27
*** chlong has joined #openstack-infra14:27
*** LindaWang has joined #openstack-infra14:28
*** LindaWang has quit IRC14:29
*** Guest82210 has quit IRC14:29
fungiclarkb: yep, no sweat14:29
fungi#status notice Gerrit will be offline for the upgrade to 2.13 starting at 15:00 UTC (in roughly 30 minutes) and is expected to probably be down/unusable for 8+ hours while an offline reindex is performed: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html14:30
openstackstatusfungi: sending notice14:30
-openstackstatus- NOTICE: Gerrit will be offline for the upgrade to 2.13 starting at 15:00 UTC (in roughly 30 minutes) and is expected to probably be down/unusable for 8+ hours while an offline reindex is performed: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html14:31
*** hongbin has joined #openstack-infra14:32
*** rbrndt has joined #openstack-infra14:33
*** david-lyle has joined #openstack-infra14:34
*** Apoorva has joined #openstack-infra14:36
*** e0ne has quit IRC14:37
*** krtaylor has joined #openstack-infra14:37
*** chlong has quit IRC14:37
*** makowals has quit IRC14:37
*** makowals has joined #openstack-infra14:38
*** dminca has joined #openstack-infra14:38
*** ykarel has quit IRC14:38
*** chlong has joined #openstack-infra14:40
openstackgerritTony Breeds proposed openstack-infra/project-config master: Modify the check-uc jobs to only trigger on upper-constratints changes  https://review.openstack.org/50450114:41
mordredclarkb, fungi, jeblair: should I follow up to the last zuul rollout email with an update on status and the next-monday rollout intent?14:44
fungimordred: maybe wait until after we're into the gerrit upgrade window so that people don't get confused? not sure if anyone would14:45
jeblairmordred, fungi: ++14:45
*** Apoorva has quit IRC14:45
mordred++14:45
clarkb"I thought you said next monday!"14:46
clarkbI've gone ahead and wgetted the war into /tmp on review.o.o14:46
clarkbbeing the same DC that probably could've happened as scheduled on the etherpad though. Was quite fast14:47
fungialso, reminder that we wanted to announce https://review.openstack.org/482856 some time today (though after discovering that zuul master isn't enqueuing changes from check to gate for us without that, i wonder if there's any point to waiting to merge it)14:47
*** makowals has quit IRC14:47
clarkbfungi: for most third party ci (maybe all?) that isn' ta problem so probably best to announce it and then merge and/or tag later this week as originally planned14:48
prometheanfiretime for some luck :D14:49
clarkbI'm expecting the reindex step to be the "go and make lunch/dinner, send emails, etc" portion of the day14:49
fungiahh, yeah i guess merging it does cause them to need config adjustments if they're following master rather than pinning to a version like puppet-openstackci defaults to14:49
clarkbif anyone is wondering where they might squeeze that stuff in14:49
jeblairfile expense reports.  review changes in gertty.14:49
*** makowals has joined #openstack-infra14:50
prometheanfirenot all charges may be on the cards yet :|14:50
prometheanfirereview, but don't +W :P14:50
*** jtomasek has joined #openstack-infra14:51
*** dminca has quit IRC14:53
*** dminca has joined #openstack-infra14:54
clarkbjeblair: did you want to be in charge of stopping/starting zuul? Will likely have to reinstall with the tobias change because puppet probably undid my install over the weekend14:54
fungiyeah, expense report catch-up is my #2 task once the reindex is underway14:54
jeblairclarkb: will do14:54
clarkbjeblair: I have a branch in /opt/zuul on that node with tobias change rebased on master locally14:54
clarkbjeblair: since I wasn't sure if there were any other changes to master that we wanted too14:54
jeblairclarkb: got it14:54
jeblairpuppet is stopped globally, right?14:55
clarkbfungi: ^14:55
*** Apoorva has joined #openstack-infra14:55
pabelangermorning14:55
fungicorrect14:56
fungihas been for a few hours now14:56
fungii've triple-checked the cronjob and ps on puppetmaster14:56
*** xarses_ has joined #openstack-infra14:56
clarkbwe will puppet infracloud but that is it14:56
fungithe cronjob for infracloud is still intact but shouldn't pose a problem14:56
clarkb++14:56
fungiyeah14:56
fungii'll prep the status alert14:56
jeblairpatched zuul installed; ready to stop.14:57
*** gongysh has quit IRC14:57
clarkbfungi: you may want to create the screen on review.o.o so that it is appropriately sized :)14:57
*** dminca has quit IRC14:58
fungistatus notice The Gerrit service at https://review.openstack.org/ is offline for the upgrade to 2.13, for an indeterminate period of time hopefully not to exceed 23:59 UTC today: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html14:59
fungi^ that look good?14:59
clarkblgtm14:59
fungiclarkb: root screen created on review.o.o14:59
fungiremember to `su - gerrit2` from there as needed14:59
clarkbya I tried to capture that in the etherpad14:59
clarkbfungi: I think you can go ahead and send that notice then we can get started15:00
*** dminca has joined #openstack-infra15:00
fungi#status notice The Gerrit service at https://review.openstack.org/ is offline, upgrading to 2.13, for an indeterminate period of time hopefully not to exceed 23:59 UTC today: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html15:01
openstackstatusfungi: sending notice15:01
clarkbI will stop gerrit and run the database backup once jeblair reports zuul is stopped15:02
jeblairi will stop zuul now15:02
-openstackstatus- NOTICE: The Gerrit service at https://review.openstack.org/ is offline, upgrading to 2.13, for an indeterminate period of time hopefully not to exceed 23:59 UTC today: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html15:02
jeblairzuul is stopped15:02
jeblairclarkb: ^15:02
clarkbok stopping gerrit now15:02
jeblair(i'm going to work on a maintenance webpage for gerrit)15:03
clarkbok15:03
fungithanks! i remember someone mentioning that but kept forgetting what the status was15:03
clarkbok gerrit is stopped. Starting the db backup now15:03
openstackstatusfungi: finished sending notice15:04
fungifor those following along, reminder that the plan pad is: https://etherpad.openstack.org/p/gerrit-2.13.-upgrade-steps15:04
clarkbdb backup is in progress15:05
clarkbthinking about disabling gerrit should we move the init script aside too?15:05
clarkbrealizing the reindex will take time and ^ may be good belts and suspenders?15:05
mordredclarkb: ++15:05
clarkbfungi: maybe you can do that?15:05
fungii suppose that's not a terrible notion in case of unanticipated reboot15:05
*** gouthamr has joined #openstack-infra15:05
fungishall i just `exit 0` after the shebang?15:06
fungii have done that ^ for now15:06
clarkbI think its a symlink to the thing the war init installs? in which case that might be undone when we drop the new war in place (can just double check that afterwards)15:06
clarkbwe are about half done on the db backup based on file sizes15:07
fungiahh, yep, it's /home/gerrit2/review_site/bin/gerrit.sh so i guess i'll undo that and mv the symlink out of /etc/init.d/ to /root/ temporarily15:07
*** gongysh has joined #openstack-infra15:08
jeblairhow does this look? https://review.openstack.org/15:08
clarkbjeblair: ++15:08
jeblairi edited 50-review.openstack.org.conf in place, but backed it up to ~root.  i will update etherpad with revert steps15:09
*** armax has joined #openstack-infra15:09
clarkbok15:09
fungijeblair: maintenance page lgtm, thanks!15:10
pabelangerjeblair: thanks, I know I was on the hook for cowsay15:10
clarkbalright db backup is complete15:10
*** Sree has quit IRC15:11
jeblairpabelanger: we'll have plenty of time to update the content :)15:11
clarkbmordred: anythin gwe should do to check the validity of the db backup?15:11
*** Sree has joined #openstack-infra15:11
clarkbI'm moving ahead to updating the war15:12
fungithe cow says: maintenance15:12
clarkbthat sha1sum lgtm. Anyone else want to double check it?15:12
fungihttps://i.pinimg.com/736x/8e/c0/2c/8ec02c37ebafeed98cd47abd292e99af.jpg15:12
*** gongysh has quit IRC15:12
pabelangerfungi: hah15:13
fungiclarkb: checksum matches what i downloaded15:14
jeblairit matches the etherpad; that's immutable, right?15:14
fungiheh15:14
*** Sree has quit IRC15:16
clarkbok moving on to cleaning up old libs15:16
clarkbnow on to editing the secure.config15:17
fungilookin' good15:17
clarkbfungi: same password for db connection right?15:18
*** eumel8 has quit IRC15:18
*** annegentle has joined #openstack-infra15:18
*** annegentle has quit IRC15:18
*** annegentle has joined #openstack-infra15:18
fungiclarkb: correct, same account15:19
fungii simply added access to the new db to the existing account in that trove instance15:19
fungialso noticed we have a couple extra databases in that instance we could probably clean up (later)15:19
*** martinkopec has joined #openstack-infra15:20
clarkbok, I have edited and written to that file (still open in vim in root screen_ if someone else can sanity check that15:20
fungiseems fine to me15:22
clarkbthe next step is running gerrit init. Is there anything else we want to do before that?15:23
clarkbmordred: ^ maybe check the db backup validity somehow/15:23
clarkbthe size looked right at least15:24
jeblairit looks like a db backup and it looks appropriately large.15:25
clarkbok not hearing any opposition I'm moving ahead and running the init step15:26
clarkbwill run it under time too to collect data on production site so that we can record that and have it for the future15:27
fungiperfect15:27
*** Apoorva has quit IRC15:28
clarkbit is running, expected to take a few minutes15:28
*** hashar is now known as hasharAway15:28
mordredclarkb: yah - I'm comfortable with db backup validity - sorry for delay15:30
clarkbone of the underappreciated pluses to do this over irc is the built in timing info15:30
mordredright?15:30
fungiindeed, makes reconstructing timelines later a lot easyer15:31
fungieasier too15:31
clarkband I just don't have to explicitly mark down a timestamp when I say " expected to take a few minutes "15:31
*** dizquierdo has quit IRC15:31
clarkbinstead I can sip my tea15:31
*** dminca has left #openstack-infra15:32
fungias a larger example, once the reindexing is underway i'm going to go grocery shopping ;)15:32
*** vhosakot has joined #openstack-infra15:33
mordredyah15:35
*** martinkopec has quit IRC15:40
fungiit's almost done i think15:41
fungioh, maybe not15:41
Diabelkohello \o15:42
DiabelkoI15:42
DiabelkoI'm back with more questions15:42
clarkbthis should go relatively quickly compared to the schema updates iirc15:42
Diabelko:)15:42
DiabelkoDo you have any particular date in mind for Zuul v3 release?15:42
clarkbDiabelko: we are currently in the middle of a Gerrit upgrade. Discussing that in #zuul might be easiest today so that we can keep this channel focused on the gerrit upgrade15:44
Diabelkook, thanks and sorry then15:44
clarkbno problem15:45
fungii guess we didn't note the final schema revision number15:46
clarkb129 iirc15:47
fungioh, so nearly there after all15:47
*** camunoz has joined #openstack-infra15:49
smcginnisWasn't the reindexing the part that was expected to take hours?15:51
clarkbsmcginnis: yes, but this was also expected to take a while15:51
*** lin_yang has joined #openstack-infra15:51
clarkbit took almost 4 minutes on review-dev which hsa significantly less data on it15:51
*** Sree has joined #openstack-infra15:53
clarkbfungi: I feel like 127 heard you and is now taking its time15:56
fungiyeah, i shouldn't tempt fate15:56
fungithat must be one serious schema bump15:59
*** sbezverk has quit IRC16:00
fungihttps://gerrit.googlesource.com/gerrit/+/master/gerrit-server/src/main/java/com/google/gerrit/server/schema/Schema_127.java16:02
clarkbthis is moving all the data into that new database16:03
clarkbI think?16:04
fungihard to say... maybe? http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=34&rra_id=all16:05
*** ijw has joined #openstack-infra16:05
clarkbaccount_patch_reviews is the new table in accountPatchReviewDb I think16:05
*** trown is now known as trown|lunch16:06
*** dtantsur|afk is now known as dtantsur16:08
clarkbfungi: if you still have a db connection handy you should be able to easily check16:08
fungi974459 rows so far16:10
fungi985076 noq16:10
funginow16:10
fungiso growing steadily, yes16:10
mordredclarkb, fungi: yes - that migration should be transferring data16:10
fungiselect count(*) from account_patch_reviews;16:10
fungiif you want the quick and dirty rowcount16:11
mordredoh holy jesus - that's TERRIBLE SQL16:11
clarkbjust imagine if we had run it against an h2 db16:11
mordredit could have been done in a single line of sql, at least for mysql, rather than a select into java followed by a billion inserts16:11
*** supertakumi86 has joined #openstack-infra16:12
fungimordred: i expect that's attempting to emulate a h2 db in sql ;)16:12
fungiwritten by someone who mostly just knows h2?16:12
clarkbwell mysql only works on our branch so ya...16:12
mordredfor the record: "INSERT INTO account_patch_reviews (account_id, change_id, patch_set_id, file_name) SELECT account_id, change_id, patch_set_id, file_name from account_patch_reviews;"16:12
mordrednot that we can do anything about that now16:12
clarkbya time to just ride it out16:13
fungiup to 1110435 rows now, but no idea what the end count should be16:13
mordredyup. oh - is this the thing moving things from the one schema to the other?16:13
clarkbmordred: ya I think so16:13
clarkbmordred: by default into an h2 db16:13
clarkbbut for us into the mysql db you unbroke16:14
*** rcernin has quit IRC16:14
mordredfungi: if you do "select count(*) from account_patch_reviews" on the other db16:14
mordredfungi: that should tell you target row count16:14
fungiahh, i missed this is just migrating a table from one to the other16:14
*** ramishra has quit IRC16:14
mordredfungi: I'm guessing this is copying data from reviewdb to the new schedma right?16:14
mordredso it's select count(*) from reviewdb.account_patch_reviews ; select count(*) from accountPatchReviewDb.account_patch_reviews;16:15
fungi11566783 in reviewdb.account_patch_reviews so we're ~10% complete now :(16:15
mordredboo16:15
clarkbwow16:15
mordredyah- it's a hideously inefficient way to do that16:15
*** egonzalez has quit IRC16:16
*** pcaruana has quit IRC16:16
clarkbgood thing we allocated all the time for this upgrade :/16:16
fungihave a feeling the outage is going to end up well past midnight utc given we're not even on to the reindex yet16:16
fungimaybe 2.13 offline reindex is faster (fingers crossed)16:17
mordredhttp://paste.openstack.org/show/621345/ <-- in case somehting goes wrong and we have to abort and try again later, we should update that code to do that16:17
clarkbits unfortunate that all of this code is basically unused by gerrit's maintainers16:17
fungiwell, 11% now so it's going fast-ish i suppose16:17
*** Sree has quit IRC16:20
fungi12%... maybe i should go get my grocery shopping out of the way now16:20
*** Sree has joined #openstack-infra16:20
*** sbezverk has joined #openstack-infra16:22
*** dave-mccowan has quit IRC16:22
clarkbya this is on the order of a couple hours? I think I did that math right16:23
fungiwe seem to be inching along at ~0.34% per minute, so at this rate we're looking at completion of the schema 127 migration at least 4 hours from now16:23
*** gongysh has joined #openstack-infra16:23
clarkbhrm fungis math is probably better than my math16:23
fungi(rough numbers based on timestamps in irc scrollback)16:23
clarkbthat would almost certainly push the upgrade past 0000 UTC16:24
fungiindeed16:24
jeblairis it worth considering aborting the current process, applying mordred's change, and restarting?16:24
fungii doubt we'll even start the reindex until 20:30-21:00 utc at the earliest16:24
fungiwe'd need to rebuild a new gerrit war with the new schema migration, correct?16:25
clarkbjeblair: possibly? We'd have to get a new war built with that fix compiled in which may be painful (have to undo current upgade and get zuul running again)16:25
clarkbmordred: ^ how much faster would that be?16:25
*** Sree has quit IRC16:25
fungiwe could try a manual gerrit build, but...16:25
fungiif things go wrong, we could end up losing more time on the workaround than the tested (if slow) path16:26
clarkbor we could do that migration step by hand but figuring out how to properly record that may also be tricky16:26
jeblairtrue... i guess all we'd really need to do is abort, perform 127 by hand, then resume?16:26
mordredjeblair: yah. I was thinking that earlier16:26
fungiassuming it can figure out 127 is actually applied, right16:27
mordredsince the db schema states record which schema version they are16:27
*** e0ne has joined #openstack-infra16:27
mordredall the migration does is check the schema version in the db16:27
fungiokay, so that can be updated by hand i suppose16:27
mordredyup16:27
clarkbI'd defer to mordred/jeblair on deciding if that is a route we want to take as they are more familiar both with mysql and hte gerrit db16:27
fungiyeah, i'm certainly open to it if it shaves another 4+ hours off the maintenance16:28
mordredI think aborting the current java process, which should abort the sql16:28
*** dizquierdo has joined #openstack-infra16:28
*** gongysh has quit IRC16:28
mordredthen doing the sql by hand - should be fine16:29
clarkbhas it committed all of the previous migrations?16:29
fungiwe have a solid db backup, so worst case we just end up with gerrit down for longer when we have to backtrack and retry... which is basically what we're already faced with16:29
mordredthe jdbc is configured to do 1000 row batches16:29
dmsimardI would tend to err more on the conservative/safe side of things and say that around midnight UTC most people are no longer actively working and bleeding beyond midnight is probably not a huge impact16:29
clarkbdmsimard: ya and we intentionally chose today as it was expected to be quieter16:29
*** jpich has quit IRC16:29
jeblairdb says we are at version 12616:29
fungidmsimard: yeah, but i'm not sure i personally want to be awake too much past that either ;)16:29
fungi(if it can be helped, that is)16:30
mordredso if we abort in the middle of one of the inserts (which we certaily will) the rollback size will only be 1000 rows, which will not be a long rollback16:30
jeblairclarkb: that suggests that we have committed previous updates16:30
clarkbjeblair: ya16:30
dmsimardfungi: of course :D16:30
clarkbmordred: but also we'd probably have to drop the entire table and start from scratch? or is ther ea good way to only insert what we havne't already insterted?16:30
clarkbmordred: remember that we have the auto incrementing file_id on that table in our setup16:31
jeblairi'd drop and start from scratch since this will be so much faster anyway16:31
mordredyah. lemme try something real quick16:31
*** baoli has quit IRC16:32
jeblairclarkb: and that should clear out the file_id autoincrement16:32
clarkbjeblair: ya16:32
clarkbfungi: where are we at now percentage wise?16:32
*** baoli has joined #openstack-infra16:33
fungi~16.5%16:33
* fungi also just realized he did a #status notice rather than alert16:33
clarkbso ya ~0.4%/minute16:34
* tosky points out that everyone probably noticed it at this point16:34
fungi#status alert The Gerrit service at https://review.openstack.org/ is offline, upgrading to 2.13, for an indeterminate period of time hopefully not to exceed 23:59 UTC today: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html16:34
openstackstatusfungi: sending alert16:34
clarkb~3.5 hours away16:34
fungitosky: well, point being this will update channel topics too16:34
toskyoh, sure16:34
fungiso newcomers also notice, hopefully16:35
-openstackstatus- NOTICE: The Gerrit service at https://review.openstack.org/ is offline, upgrading to 2.13, for an indeterminate period of time hopefully not to exceed 23:59 UTC today: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html16:36
*** ChanServ changes topic to "The Gerrit service at https://review.openstack.org/ is offline, upgrading to 2.13, for an indeterminate period of time hopefully not to exceed 23:59 UTC today: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120533.html"16:36
mordredclarkb, fungi, jeblair: ok - I'm not sure the insert into / select from will be quicker - just did some poking at that16:37
*** Sree has joined #openstack-infra16:37
*** nicolasbock has quit IRC16:37
clarkbgiven that uncertaintly and the need to drop and restart maybe we just ride it out and complete at ~2000 ?16:38
*** panda is now known as panda|bbl16:38
mordredclarkb: yah- I think that's best here16:41
clarkbok16:41
clarkbthat also means that we won't get to secondary index reindenxing until much later in the process... which likely means a late night :/16:41
mordredclarkb: remind me again why we didn't make file_id the primary key of the table since it's an auto-inc?16:41
mordredclarkb: I should just not ask that question16:42
clarkbmordred: I think becuase that other key already was the primary key?16:42
mordredclarkb: I think our intent was "modify this as little as possible from upstream but still work"16:42
clarkbya16:42
fungiecho $((`sudo -H mysql -e 'select count(*) from account_patch_reviews' accountPatchReviewDb|tail -n1`*100/11566783))16:43
fungifor anybody who wants a quick percent completion check16:43
fungijust hit 19%16:44
fungiif the decision is to just ride this out, i'm going grocery shopping nowish16:44
clarkbproposal: run as is, check in at 1930UTC. Then we either continue because it is close or have a sad and consider rollback. If we continue then do a sanity check during reindexing?16:45
clarkbAbsolute worst case we could consider picking it up in the morning again and basically double our outage time :/16:45
fungisanity check of the accountPatchReviewDb.account_patch_reviews table you mean?16:45
clarkbfungi: no, I mean sanity check the runtime of reindexing once we get it going16:46
fungiahh16:46
clarkbto give us an indication of when it will end. It does give you data about completion rates. Biggest problem is going to be large projects like nova because you can only single thread reindex those16:46
clarkb(so basically once we get to the last project (like nova) I think we can roughly guestimate completion time)16:47
fungiyeah, we've done plenty in recent history and know what sort of runtime we're looking at there16:47
clarkbjeblair: mordred ^ does that sound reasonable?16:47
fungihopefully with 2.13 it's relatively consistent (if not faster)16:47
mordredclarkb: yes. I agree with the above16:48
fungii think zaro also got a patch in a while back to optimize reindexing by sorting to reindex the projects with the most changes first16:48
*** camunoz has quit IRC16:48
mordredat this rate I do expect this db copy to take an additional 4 hours16:48
jeblairoh i hope that made it in :)16:48
fungiso that it ends up with less time at the end where threads are sitting around idle16:49
clarkbok lets keep an eye on it but not worry about a status check until 1930UTC that should be long enough to do a grocery run :)16:49
mordredas it seems to be doing roughly 3 minutes per percent complete, and we're at 20%16:49
jeblairclarkb: sounds good16:49
fungimordred: that jives with my earlier estimates too, yes16:49
fungiso seems to be steady at that rate at least16:49
fungi21%16:49
mordred++16:49
fungiheaded to the store, bbs16:50
*** jpena is now known as jpena|away16:50
mordredit's honestly going to be a fairly constant number based on disk speed since this is working on 1000 row chunks16:50
* clarkb wonders how large shops like sony/sap/etc manage these upgrades sanely16:51
jeblairdisk+network16:51
*** lewo` has joined #openstack-infra16:51
mordredthat said it IS going to slow down slightly as we continue, because of updating the two secondary indexes  - but since those are each fairly small I don't expect those index updates to make things slower aggressively16:51
clarkbI'm going to put dinner in the crock pot. Will check back in after16:52
mordred(the initial insert should remain linear since innodb does inserts in primary key order into a b+tree)16:52
jeblairhttps://review.openstack.org/status.html16:52
jeblairthat's % complete updates once a minute16:52
mordredjeblair: ++16:53
clarkbcool thank you for that16:53
jeblair(it's fungi's one-liner in a loop)16:53
*** lewo has quit IRC16:54
smcginnisNice jeblair. ;)16:56
jeblairi, erm, made a small update.16:57
*** dave-mccowan has joined #openstack-infra16:58
*** claudiub has joined #openstack-infra16:58
smcginnisHah, love it!16:58
mordredjeblair: haha. nice16:58
*** ralonsoh has quit IRC16:59
clarkbI refreshed before reading irc :)17:00
*** shardy has quit IRC17:00
zaroAhh nice, you guys are upgrading :)17:01
zarofungi: i think you were referring to this? https://gerrit-review.googlesource.com/c/gerrit/+/7848117:01
jeblairzaro: yes!  and thank you so much for that. :)17:02
*** efoley has quit IRC17:02
clarkbzaro attempting to at least. thus db migration is not fast17:02
jeblair25%17:04
*** gouthamr has quit IRC17:04
kmallocI just wanted to say the "hey gerrit is down" page is nice.17:05
kmallocthanks :)17:05
jeblairand zaro's change is in 2.13, so we should benefit from that17:06
jeblairkmalloc: thanks :)17:06
kmallocalso, hi everyone, welcome back from the PTG (to IRC, if not physically at your normal-ish location)17:06
zaroclarkb: yep, large companies have multi-master (ish) setup so they can do this without downtime.17:08
*** gongysh has joined #openstack-infra17:09
*** Apoorva has joined #openstack-infra17:11
*** srobert_ has joined #openstack-infra17:15
mordredkmalloc: ++17:17
*** srobert has quit IRC17:17
* mordred thinks the gerrit is down page will likely cut down on the number of people who come in to let us know gerrit is down this time17:17
*** Swami has quit IRC17:17
*** srobert_ has quit IRC17:17
*** srobert has joined #openstack-infra17:18
kmallocmordred: so... did you know... gerrit is down17:18
* kmalloc ducks.17:18
pabelangerhttp://paste.openstack.org/raw/621349/17:19
pabelanger:)17:19
toskybut the page that gerrit is down is only visible from the homepage of review.gerrit.org; if you try to access a review, you see only "Not Found"17:19
clarkbzaro I thought multi master was still only a google thing17:22
*** trown|lunch is now known as trown17:22
mordredtosky: hrm.17:23
*** bobh has quit IRC17:25
*** sdague has quit IRC17:25
zaroclarkb: your right, not fully multi-master more like active/passive17:26
zarowith the help of a bunch of plugins17:26
*** camunoz has joined #openstack-infra17:26
*** rhallisey has quit IRC17:27
mordredzaro: like upgrade passive, failover promoting passive to active and old-active to passive, upgrade old-active?17:28
*** rhallisey has joined #openstack-infra17:28
*** sdague has joined #openstack-infra17:29
*** dtantsur is now known as dtantsur|afk17:30
*** ociuhandu has quit IRC17:31
*** chlong has quit IRC17:33
*** tosky has quit IRC17:33
*** nicolasbock has joined #openstack-infra17:33
*** rbrndt has quit IRC17:33
*** electrofelix has quit IRC17:35
*** ijw has quit IRC17:35
*** nicolasbock has quit IRC17:38
*** dave-mcc_ has joined #openstack-infra17:39
*** dave-mccowan has quit IRC17:39
zaroyes, i know that’s what erricson does for sure.17:44
*** Sree has quit IRC17:45
*** gongysh has quit IRC17:46
*** sambetts is now known as sambetts|afk17:47
*** baoli_ has joined #openstack-infra17:47
*** chlong has joined #openstack-infra17:48
*** Swami has joined #openstack-infra17:50
*** baoli has quit IRC17:50
*** nicolasbock has joined #openstack-infra17:50
*** baoli_ has quit IRC17:51
*** baoli has joined #openstack-infra17:51
fungigroceries achieved17:52
fungijeblair: the cowsay addition to the migration status is a nice touch!17:52
fungithe cow says: now it's time for lunch17:53
*** gongysh has joined #openstack-infra17:53
*** srobert has quit IRC17:53
*** jtomasek has quit IRC17:57
mordredfungi, clarkb, jeblair: how's https://etherpad.openstack.org/p/kpYi29mKM9 for a followup email on v3 status?18:02
*** liusheng has quit IRC18:02
*** liusheng has joined #openstack-infra18:03
clarkbmordred: lgtm18:05
clarkbzaro: gotcha18:06
*** ijw has joined #openstack-infra18:07
fungimordred: lgtm^2 thanks!18:07
*** ijw has joined #openstack-infra18:07
clarkbnot to get ahead of ourselves, but how comfortable are people with a potential later evening to get gerrit up? I can be around but would prefer to not be the only root checking gerrit is happyness this evening18:07
fungii will stick around18:07
fungithough later gets later earlier where i am18:07
clarkbya18:08
jeblairclarkb: i'm on board18:08
clarkbok cool, thanks everyone18:08
jeblairmordred: making some minor changes to etherpad18:08
*** tosky has joined #openstack-infra18:09
jeblairmordred: s/some/one/.  lgtm.18:09
*** jdandrea has joined #openstack-infra18:14
mordredclarkb: I'm on board18:15
*** tesseract has quit IRC18:15
mordredjeblair: I have incorporated your change - does my version of your change look ok?18:15
jeblairmordred: that was fungi's, but it looks good to me.  mine is "we're still" in line 20.18:17
*** dizquierdo is now known as dizquierdo_afk18:17
jeblair(previously; sentence no verb)18:18
pabelangerclarkb: I too can be around this evening18:18
jdandrea@clarkb At the PTG jaypipes suggested I reach out to you about project ownership changes on launchpad et. al - should I email w/Q's or PM here?18:18
jeblairjdandrea: what needs to happen?18:18
jdandreajeblair Previous owner is no longer involved and has also left place of business. Now re-reviewing Project Owner's Guide to see where changes may be needed. (I'm also PTL.)18:20
jeblairjdandrea: which project(s)?18:20
jdandreaFrom a launchpad POV: https://launchpad.net/valet18:20
jdandreaMay be others (e.g., IRC, etc.)18:20
*** hasharAway is now known as hashar18:21
fungiit doesn't look like the openstack admins group owns the maintainer group for that project (unsurprising for something which isn't an official openstack deliverable)18:22
jeblairagreed18:22
jdandreafungi *nod* ... and I can't speak to the completeness of the setup, ofc ... but I'll dot i's and cross t's if need be. Whatever the appopriate steps to remediate are, I'm happy to do 'em.18:23
fungijdandrea: more that we can't really do anything to help other than point you to the launchpad admins, i expect18:23
jdandreaAye, not official.18:23
jdandreafungi That's fine. I'll check with them. As for other areas (IRC, Wiki?, gerrit [down for maint atm] f'rinstance), not sure what to do there.18:24
clarkbzaro: do you know why this table is moved into its own db?18:24
*** supertakumi86 has quit IRC18:24
jeblairjdandrea: we don't really get involved until the project applies to join as an official project (the guide should have info about that).  but even from a technical point of view, we don't have that access.  so to remedy this, you'll need to ask one of the "administrator" members of this group: https://launchpad.net/~valet-pathfinders/+members  to elevate you to an administrator.  if they aren't responsive, you can ask the launchpad ...18:24
jeblair... admins.18:24
clarkbseems like a lot of effort and upgrade pain for data that is essentially in the same format just in a different location18:24
fungijdandrea: you have an irc channel not listed at https://wiki.openstack.org/wiki/IRC i guess18:24
jdandrea@jeblair Already on it, mhm. Yes, IRC appears unlisted (makes sense). P'raps I contact freenode folks in that case.18:25
fungiahh, yeah looks like we at least have an accessbot entry for a #openstack-valet channel18:25
*** bobh has joined #openstack-infra18:26
*** mixos has joined #openstack-infra18:26
jeblairjdandrea: if accessbot is okay with it, the irc channel should be fine; you'll just want to add it to the openstack wiki to help others find it18:26
*** srobert has joined #openstack-infra18:26
fungijdandrea: as for gerrit, yeah there are likely review groups you may need to be added into for those repos... again similar to launchpad it's better if you can get existing members of those review groups to add you (easier to check that once the gerrit maintenance is over, so maybe revisit tomorrow?)18:27
jdandrea@jeblair OK. But do I have wiki editing access, I wonder (unsure if it's tied to the project or if I just need an AuthN'ed ubuntu one login).18:28
jdandrea@fungi Aye, Gerrit we'll handle later.18:28
fungijdandrea: the wiki doesn't have any project-specific acls so shouldn't require any adjustment18:28
jdandreaAppreciate the advisement.18:28
jdandrea*nod* ok18:28
fungias long as you can log into the wiki, you can edit the same pages as everyone else (except those locked to admin-only edits)18:29
*** bobh has quit IRC18:31
*** bobh has joined #openstack-infra18:31
jeblairclarkb, fungi: is it the case that changing the label case in the config only works if this change is present?  http://git.openstack.org/cgit/openstack-infra/zuul/commit/?id=00ae6c9b0959c9850180f1dfe684149f4d8e268e18:33
jeblairclarkb, fungi: i'm trying to catch up with the reverts, etc, over the weekend18:33
mordredjeblair: gotcha - sending now18:34
*** jtomasek has joined #openstack-infra18:34
clarkbjeblair: yes as well as tobiash's change for checking the NEED results properly18:34
clarkb--Verified is not a valid arg to gerrit review re the change you linked18:35
clarkbbut --label Verified=1 is18:35
fungiyup, caught us by surprise18:36
jeblairclarkb: so, if you're gating, *both* of those changes must be running *before* the case can be changed in the config?18:36
*** e0ne has quit IRC18:36
clarkbcorrwct18:37
*** e0ne has joined #openstack-infra18:37
clarkbbut if not gating only the one you linked18:37
clarkbalso the one you linked got lost in the connections changes iirc so a change got pushed in july to add it back in18:38
jeblairso there's no sequence that folks can do to avoid force merging a change...?18:38
clarkbthe fix for --label can go in before anything else18:38
clarkbbut then you need both layout.yaml updatr and tobiash change roughly together18:39
clarkbso ya I think no proper sequemce18:39
tobiashI agree18:39
*** e0ne has quit IRC18:39
jeblair1) run with label; 2) merge case config change; 3) restart with case-sensitive change18:40
jeblairi guess ^ that would work18:40
jeblairas long as 3 happens right after 218:40
tobiashJust wanted to suggest that...18:40
tobiashBut was too slow ;)18:41
jeblairso we should probably issue 2 point releases; first with (1), then wait a week and then (3)....18:41
jeblairthat way people consuming from releases have a path forward18:41
clarkbjeblair: ya18:41
jeblairokay, i will try to write that up in an etherpad18:41
jeblairwell, the change for (1) doesn't really get you anything if you're gating, so maybe we should just stick with one relase in a week.  but still tell folks that they should merge the config change right before upgrading/restarting.18:51
clarkbthat sould work as well18:52
mordredjeblair: ++18:52
*** rbrndt has joined #openstack-infra18:52
*** jtomasek has quit IRC18:53
clarkbI have to say cowsay status updates has made me eel significantly better about this whole situation18:54
jeblairclarkb: maybe we can pipe the reindex to a file and have another cow read that out to us?18:54
clarkbhahaha18:55
*** caphrim007 has joined #openstack-infra18:55
jeblair(it's funny, but i'm actually serious since i'm always in favor of tools that let everyone participate in sysadminning :)18:56
clarkbI think the reindexing output is ncurses? I want ot say it has a spinning bar thing. Might make it interesting to pipe to cowsay18:56
fungiif memory serves, it's using ansi to back up the cursor and overwrite18:57
jeblairmaybe we can split it to a file and i can write a quick parser18:58
clarkbjeblair: you can test on review-dev. It is a 16 minute or so reindex there (though maybe we don't want to take that server offline if we need to cross check anything?)18:59
*** mixos has quit IRC19:00
jeblairclarkb, fungi: maybe we can just "C-a H" and use the screen log?19:00
jeblairmaybe that will work with no substantial change to the process19:01
* jeblair lunches19:02
*** spzala has joined #openstack-infra19:02
clarkbthat is a good idea. I'll be back for 1930 check in.19:02
fungiyeah, that's probably fine if we don't accidentally cat any files containing passwords in that session19:02
*** chlong has quit IRC19:12
*** mixos has joined #openstack-infra19:15
*** tiswanso has quit IRC19:24
*** tinwood has quit IRC19:24
*** bobh has quit IRC19:24
*** chlong has joined #openstack-infra19:24
*** tiswanso has joined #openstack-infra19:25
*** ociuhandu has joined #openstack-infra19:25
*** tinwood has joined #openstack-infra19:26
clarkbzaro: are you around? is the reindex thread count half the cpu count?19:27
*** tiswanso has quit IRC19:28
*** annegentle has quit IRC19:28
clarkblooks like I ran threads = 4 on review-dev with 8 vcpu and we have 16 vcpu on review.o.o so thinking we should bump that number up to 819:28
* clarkb edits etherpad and makes it 819:28
*** tiswanso has joined #openstack-infra19:29
clarkbwe also have twice the memory on review.o.o so that should scale up mostly cleanly19:30
clarkbok it is 1930ish now19:30
dmsimardhow are things looking ?19:31
clarkbcowsays says we are 76% complete on that migration19:31
clarkbdoing rough math I think that means it should complete in about an hour19:31
*** slaweq_ has joined #openstack-infra19:31
clarkb3 minutes per percent, 3 * 24 = 72 minutes19:31
jeblairclarkb: ack.  will be back in 1h.  :)19:32
clarkbya I think lets sync back up again at 2030 as I expect we will be close to actually doing thinsg then19:33
*** srobert has quit IRC19:33
prometheanfirewom 2419:33
*** dave-mcc_ has quit IRC19:35
*** yamahata has quit IRC19:39
fungithere's a strong chance the schema migration will finish at an inconvenient time for dinner with inlaws, and i may disappear for a couple hours right around that timeframe :(19:41
fungibut i'll be back on after to hopefully watch the exciting reindex19:41
clarkbthats ok ya for that reason :)19:42
clarkbwe go from watching glue set to paint dry19:42
fungispackle, primer and paint19:42
*** ijw has quit IRC19:42
*** florianf has quit IRC19:45
*** tiswanso has quit IRC19:45
*** Sree has joined #openstack-infra19:46
*** dave-mccowan has joined #openstack-infra19:50
*** Sree has quit IRC19:50
*** spzala_ has joined #openstack-infra19:51
*** spzala has quit IRC19:52
mordredclarkb, fungi: I bought both primer and paint this weekend19:57
jeblairlast year i watched plaster dry; it's actually kind of awesome.  changes color and sheen over the course of about a week.  it's really pretty.19:57
fungii'm still trying to use drywall mud and wet sanding to smooth out the irregularities in our master bathroom following a very frustrating wallpaper removal episode19:59
fungiprimer and paint are waiting patiently for me19:59
mordredwe've got some cabinets we're going to repaint and have selected a couple of doors to use to test the options of "just paint", "prime and paint", "sand, prime and paint" and "sand, prime, paint, sand" and "sand, prime, paint, sand and polycrylic finish"20:00
mordredobviously all on a scale of "is this option better enough to warrant the effort"20:00
fungispam, spam, spam, eggs, beans and spam20:00
*** e0ne has joined #openstack-infra20:00
sdakeis cdent on pto - or does he have a new nickname now? :)20:00
*** ccamacho has quit IRC20:01
sdakefungi - for some reason i think chris's nick was not cdent, could your verify?20:02
*** hashar has quit IRC20:03
*** tiswanso has joined #openstack-infra20:03
fungisdake: according to my client log, he seems to have disconnected around utc midnight friday and not rejoined20:03
*** tiswanso has quit IRC20:03
fungiif his client used the same ident but a different nick, i would see that in my logs too, but he doesn't appear to have done20:03
*** tiswanso has joined #openstack-infra20:04
fungiodds are he's still travelling20:04
fungior taking a few days off20:04
sdakefungi thanks man20:04
fungiyou're welcome20:05
* fungi gets back to the excitement which is filing expense reports20:05
clarkbI should do that20:07
clarkbbyt last I checked not everything had posted yet :(20:07
fungii use receipts, the old fashioned way20:07
fungiwell, coupled with a semi-ancient usb flatbed scanner20:08
jeblairmordred: you may want to add "trisodium phosphate" to the beginning of that list (ie, "tsp, prime, paint").  sanding should obviate the need for tsp though.20:09
*** yamahata has joined #openstack-infra20:12
mordredjeblair: indeed!20:13
fungior a nice long soak in aqua regia20:14
fungiafter which you shouldn't have anything at all which needs painting20:14
*** esberglu has quit IRC20:14
*** annegentle has joined #openstack-infra20:14
*** ijw has joined #openstack-infra20:15
mordredjeblair: the question there would be which thing, sanding or tsp, has better impact when divided by effort (with tsp we might need to mask/protect more things, but then we'd be sponging and not actually sanding which could certainly be a win)20:15
mordredjeblair: so - yah - good thing to test20:15
mordredsince we're doing kitchen cabinets with fluting, sanding will be extra suck20:17
jeblairhttps://etherpad.openstack.org/p/ppQOrvMAz820:21
jeblairhow's that look for the case sensitive zuul email?20:21
* clarkb tastes chili then will look20:22
*** armax has quit IRC20:23
clarkbjeblair: I think that is a very clear description of the steps to take +120:25
mordredjeblair: I like it - I think it could use a note that the driver for this is that in newer gerrit labels are case sensitive20:28
mordredjeblair: (we're not just doing it for fun)20:28
*** cody-somerville has joined #openstack-infra20:29
*** cody-somerville has quit IRC20:29
*** cody-somerville has joined #openstack-infra20:29
clarkb2030 is here and sure enough 127 appears to be near done20:29
fungi97% complete and i have to disappear for dinner :/20:29
clarkbfungi: enjoy20:29
*** esberglu has joined #openstack-infra20:29
fungii'll be back on as soon as possible. sorry all!20:30
*** esberglu has quit IRC20:30
*** esberglu has joined #openstack-infra20:30
clarkbI'll ping when it is complete then track the remainder of the db migration20:30
jeblairmordred: to the bsest of my knowledge, it's actually that certain database configurations make it case sensitive; it's not necessarily driven by gerrit upgrades itself...20:30
*** srobert has joined #openstack-infra20:30
clarkbjeblair: ya thinking about that more we've used the binary colation in mysql for a long time so notsure if that is the case (I thought it may have been a switch)20:31
jeblairclarkb: oh, i thought we tested it both ways20:31
jeblairso perhaps the shared understanding is more like "dude a bunch of stuff is changing and it's way too complicated for us to untangle".20:32
clarkb++20:32
clarkbmordred: when the migration is over can you do the "make sure accountPatchReviewDb" has data in mysql and not h2 step (though maybe thats trivial since that is where cowsay gets its data from20:33
clarkbya I think that is redundant iwth cowsay doing maths20:33
jeblairclarkb, mordred: how's the new sentence in there?20:34
clarkbwfm20:34
jeblairi will strike the second unfortunately so things aren't double unfortunate.20:35
clarkb99%20:36
clarkbits done20:38
pabelangeryay20:38
smcginnisBut the cow still says 99%. I need closure!20:39
jeblairsmcginnis: refresh?20:39
smcginnisOh, whew.20:39
smcginnis:)20:39
clarkbif someone can just double check accountPatchReviewDb has contnet, I am moving forward with the jar installations20:40
jeblairclarkb: on it20:40
jeblairmordred: can i 'drop table testmt' from accountPatchReviewDb ?20:41
*** annegentle has quit IRC20:41
mordredjeblair: yes20:41
jeblairclarkb: i agree with the cow; the table has data, and some rows i spot checked look sane.20:42
jeblairmordred: done20:43
clarkbjeblair: thanks20:43
clarkbI have sorted the jar situation too20:43
clarkbbacking up the 2.11 secondary index now20:44
jeblairpabelanger: are we really building nodes? (nodepool graph looks like it)20:46
clarkbok infra-root, ready for reindexing?20:46
jeblairi have started a screen log20:46
clarkbI bumped the threads from 4 on review-dev to 8 on review.o.o as there are twice as many vcpu on review.o.o20:46
pabelangerjeblair: ya, looks like nodepool is trying, but failing to connect to gearman20:47
clarkbhere goes20:47
jeblairpabelanger: oh okay.  thanks.20:47
jeblairclarkb: ++20:47
clarkbjeblair: there is a percentage number there so just have to grep that out if possible20:49
*** e0ne has quit IRC20:49
*** e0ne has joined #openstack-infra20:50
jeblairya, i'll start working on that now20:50
*** e0ne has quit IRC20:50
*** e0ne has joined #openstack-infra20:51
jeblairi thought it had a thing telling us project names too?20:51
clarkbjeblair: I think that is the second part of the reindex20:51
*** e0ne has quit IRC20:51
clarkbI want to say this going as quickly as it appears to be is just a trick to make usfeel better20:51
clarkbbut once changes are done it goes thorugh and does project by project20:51
*** e0ne has joined #openstack-infra20:51
*** kgiusti has left #openstack-infra20:52
*** e0ne has quit IRC20:52
*** bobh has joined #openstack-infra20:52
*** e0ne has joined #openstack-infra20:52
*** e0ne has quit IRC20:52
clarkbgerrit needs a gc command20:53
*** e0ne has joined #openstack-infra20:53
*** e0ne has quit IRC20:53
clarkbjeblair: ya it just added a second percentage related to projects20:53
*** jpena|away is now known as jpena|off20:54
*** dprince has quit IRC20:56
*** dizquierdo_afk is now known as dizquierdo20:56
*** jcoufal has quit IRC20:57
*** tiswanso has quit IRC21:00
*** trown is now known as trown|outtypewww21:01
*** yamahata has quit IRC21:01
*** yamahata has joined #openstack-infra21:01
*** srobert has quit IRC21:02
jeblairthere are exceptions in the log21:02
jeblaircannot merge xxx ; missing blob yyy21:03
clarkbya I just read them those are mergeability checks and appears to have tracebacks when they fail21:03
clarkbin the general case I am guessing that is ok?21:04
*** tiswanso has joined #openstack-infra21:04
*** srobert has joined #openstack-infra21:04
clarkbjeblair: are you able to pull the full traceback out of your screen log and paste it somewhere?21:04
jeblairclarkb: yep21:04
clarkbI wonder if that is just the normal way that unmergeable changes show up when reindexed21:05
jeblairhttp://paste.openstack.org/show/621357/21:05
clarkbalso when doing this on review-dev when there were fatal reindex problems reindexing stopped iirc21:05
jeblairclarkb: i think the missing blob suggests that there's an error in a git repo21:05
clarkbjeblair: ya21:06
clarkbmaybe related to the nova specs problem(s)?21:06
jeblairclarkb: this may be related to the errors we saw in ... i can't remember the repo name.... ya that one21:06
clarkbianw: ^ if you are around does that traceback look familar21:06
jeblairclarkb: there's at least one other traceback, but it looks similar.  just different shas21:06
clarkbya it has gone by a few times definitely more than just two21:07
jeblaircow is updated: https://review.openstack.org/status.html21:10
clarkbnote it requires a hard refresh, or at least did for me21:11
clarkbalso that look sawesome21:11
jeblairi regexed it a bit since the native output is confusing21:11
ianwclarkb: http://paste.openstack.org/show/621357/?21:11
clarkbya native output didn't seem to rewrite the line properly when it added the projects number21:11
clarkbianw: ya21:11
ianwoh right, like the corrupt infra-specs repo ... sorry just catching up ... looking21:12
clarkbtyty21:12
clarkbI bet those last 3 projects are nova, neutron and openstack-manuals nad they have probably been running since the start :)21:13
clarkbI expect we'll be sitting at 99% for a while21:14
*** thorst has quit IRC21:15
*** ldnunes has quit IRC21:19
ianwclarkb: ahh, yeah, no sorry ... that manifested only as the remote end (github) rejecting the push (actually still happens in the replication log ... i think agreed with fungi there was probably no better ideas than http://lists.openstack.org/pipermail/openstack-dev/2017-June/119166.html atm)21:20
clarkbok thanks for looking. I imagine that those things are likely related. Haven't seen any recent tracebacks fly by21:20
*** srobert_ has joined #openstack-infra21:21
clarkbwe appear to be indexing about a couple hunrdred changes per minute and have about 10k change sto go21:23
clarkbso ~50 minutes away from being done?21:23
*** srobert has quit IRC21:24
clarkbanother thing to look into is if we can do the database migration and the secondary reindex at the same time21:25
clarkbzaro: ^ any idea if that is a possibility? Seems potentially dangerous if the secondary index has strong deps on the mysql db though21:25
*** srobert_ has quit IRC21:25
dmsimardCouple hundred per minute, 10k changes to go at 50 minutes ? Math seems wrong :)21:27
*** Goneri has quit IRC21:27
*** cody-somerville has quit IRC21:27
clarkbdid I do that wrong?21:27
jeblairseems about right to me21:29
jeblairi just clocked 99 in 30 seconds21:29
*** gouthamr has joined #openstack-infra21:30
*** tiswanso has quit IRC21:30
*** thorst has joined #openstack-infra21:33
dmsimardAh ok, thought it was in the upper hundreds :p21:35
*** baoli has quit IRC21:37
*** thorst has quit IRC21:37
*** wolverineav has quit IRC21:38
*** jkilpatr has quit IRC21:39
jeblairheh, more like almost exactly 2.0 hundreds.  :)21:39
*** aeng has joined #openstack-infra21:40
*** mat128 has quit IRC21:40
*** xyang1 has quit IRC21:40
*** lbragstad has joined #openstack-infra21:43
*** Sukhdev has joined #openstack-infra21:45
*** slaweq_ has quit IRC21:50
clarkbI'm drinking mate now21:50
clarkbI think that means the upgrade is serious now21:50
bkeromate++21:51
*** dizquierdo has quit IRC21:51
ianwi think that means something different in .au21:52
clarkbianw: haha21:52
*** esberglu has quit IRC21:52
*** tiswanso has joined #openstack-infra21:52
zaroclarkb: those errors do seem familiar and benign21:53
clarkbhttps://en.wikipedia.org/wiki/Mate_(beverage) that though not really since its actually a tea and mate blend21:53
clarkbzaro: thanks for looking and good to know21:53
zaroi don’t remember if mysql and index can be done at same time.  i thought the sequence was always db migration then reindex?21:55
clarkbzaro: ya that is how we've always done it21:56
zarodb migration is usually pretty quick so probably no benefit to do them in parallel anyways21:57
*** bobh has quit IRC21:57
*** tiswanso has quit IRC21:57
*** dave-mccowan has quit IRC21:58
clarkbzaro: usually except for this time it was 5 hours :/21:58
clarkbok changes is up to 99%21:59
*** mriedem is now known as mriedem_afk21:59
*** andreww has joined #openstack-infra22:03
*** armax has joined #openstack-infra22:03
*** xarses_ has quit IRC22:03
*** rbrndt has quit IRC22:04
mordred\o/22:04
ianwanyone interested in claiming "openstack" for https://keybase.io/blog/introducing-keybase-teams ?22:04
ianwlooks like it's reserved, sensibly22:05
clarkbfungi maybe? I also know someone that works there if we need help22:05
*** xarses_ has joined #openstack-infra22:05
clarkbmordred does too22:05
*** gouthamr has quit IRC22:05
mordredclarkb: I do?22:05
*** dave-mccowan has joined #openstack-infra22:05
clarkbthta is where chromakode is last I checked22:06
mordredoh! neat22:06
mnasermordred while gerrit is down, im trying to look around in shade's code for the potential of this bug.. seen this mentioned in the canadian openstack users slack .. "I have found one problem with my clean keystone urls with the single host, the openstack shade library strips the /image/ path of the url, but just for the glance endpoint, everything else works"22:06
dmsimardsomeone mentioned on hackernews that there are some team names that are reserved, yes22:06
mordredmnaser: yah - I got a bug on that topic22:06
mordredmnaser: https://storyboard.openstack.org/#!/story/200120322:07
mnaserhaha22:07
mnaserits the same person :p22:07
*** andreww has quit IRC22:07
mordredmnaser: it's actually almost certainly actually in ksa - we just recently stopped doing logic related to that in shade directly22:07
mordredmnaser: I havne't gotten a chance to poke and try to reproduce22:07
mnasermordred i'm bored and no gerrit all day => i find out at how much i do openstack daily22:08
mnaserso ill try to see if i can reproduce with a test case22:08
mordredmnaser: sweet! that would definitely be helpful22:08
clarkbmnaser: sorry for the forced boredom :)22:08
clarkbI definitely do not find these upgrades boring myself... quite the opposite22:08
mordredbtw - openstack/publiccloud-wg runs gate-api-wg-python27-ubuntu-xenial in check and gate and not gate-publiccloud-wg-python27-ubuntu-xenial22:09
mordredanybody know if that's a bug or on purpose?22:09
mnaserclarkb no worries, i didnt know how a) useful gerrit was to going back on pending changes and b) how much time i spend working on openstack :p22:09
mordred(saw it while spot-checking shared queues)22:09
jeblairi'm totally going to get my expense report done today, which is the most on-time i will have ever filed one.22:09
clarkbmordred: that seems like a bug at first thought22:09
clarkbdue to how those jobs work22:09
*** chlong has quit IRC22:09
mordredit really does22:09
jeblairprobably copy pasta22:09
*** rbrndt has joined #openstack-infra22:10
mordredI've added it to my stack :)22:10
*** sdague has quit IRC22:10
clarkbjust 3000 changes to go22:12
dmsimardthat's less than 900022:12
* dmsimard tired22:12
bkerojeblair: kudos22:12
*** dave-mccowan has quit IRC22:13
jeblairi have the live view of the reindex in a terminal window, but i *still* use the web browser because cowsay is a *significant* ui improvement!  it's really easy to underestimate the importance of information design.  :)22:15
clarkbI've noticed that I put the new db config in a different location than it is in puppet in the secure.config file. I am going to add a step to the process to move it so that that file doesn't end up being updated unnecessarily by puppet22:16
clarkbthis is my ocd22:16
*** mixos has quit IRC22:16
*** stevebak` is now known as stevebaker22:17
*** claudiub has quit IRC22:17
*** jkilpatr has joined #openstack-infra22:18
clarkbits done22:19
jeblairi stopped the cowsayer22:19
clarkbok I'm moving on to next step of installing plugins22:19
*** Apoorva_ has joined #openstack-infra22:20
*** slaweq_ has joined #openstack-infra22:20
jeblair(screenlog is still enabled, but that seems useful so maybe let's leave it be for now :)22:20
clarkb++22:22
clarkbjeblair: the step to move the new db section in the config will grab the db password in the log22:22
clarkb(so good ot have cowsayer off)22:22
clarkbjeblair: you ready for me to do ^ (eg no expected leakages?)22:22
*** Apoorva has quit IRC22:23
jeblairclarkb: yep.  i chmoded the log too.22:23
clarkbok22:23
*** rlandy is now known as rlandy|brb22:24
clarkbhow does that look?22:24
*** spzala_ has quit IRC22:24
*** spzala has joined #openstack-infra22:24
* clarkb is reenabling the init script now22:25
jeblairclarkb: erm, i'm not up to date on this.  looks generally reasonable though.  :)22:25
clarkbjeblair: the next two steps are to undo your apache changes and to start gerrit, do you want to go ahead and clean up the apache stuff now?22:26
jeblairclarkb: yep.  i can do that if you want22:26
clarkbplease22:26
jeblairclarkb: apache done22:26
clarkbok, everyone ready to start gerrit?22:27
jeblairinfra-root: ^22:27
jeblairclarkb: ++22:27
mordredclarkb: ++22:27
clarkbok here we go22:27
clarkb[2017-09-18 22:28:06,490] [main] INFO  com.google.gerrit.pgm.Daemon : Gerrit Code Review 2.13.9-4-g2a605d5 ready says the log22:28
clarkbmight be a little longer before it is properly ready though22:28
jeblairlemme restart apache to clear any negative cache22:28
clarkbok22:28
jeblairdone and it's accessible now22:29
*** spzala has quit IRC22:29
clarkbI can get it22:30
jeblairmy gertty is happy22:30
clarkbI think we may need more css fixes for the logo stuff22:30
*** felipemonteiro_ has quit IRC22:30
*** Apoorva_ has quit IRC22:30
clarkbbut that is not urgent22:30
jeblairclarkb: what's wrong with the logo?22:30
clarkbit pushes everything to the right so we hvae horizonal scroll bar22:30
*** Apoorva has joined #openstack-infra22:30
clarkbthis was problem before too, guessing the newer css just needs to be accomodated22:31
clarkbI'm going to log out then in22:31
clarkbthat works22:31
jeblairah, i'm anonymous, that explains the discrepancy22:31
clarkbsomeone should push a new change before we get zuulrunning22:32
jeblairmordred: ^22:32
mordredjeblair: I can push changes22:32
clarkbsearch seems to work22:32
*** baoli has joined #openstack-infra22:33
mordredactually - new gerrit seems to have fixed my horizontal scroll issues22:33
clarkbweird maybe its my local browser cache?22:34
mordredmaybe so22:34
mordredclarkb: you wanted me to push up a change real quick?22:34
clarkbyes please22:34
clarkblets make sure that process works and if so I think we can start zuul up again22:34
jeblairheh, there's a bunch of updates right at 22:39, likely from gertty users.  :)22:34
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Rework openstack doc publishing to not be overridable  https://review.openstack.org/50423322:34
mordredI'll keep that at one change for now :)22:35
clarkbI just dropped a test +1 on that change22:35
clarkbseemed to work22:35
jeblairdo we want a new-from-scratch change?22:36
*** baoli_ has joined #openstack-infra22:36
mordredI can push some of those too if we want them22:36
clarkbsure22:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add tag-releases to the mapping  https://review.openstack.org/50450522:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add specs publication jobs  https://review.openstack.org/50450622:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add mapping entries for proposal jobs  https://review.openstack.org/50460822:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add publish-service-types-authority job and mapping  https://review.openstack.org/50460922:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add xstatic-check-version and openstack-tox-pypy  https://review.openstack.org/50461022:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Remove liberty/mitaka job regexes  https://review.openstack.org/50496422:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Remove unmatched single quotes from jenkins jobs  https://review.openstack.org/50496522:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add mapping file setting to skip jobs from share queues  https://review.openstack.org/50496622:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Have publiccloud-wg gate on itself, not api-wg  https://review.openstack.org/50496722:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make yaml2ical publication job  https://review.openstack.org/50496822:36
mordredpatches22:36
mordredyah - my web ui is better re: scrollbars22:37
jeblairgertty is not happy fetching that change22:37
jeblairgit fetch https://git.openstack.org/openstack-infra/project-config +refs/changes/65/504965/1:refs/changes/65/504965/122:37
jeblairthat command doesn't seem to work22:37
*** baoli has quit IRC22:37
jeblairoh22:37
jeblairgit.o.o22:37
jeblairwe're probabbly mirror lagged22:37
clarkboh ya22:37
* clarkb looks22:37
jeblairgit fetch https://review.openstack.org/openstack-infra/project-config +refs/changes/65/504965/1:refs/changes/65/504965/122:38
jeblairthat works fine22:38
jeblairso no worries22:38
clarkb12k tasks, will check that it falls over time22:38
mordredthere's a bunch of replication stuff happening in th elog22:39
clarkbwas 12897 now 1280122:39
jeblairyeah, and the top tasks are changing22:39
clarkbsmall sample shows it trending the right direction22:39
clarkbnow 1233922:40
clarkbdo we want that to settle to zero before starting zuul?22:40
jeblairshould we proceed?22:40
clarkbI think we can proceed if you are ready, this is normal on every gerrit restart not just upgrades22:40
jeblairokay, i'll start zuul now22:41
mordredI'm fine either way22:41
mtreinishooh, you can see all the replecation events on firehose22:42
clarkbalso we'll want to restart gerrit again at some point to pikc up the commentlink fixes22:42
clarkb(probably soon after puppet applies it today22:42
clarkboh except you know what22:42
clarkbgah22:42
clarkbgit0X.o.o are not puppeting which means review.o.o won't puppet22:43
clarkbwe may want to use kick.sh before turning puppet back on globally22:43
jeblairclarkb: why aren't git0x puppeting?22:43
clarkbinfra-root please review https://review.openstack.org/#/c/504838/1 as that is the change we need to get review.o.o puppeting again22:43
clarkbjeblair: we think epel and centos 7.4 are out of sync and causing package install fails22:44
clarkbjeblair: for git0x in particular the package failing is python-paramiko22:44
jeblairzuul is started and changes re-enqueued22:44
clarkbit is also possible its been fixed upstream of us22:44
clarkbI think we want to use kick.sh on review.o.o either way just to see it all happen in the foreground22:44
jeblairclarkb: if we're happy, i think we can clear the alert now22:46
jeblairclarkb: i +2d 83822:46
clarkbcan we get that change applied to gerrit first?22:46
*** dave-mccowan has joined #openstack-infra22:46
clarkbsince I think we want to restart gerrit again afterwards22:46
jeblairoh, yeah, if you want to do that then yes22:46
clarkbmordred: pabelanger do you want to look at https://review.openstack.org/#/c/504838/1 before I approve it?22:46
mordredclarkb: +222:47
clarkbjeblair: ya it will fix some small problems with commentlinks to do so22:47
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Rename tox_upper_constraints_file to tox_constraints_file  https://review.openstack.org/50234822:47
clarkbok puppet change is approved22:47
clarkb9482 gerrit tasks in queue so definitely falling22:48
*** baoli_ has quit IRC22:49
*** baoli has joined #openstack-infra22:49
*** inc0 has quit IRC22:53
clarkbstill waiting on a trusty node to get 838 merged22:53
*** inc0 has joined #openstack-infra22:54
*** spzala has joined #openstack-infra22:55
*** spzala has quit IRC22:55
clarkbmordred: is there an easy way to run kick.sh puppet as noop first before running properly?22:59
clarkbmordred: thinking that will be a good sanity check22:59
mordredclarkb: uhm. ...22:59
mordredclarkb: I dunno? I'd have to go read code23:00
* mordred reads23:00
*** aeng has quit IRC23:00
mordredclarkb: I believe if you add -epuppet_noop=true to the ansible-playbook invocation it will pass noop to puppet23:01
mordredbut we don't expose that in kick ... so you'd just need to run ansible-playbook I think23:01
fungiokay, back from dinner and catching up23:01
*** nicolasbock has quit IRC23:02
clarkbmordred: ya I can edit kick.sh or just run the command it runs directly23:02
clarkbthanks for looking23:02
mordredclarkb: ansible-playbook -f1 --limit review.openstack.org:localhost /opt/system-config/production/playbooks/remote_puppet_adhoc.yaml -epuppet_noop=true23:03
mordredclarkb: COMPLETELY UNTESTED23:03
*** baoli has quit IRC23:03
fungiwow, reindex only took an hour-ish?!?23:04
clarkbfungi: 90 minutes23:04
clarkbfungi: we appear to be mostly in business, just need to get puppet change merged, applied, and gerrit restarted for picking up commentlink changes and sanity check that it can be restarted23:04
*** xarses_ has quit IRC23:05
clarkbmerge should happen shortly23:05
openstackgerritMerged openstack-infra/system-config master: Upgrade review.openstack.org to Gerrit 2.13  https://review.openstack.org/50483823:05
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make yaml2ical publication job  https://review.openstack.org/50496823:05
clarkbmordred: maybe you can test that against review-dev really quickly? Also please do not run puppet against review.o.o manually untilyou have confirmed that ^ is on disk on the puppetmaster23:05
* clarkb is going to work on getting the git repo on puppetmaster updated now23:06
mordredclarkb: testing against review-dev23:06
clarkbok puppetmaster uses git.o.o so have to wait for replication to catch up (or use gerrit as remote, I will just wait as gerrit seems to be quickly catching up on replication requests)23:07
openstackgerritJesse Keating proposed openstack-infra/zuul feature/zuulv3: Move github webhook from webapp to zuul-web  https://review.openstack.org/50426723:09
jeblairclarkb: git pull --ff-only https://review.openstack.org/openstack-infra/system-config ?23:09
clarkbjeblair: does that work?23:09
jeblairya23:09
clarkbI didn't realize you could use raw url on a pull, huh23:09
* clarkb runs that23:09
clarkbthat does indeed work, neat. TIL23:10
*** cody-somerville has joined #openstack-infra23:10
*** cody-somerville has joined #openstack-infra23:10
mordredclarkb: I'm not sure if noop did anything or not23:10
caphrim007is it possible to tell shade to ignore its envvars cloud even if you have envvars?23:10
*** threestrands has joined #openstack-infra23:10
*** threestrands has quit IRC23:10
*** threestrands has joined #openstack-infra23:10
mtreinishhah, new gerrit allows you to remove votes...23:10
clarkbmordred: I think syslog on the node should tell you?23:10
caphrim007for instance, because i use a clouds.yaml for ansible, but env vars for the openstack command itself23:10
mordredcaphrim007: the openstack command supports clouds.yaml fwiw23:10
clarkbmtreinish: ya that was a regression in new change screen iirc23:10
clarkbmtreinish: should only let you remove your own votes?23:11
caphrim007mordred: that might be the answer then23:11
mordredcaphrim007: one sec though - checking  athing for the gerrit upgade23:11
*** csomerville has joined #openstack-infra23:11
mtreinishclarkb: umm, I just removed a powerkvm ci vote on: https://review.openstack.org/#/c/490903/23:11
mordredcaphrim007: if you set OS_CLOUD you can avoid passing --os-cloud to openstack and it won't create an envvars cloud23:11
jeblairthat's a pretty bad regression23:11
* mtreinish will try to remove sdague's +2 now23:11
caphrim007mordred: thanks!23:12
mtreinishhah, that worked too23:12
clarkbin old change screen you could remove your own votes23:12
*** cody-somerville has quit IRC23:12
clarkbremoving any votes seemed to go away in new change screen, I guess they added it back poorly?23:12
clarkbmtreinish: can you try removing a vote on a project you aren't core in? use an infra one say zuul if you want people to be around to add it back23:12
* mtreinish tries a project without core23:12
clarkbit does at least record that you did it23:12
mordredmtreinish: can you do thatanywhere? or only projects in which you have core?23:12
mtreinishclarkb: yeah that was my next test23:12
* mordred has admin, so isn't sure what extra powers that might entail23:13
jeblairmordred: admin has always had remove any vote23:13
mtreinishI don't have an x to remove on neutron patches23:13
mtreinishlet me try to find a project with a patch I own without core23:13
mtreinishI think it'll be on any patch I am the owner for23:14
mtreinishI have the little x's on: https://review.openstack.org/#/c/442125/23:14
clarkbmordred: reading syslog on review-dev I think it may have properly puppet applied23:14
clarkbwhich isn't the end of the world this has been tested on review-dev before review.o.o I just am paranoid23:15
mordredclarkb: properly as in without noop?23:15
clarkbmordred: ya23:15
mordredclarkb: ok. cool23:15
mtreinishyeah, I was able to remove andreaf's +1 there. I don't have d-g core23:15
*** ijw has quit IRC23:15
mordredmtreinish, clarkb, jeblair: there is a permission "remove reviewer"23:15
mordredperhaps we need to add a setting for that which restricts who can do it?23:16
clarkbmordred: ah ok, so probably we just need to update all-projects and let only admins do it23:16
clarkb(and maybe let cores do it on their own projects later23:16
mtreinishmordred: yeah, I think that'll be a good idea23:16
mordredwell - I'm not sure it's the right permission23:16
*** rbrndt has quit IRC23:16
mordredmtreinish: lemme try setting it on a thing where you're core and see if I can remove your x's23:16
mtreinishmordred: ok23:16
clarkboh remove reviewer is probably for the table above votes23:17
mordredyah - I'm guessing that's what it is23:17
clarkbbut maybe it applies to both locations? in any case that is annoying23:17
jeblairhttps://review.openstack.org/Documentation/access-control.html#category_remove_reviewer23:17
jeblair"Change owners can always remove reviewers who have given a zero or positive score (even without having the Remove Reviewer access right assigned)."23:17
mtreinishclarkb: I'm not sure there are any situations where remove vote is the right thing for even a proejct core23:17
*** aeng has joined #openstack-infra23:17
clarkbmtreinish: removing procedural -2s23:17
mtreinishoh, yeah I guess that's true23:18
jeblairit looks like that means that even that permission won't change this new behavior23:18
mtreinishthere have been times where someone is stuck because the original -2 is away23:18
mordredmtreinish: ok - I just set remove reviewer to exclusively administrators on tempest - anyhting change?23:18
clarkbmy initial thought about it is we can live with it because there is a paper trail23:18
clarkbif it comes down to it we can track down offenders23:18
jeblairmordred: ^ (pls continue test, but see above from me)23:18
clarkbmtreinish: are you able to remove negative votes?23:18
mtreinishmordred: I don't have the little x anymore on: https://review.openstack.org/#/c/504728/23:18
mordredjeblair: ah - yes - I see23:18
mtreinishclarkb: it doesn't seem that I can remove any -1s on: https://review.openstack.org/#/c/482759/23:19
clarkbmtreinish: but can remove the +1s?23:19
mordredok. cool. so removing negative votes at least isn't easily available23:19
mtreinishclarkb: yes, the +1s have little x's23:19
mordredclarkb: yah - because he's the change owner23:20
*** tosky has quit IRC23:20
mordredand they are positive or zero as per jeblair's text above23:20
clarkbok makes sense, I think we can live with that even if it is annoying23:20
jeblairmordred, mtreinish: did mordred's change remove the ability to remove -1 ?23:20
mordredmtreinish: can you remove -1'd on tempest?23:20
mordredon changes you own?23:20
jeblair(i wasn't paying attention to -1 vs +1 earlier)23:20
mordredme either23:20
mtreinishmordred: I don't have the x to remove the votes on: https://review.openstack.org/#/c/457369/ (which is a tempest change)23:20
mtreinishjeblair: all the removals before were on +1 and +2s23:20
mordredok.23:20
fungiit's possible we live with owners removing positive votes. while it seems suboptimal i can't immediately think of places where that actually gains anyone anything, even mischief-wise23:21
mtreinishjeblair: I don't seem to have the ability to remove a -1 on patches I own, or patches on proejcts I'm core on23:21
mordredmtreinish: what aobut a +1 on a tempest patch you own?23:22
mordredmtreinish: I just left one on that patch https://review.openstack.org/#/c/457369/23:22
clarkbif we are satisfied with the vote removal behavior I think next step is running puppet and restarting gerrit one last time. There are less than 1k tasks in the gerrit queue now (probably a good idea to let those finish before puppeting and restarting too)23:22
fungisounds reasonable23:22
mtreinishmordred: i just removed your +123:23
jeblairyep23:23
mordredmtreinish: htanks23:23
jeblairmordred: want to undo your change and verify no behavior change with mtreinish?23:23
clarkbhrm I guess one possible concern is that we'd start getting errors for new project replication if the git backends aren't up to date?23:23
mordredjeblair: done23:23
mordredmtreinish: I have removed the permissions setting from tempest23:24
clarkb2017-09-18 12:32:23,123 p=12458 u=root |  git01.openstack.org        : ok=7    changed=2    unreachable=0    failed=1 means that could be the case I think23:24
pabelangersorry, got pulled away for a bit. back now if needed23:24
mordredmtreinish: I expect you to be able to remove +1s and not be able to remove -1s23:24
fungiclarkb: note that restarting gerrit again will start the mirror sync back over too... not necessarily a problem, just something to keep in mind23:24
clarkbfungi: ya23:24
mordredmtreinish: (and I +1'd that patch again just to be sure)23:24
mtreinishmordred: I can still remove your +1 (and did)23:24
mordredok. so you can still do that23:25
*** ijw has joined #openstack-infra23:25
mtreinishalthough the operation seems to be slow (but that's probably just gerrit being sluggish because of all the replications)23:25
mordredand you still can't remove masayukig's vote?23:25
mtreinishmordred: nope, there is no littl x next to masayukig's -1 there23:25
mordredcool23:25
jeblair8cc59d46 waiting .... 23:24:07.035      (retry 1) [6aa59802] push git@github.com:openstack/nova-specs.git23:25
mordredok - we do not need a global permissions change23:25
jeblairmordred: thanks23:26
fungiat least we know nova-specs is in need of repair still23:26
clarkbhttp://git.openstack.org/cgit/openstack/networking-lagopus is the only one we'd need to get in place on git backends that I can see23:26
clarkbI'm going to quickly see if I can somewhat forcefully do that23:27
*** erlon has quit IRC23:27
clarkbI think I can update /home/cgit/projects.yaml then run create-cgitrepos command23:28
jeblairclarkb: i vote for manually updating the commentlinks and leave puppet/restarting for when git.o.o is fixed.23:28
clarkbjeblair: ok that also works for me23:28
fungisure, no objections23:29
jeblair(i'm not opposed to the other; i just mostly don't want to have to fix the world before finishing this maint)23:29
clarkbjeblair: ++23:29
clarkbfungi: is review.o.o in the emergency file?23:29
funginope23:29
*** hichihara has joined #openstack-infra23:29
clarkbdo we want to leave puppet off globally or add review.o.o to the emergency file and enable puppet again?23:29
fungii only disabled puppet globally and then the gerrit initscript23:29
jeblairsuggest adding review/git to emergency23:30
fungiyeah, sounds safe23:30
jeblairor i guess just review should be fine?23:30
clarkbjeblair: just review should be fine23:30
clarkbif someone can get that done I will work on the commentlinks now23:30
jeblairi'll do emergency23:30
*** mixos has joined #openstack-infra23:31
jeblairpabelanger: zuulv3-dev.o.o is dead, right?23:31
pabelangerjeblair: yes23:31
jeblairit still has dns records, we should clean those up23:31
pabelangeragree, I can take care of that23:32
fungioh, right, just remembered it was pointed out earlier i left behind an aaaa for apps.o.o23:32
fungii'll do that and zuulv3-dev.o.o record cleanup now23:32
ianwjeblair: if we want we could try the update-ref stuff from http://lists.openstack.org/pipermail/openstack-dev/2017-June/119166.html on nova-specs to see if we can get rid of this.  although it's one more thing, it might not be bad to make sure it clears up while people are watching23:32
clarkbok comment links config is updated on disk23:33
* mordred needs to step away for a sec - back in a couple23:34
clarkbjeblair: fungi let me know when you are ready for gerrit to be restarted again?23:34
*** hongbin has quit IRC23:34
clarkber s/\?/\./23:34
pabelangerzuulv3-dev.o.o DNS removed23:34
jeblair#status log review.openstack.org added to emergency file until git.o.o puppet is fixed and we can supervise a puppet run on review.o.o23:34
openstackstatusjeblair: finished logging23:34
Sukhdevclarkb : can I have you relook at this please - https://review.openstack.org/#/c/503829/23:34
jeblairclarkb: ready23:34
clarkbok going to stop, check, start now23:35
fungifire at will23:35
*** yamamoto has joined #openstack-infra23:36
clarkbshould be up again23:37
fungipabelanger: you beat me to the zuulv3-dev dns cleanup, but i went ahead and removed the stray apps.o.o aaaa23:37
clarkbapache may need to learn to agree which should happen shortly23:37
pabelangerreview.o.o online for me23:37
clarkbchange link links seem to work now23:38
*** s-shiono has joined #openstack-infra23:38
jeblairmanaged to restart without bumping any gating changes out either :)23:38
clarkbI've updated the etherpad with what needs to be done once puppet is happy again23:39
jeblairstatus ok now?  :)23:39
clarkbI think at this point we are done23:39
clarkbya23:39
jeblairclarkb: you want to issue the command?23:40
*** aeng has quit IRC23:40
*** Sukhdev has quit IRC23:40
clarkbsure23:40
clarkbits just #status ok right?23:40
jeblairclarkb: say something afterwords23:41
jeblairlike "#status ok gerrit is happy, maintenance completed on time, recheck if needed, etc."23:41
clarkb"review.openstack.org Gerrit 2.13 upgrade is functionally complete. The Infra team will be cleaning up bookkeeping items over the next couple days. If you have any questions please let us know"23:41
clarkbhow is ^23:41
jeblairwfm23:42
clarkb#status ok review.openstack.org Gerrit 2.13 upgrade is functionally complete. The Infra team will be cleaning up bookkeeping items over the next couple days. If you have any questions please let us know23:42
openstackstatusclarkb: sending ok23:42
*** david-lyle has quit IRC23:43
clarkbalso super thank you to everyone that helped. I was really worried we'd all be running on caffeine late tonight. We got it done in the allocated time in the end.23:43
clarkbI will make sorting out git backends my priority for tomorrow23:44
*** ChanServ changes topic to "Discussion of OpenStack Developer and Community Infrastructure | docs http://docs.openstack.org/infra/ | bugs https://storyboard.openstack.org/ | source https://git.openstack.org/cgit/openstack-infra/ | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-infra/"23:44
-openstackstatus- NOTICE: review.openstack.org Gerrit 2.13 upgrade is functionally complete. The Infra team will be cleaning up bookkeeping items over the next couple days. If you have any questions please let us know23:45
clarkbjeblair: I didn't notice the git backend thing until this last weekend :/ didn't think at the time there would be any conflict with the gerrit upgrade23:45
fungii'm shocked this completed on time23:45
fungiexcellent planning!23:46
clarkbI'll send a follow up to the mailing list thread shortly as well23:46
jeblairfungi: i think zaro saved us :)23:46
clarkbya reindexing was much quicker than expected23:46
clarkbas for turning on puppet for everything else do we want to do that now or wait for morning?23:47
clarkb(I feel like I've got enough disctractions for finishing up today that that can wait for tomorrow)23:47
jeblairi'd like to afk for now; if other infra-roots will be around and want to babysit that, that's cool.  otherwis, i think it can wait until tomorrow.23:48
clarkblets do it tomorrow then. We managed to finish at a reasonable time lets not find excuses to work late anyways :)23:48
*** jmorgan1 has left #openstack-infra23:49
clarkbjeblair: does statusbot notify when the ok is finished sending?23:50
clarkbI want to put a status log item in about not approving new project changes until git backends are working and I think overlapping notices hsa caused problems in the psat23:51
jeblairclarkb: it should i think...23:51
jeblairapparently not?23:52
jeblairit looks like it's done23:52
clarkbok safe to send #status log then?23:52
pabelangerpuppet for morning works for me23:52
jeblairclarkb: gimme a sec23:52
fungiyup, better the few people who stick around focus on keeping an eye out for reports of broken gerrit behaviors instead23:52
pabelangercan get started earily if people would like23:52
*** Sukhdev has joined #openstack-infra23:52
*** aeng has joined #openstack-infra23:53
*** tiswanso has joined #openstack-infra23:53
*** mriedem_afk is now known as mriedem23:55
fungigertty seems to be working well for me23:55
*** mixos has quit IRC23:58
clarkbok mail sent to the dev list23:58
*** tiswanso has quit IRC23:58
jeblairclarkb: as best as i can tell, statusbot is somehow stuck or broken at the twitter stage.  i do not know if anything further will work.  perhaps the folks who merged the twitter change can help?23:58
fungiclarkb: thanks! i was about to ask if you needed someone to send that23:58
clarkbjeblair: fun :/23:58
* clarkb will just make a note here23:58
jeblairwell, i think *something* needs to be done.23:59
clarkbproject-config reviewers infra-root and AJaeger please avoid merging new project creation changes until after we have the git backends puppeting properly23:59
fungii wonder if twitter integration can be replaced with an rss/atom feed23:59

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