Friday, 2017-07-28

clarkboh nice there is already a compare_digest function00:00
*** thorst has quit IRC00:00
SpamapSclarkb: hooray00:02
SpamapSclarkb: I can probably just add that to this patch.00:02
clarkbSpamapS: kk I think it deserves a comment so that it doesn't get undone00:02
SpamapSclarkb: agreed00:02
clarkbSpamapS: https://docs.python.org/3.5/library/hmac.html#hmac.compare_digest is the function to use00:03
SpamapSperfect00:05
SpamapShrm for some reason logs from the github webhook listener don't end up in test fail00:05
*** kgiusti has quit IRC00:05
*** ianychoi__ is now known as ianychoi00:06
*** iyamahat__ has joined #openstack-infra00:07
*** iyamahat_ has quit IRC00:07
*** skelso has joined #openstack-infra00:09
SpamapSah got it00:09
*** ijw has joined #openstack-infra00:10
*** sekelso has joined #openstack-infra00:11
SpamapSwow some of the github tests are looooooooooong00:11
*** Apoorva_ has quit IRC00:12
*** Apoorva has joined #openstack-infra00:13
SpamapSclarkb: thanks for finding compare_digest!00:13
*** yamamoto has quit IRC00:14
*** skelso has quit IRC00:14
*** gmann has quit IRC00:15
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Github - Require webhook_token  https://review.openstack.org/48824000:15
SpamapSclarkb: ^00:15
*** gmann has joined #openstack-infra00:15
*** yamamoto has joined #openstack-infra00:15
*** jascott1 has joined #openstack-infra00:15
SpamapSand with that, I believe there's an OpenStack birthday party in Pasadena I need to get to. :)00:15
*** annegentle has joined #openstack-infra00:16
*** feisky has joined #openstack-infra00:17
*** feisky has quit IRC00:19
*** mugsie has quit IRC00:19
*** feisky has joined #openstack-infra00:19
*** yamamoto has quit IRC00:20
*** mugsie has joined #openstack-infra00:21
*** mugsie has quit IRC00:21
*** mugsie has joined #openstack-infra00:21
*** feisky has quit IRC00:24
*** feisky has joined #openstack-infra00:24
*** feisky has quit IRC00:26
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Github - Require webhook_token  https://review.openstack.org/48824000:26
*** rama_y_ has quit IRC00:28
*** shu-mutou-AWAY is now known as shu-mutou00:34
*** marst has quit IRC00:34
*** masber has joined #openstack-infra00:35
*** psachin has joined #openstack-infra00:36
*** jascott1 has quit IRC00:37
*** rossella__ has quit IRC00:38
*** LindaWang has joined #openstack-infra00:38
*** aeng has quit IRC00:40
*** rossella__ has joined #openstack-infra00:40
*** aeng has joined #openstack-infra00:43
fungiif zuulv3 is relying on pyca/cryptography anyway, i believe it has some constant-time primitives00:43
fungithough i haven't checked that00:43
fungior, yeah, stdlib ;)00:43
fungii mostly just leave security-sensitive string comparisons to other libraries whose business it is to avoid leaks and side channel attacks of that sort00:45
fungitoo easy to get it wrong, the more eyes on the algorithm and implementation the better00:45
*** yamamoto has joined #openstack-infra00:48
*** liusheng has quit IRC00:52
*** boris-42_____ has joined #openstack-infra00:54
*** boris-42_____ is now known as boris_4200:54
clarkbSpamapS: there are different numbers of 0's in your test secrets. Not that its making me slowly go crazy inside or anything :P00:54
*** masber has quit IRC00:54
fungiadmit it, you were already most of the way there, right?00:54
*** cuongnv has joined #openstack-infra00:56
*** liusheng has joined #openstack-infra00:57
clarkbSpamapS: I left a +2 but I think there are some unnecessary str() conversions happening00:57
clarkbdigest() returns bytes, hexdigest returns string00:59
* fungi agrees with this, having used both extensively01:00
fungihexdigest() is merely a convenience function because lots of people want a string with the hexadecimal version of the digest01:01
fungiit's probably implemented as a one-liner in the lib01:02
boris_42can someone help me with debugging the gates =(  http://logs.openstack.org/17/486017/5/check/gate-rally-dsvm-rally-ubuntu-xenial/0d5e699/console.html#_2017-07-26_04_53_23_134562 it says that there is no Swift here installed here just after we merged https://review.openstack.org/#/c/486749/ and this particalar job is configured with swift01:02
boris_42https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L1302701:02
boris_42here is the job template https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/rally.yaml#L590-L60401:03
boris_42thanks!01:04
openstackgerritMerged openstack-infra/tripleo-ci master: Set ssh_user for oooq mulitnode jobs  https://review.openstack.org/48072301:04
fungihttps://bugs.launchpad.net/trove/+bug/1706528/+subscriptions01:06
openstackLaunchpad bug 1706528 in OpenStack DBaaS (Trove) "Missing ops in Install and configure for Red Hat Enterprise Linux and CentOS in Installation Guide" [Undecided,New] - Assigned to Fan Zhang (fanzhang)01:06
fungier, that was a mispaste01:07
clarkbboris_42: http://logs.openstack.org/17/486017/5/check/gate-rally-dsvm-rally-ubuntu-xenial/0d5e699/logs/local.conf.txt.gz shows that swift is in enabled services then is later explicitly disabled01:07
*** masber has joined #openstack-infra01:08
clarkbthat implies DEVSTACK_GATE_SWIFT was not equal to 101:09
clarkbhowever we see it is equal to 1 in the console log01:09
clarkbhttp://logs.openstack.org/17/486017/5/check/gate-rally-dsvm-rally-ubuntu-xenial/0d5e699/console.html#_2017-07-26_04_34_13_112440 then disables it after checking 1 -ne 101:10
*** harlowja has quit IRC01:10
clarkboh the logic is flipped01:10
clarkbboris_42: your job needs to make that check DEVSTACK_GATE_SWIFT -eq 1 not -ne 101:10
*** hongbin has joined #openstack-infra01:11
clarkband with that dinner01:11
boris_42@clarkb oh fuuu01:12
boris_42@clarkb thanks for help!01:12
boris_42will update it01:12
*** liujiong has joined #openstack-infra01:17
fungigood catch, clarkb! i was looking, totally skimmed past the -ne there01:19
openstackgerritBoris Pavlovic proposed openstack-infra/project-config master: Fix swift enable/disable logic in rally-dsmv-* jobs  https://review.openstack.org/48824801:21
boris_42@clarkb thanks again! I tried to refactor a bit this in all templates ^01:22
boris_42@clarkb so I fixed all templated + removed parameter swift from jobs01:22
boris_42so now it should work as expected01:22
*** baoli has quit IRC01:27
*** baoli has joined #openstack-infra01:28
*** askb has joined #openstack-infra01:30
*** Guest22487 is now known as yland01:31
*** yland has left #openstack-infra01:32
*** ijw has quit IRC01:32
*** gouthamr has quit IRC01:33
*** rossella__ has quit IRC01:39
*** rossella__ has joined #openstack-infra01:42
*** esberglu has quit IRC01:42
*** masber has quit IRC01:50
*** masber has joined #openstack-infra01:52
fungiplaying around with python 3.6, it's a lot pickier about backslash escapes in non-raw strings01:52
*** larainema has quit IRC01:52
fungiloads of deprecationwarnings01:52
*** yamamoto has quit IRC01:53
*** thorst has joined #openstack-infra01:57
*** thorst has quit IRC02:02
*** iyamahat__ has quit IRC02:06
*** esberglu has joined #openstack-infra02:11
*** rama_y has joined #openstack-infra02:12
*** yamahata has quit IRC02:15
*** esberglu has quit IRC02:16
*** Apoorva_ has joined #openstack-infra02:23
*** annegentle has quit IRC02:23
*** Apoorva has quit IRC02:27
*** lbragstad has joined #openstack-infra02:27
*** Apoorva_ has quit IRC02:27
*** lbragstad has quit IRC02:29
*** zhurong has joined #openstack-infra02:31
*** iyamahat has joined #openstack-infra02:39
*** iyamahat has quit IRC02:39
*** iyamahat has joined #openstack-infra02:39
*** vhosakot has quit IRC02:40
*** rama_y has quit IRC02:46
*** markvoelker has quit IRC02:50
*** makowals has quit IRC02:55
smcginnisWe've had someone start to use 3.6 and propose patches for things like the backslash escapes.02:56
smcginnisLooks like it will take some work if we step up from 3.5 to 3.6.02:57
openstackgerritzhurong proposed openstack-infra/project-config master: Publish python-senlinclient docs  https://review.openstack.org/48827003:11
*** makowals has joined #openstack-infra03:11
*** masber has quit IRC03:13
*** baoli has quit IRC03:18
openstackgerritzhurong proposed openstack-infra/project-config master: Publish python-muranoclient docs  https://review.openstack.org/48827003:19
*** gouthamr has joined #openstack-infra03:22
*** thorst has joined #openstack-infra03:23
openstackgerritzhurong proposed openstack-infra/project-config master: Publish python-solumclient docs  https://review.openstack.org/48827303:24
*** armax has quit IRC03:27
*** mriedem_afk is now known as mriedem03:29
*** sekelso has quit IRC03:30
*** zhurong has quit IRC03:31
*** ykarel has joined #openstack-infra03:31
*** ramishra has joined #openstack-infra03:34
*** abelur_ has joined #openstack-infra03:34
*** armax has joined #openstack-infra03:36
*** thorst has quit IRC03:37
*** hongbin has quit IRC03:41
*** rama_y has joined #openstack-infra03:42
*** boris_42 has quit IRC03:43
*** armax has quit IRC03:43
*** armax has joined #openstack-infra03:43
*** armax has quit IRC03:44
*** armax has joined #openstack-infra03:44
*** armax has quit IRC03:44
*** gouthamr has quit IRC03:46
*** yamamoto has joined #openstack-infra03:48
*** masber has joined #openstack-infra03:48
*** yamamoto_ has joined #openstack-infra03:51
*** yamamoto has quit IRC03:53
*** gouthamr has joined #openstack-infra03:58
*** gouthamr has quit IRC03:58
*** yamamoto has joined #openstack-infra03:58
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Use mypy to do static type checking  https://review.openstack.org/48816103:58
*** yamamoto_ has quit IRC03:58
*** gouthamr has joined #openstack-infra03:59
*** makowals has quit IRC04:02
fungismcginnis: good news is those are just warnings and shouldn't break anything until at least 3.8 according to what i was reading04:04
*** nicolasbock has joined #openstack-infra04:08
*** makowals has joined #openstack-infra04:11
*** yamamoto_ has joined #openstack-infra04:16
*** yamamoto has quit IRC04:17
*** links has joined #openstack-infra04:20
*** mriedem has quit IRC04:20
*** makowals has quit IRC04:25
*** yuanying has quit IRC04:28
*** amotoki_ has joined #openstack-infra04:29
*** gouthamr has quit IRC04:30
*** makowals has joined #openstack-infra04:31
*** iyamahat has quit IRC04:32
*** harlowja has joined #openstack-infra04:32
*** yamamoto has joined #openstack-infra04:40
*** yamamoto_ has quit IRC04:41
*** yuanying has joined #openstack-infra04:42
*** makowals has quit IRC04:43
*** sree_ has joined #openstack-infra04:49
*** makowals has joined #openstack-infra04:49
*** sree_ is now known as Guest9151804:49
*** psachin has quit IRC04:50
*** yamamoto has quit IRC04:51
*** markvoelker has joined #openstack-infra04:51
*** makowals has quit IRC04:53
*** yamamoto has joined #openstack-infra04:53
openstackgerritMerged openstack-dev/pbr master: Updated from global requirements  https://review.openstack.org/48798104:54
*** jesusaur has joined #openstack-infra05:00
*** yamamoto has quit IRC05:01
*** makowals has joined #openstack-infra05:01
*** psachin has joined #openstack-infra05:08
*** gyee has quit IRC05:13
openstackgerritMerged openstack/diskimage-builder master: Switch openSUSE to 42.3 by default  https://review.openstack.org/48730405:14
*** dhajare has joined #openstack-infra05:20
*** lewo has joined #openstack-infra05:23
*** markvoelker has quit IRC05:25
*** harlowja has quit IRC05:28
*** zhurong has joined #openstack-infra05:30
*** Diabelko has quit IRC05:40
*** harlowja has joined #openstack-infra05:41
*** thorst has joined #openstack-infra06:00
*** oomichi has quit IRC06:05
*** yuanying has quit IRC06:05
*** oomichi has joined #openstack-infra06:06
*** yuanying has joined #openstack-infra06:06
openstackgerritOpenStack Proposal Bot proposed openstack/diskimage-builder master: Updated from global requirements  https://review.openstack.org/48798906:06
*** thorst has quit IRC06:06
*** yuanying_ has joined #openstack-infra06:07
*** yuanying has quit IRC06:10
*** oomichi has quit IRC06:10
*** oomichi has joined #openstack-infra06:11
*** rcernin has joined #openstack-infra06:15
*** pgadiya has joined #openstack-infra06:18
*** markvoelker has joined #openstack-infra06:22
*** florianf has joined #openstack-infra06:24
openstackgerritNishant Kumar proposed openstack-infra/project-config master: Create new repo cinder tempest plugin  https://review.openstack.org/48630306:28
*** esberglu has joined #openstack-infra06:31
*** esberglu has quit IRC06:36
*** yamamoto has joined #openstack-infra06:36
*** yamamoto has quit IRC06:44
*** yamamoto has joined #openstack-infra06:45
*** aeng has quit IRC06:45
*** lrossetti has quit IRC06:46
*** sshnaidm|off has quit IRC06:47
*** harlowja has quit IRC06:48
*** lrossetti has joined #openstack-infra06:51
*** dtantsur|afk is now known as dtantsur06:51
*** rossella__ has quit IRC06:52
*** jtomasek has joined #openstack-infra06:54
*** rossella__ has joined #openstack-infra06:54
*** markvoelker has quit IRC06:55
*** ccamacho has joined #openstack-infra07:03
openstackgerritBoris Pavlovic proposed openstack-infra/project-config master: Fix swift enable/disable logic in rally-dsmv-* jobs  https://review.openstack.org/48824807:04
*** oomichi has quit IRC07:05
*** oomichi has joined #openstack-infra07:06
*** adisky__ has quit IRC07:06
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Ensure tox-cover is non-voting  https://review.openstack.org/48717307:08
*** masber has quit IRC07:13
*** masber has joined #openstack-infra07:20
*** kaisers has quit IRC07:20
*** amotoki_ has quit IRC07:24
*** dizquierdo has joined #openstack-infra07:25
*** yamamoto has quit IRC07:25
*** yamamoto has joined #openstack-infra07:27
*** kaisers has joined #openstack-infra07:29
*** iyamahat has joined #openstack-infra07:33
*** amotoki_ has joined #openstack-infra07:36
*** kaisers has quit IRC07:37
*** masber has quit IRC07:38
*** iyamahat has quit IRC07:38
*** amotoki_ has quit IRC07:40
*** kaisers1 has joined #openstack-infra07:42
*** ralonsoh has joined #openstack-infra07:44
*** amotoki_ has joined #openstack-infra07:44
*** martinkopec has joined #openstack-infra07:45
*** liujiong has quit IRC07:46
*** egonzalez has joined #openstack-infra07:47
dtantsurhi infra! we are about to remove the ssh drivers from ironic, could you please land https://review.openstack.org/#/c/480624/ to avoid breaking a lot of CI jobs?07:48
dtantsuryolanda: ^^^07:48
*** ykarel is now known as ykarel|lunch07:49
*** markvoelker has joined #openstack-infra07:52
*** yamamoto has quit IRC07:58
*** amotoki_ has quit IRC07:58
*** yamamoto has joined #openstack-infra07:59
*** claudiub has joined #openstack-infra07:59
*** ralonsoh has quit IRC07:59
*** ralonsoh has joined #openstack-infra08:00
*** thorst has joined #openstack-infra08:03
*** strigazi_ is now known as strigazi08:04
*** thorst has quit IRC08:07
openstackgerritMerged openstack-infra/project-config master: Fix swift enable/disable logic in rally-dsmv-* jobs  https://review.openstack.org/48824808:08
*** e0ne has joined #openstack-infra08:13
*** yamamoto_ has joined #openstack-infra08:13
*** adisky__ has joined #openstack-infra08:13
openstackgerritMerged openstack-infra/project-config master: snaps: add snap-cinder definitions  https://review.openstack.org/48527608:15
*** hashar has joined #openstack-infra08:15
*** yamamoto has quit IRC08:16
openstackgerritMerged openstack-infra/project-config master: Adds oswin-tempest-plugin repository  https://review.openstack.org/48609508:16
*** openstackgerrit has quit IRC08:18
*** esberglu has joined #openstack-infra08:19
*** derekh has joined #openstack-infra08:20
AJaegeryolanda: do we have currently any meetings? Otherewise, could you approve https://review.openstack.org/#/c/486907/ , please?08:21
*** dhajare has quit IRC08:22
*** openstackgerrit has joined #openstack-infra08:22
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Fix dynamic layout with regex approval filters  https://review.openstack.org/48833108:22
*** kjackal_ has joined #openstack-infra08:22
*** sambetts|afk is now known as sambetts08:23
*** e0ne has quit IRC08:23
*** esberglu has quit IRC08:23
*** kjackal__ has joined #openstack-infra08:24
*** lucas-afk is now known as lucasagomes08:25
*** alexchadin has joined #openstack-infra08:25
*** markvoelker has quit IRC08:26
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Ignore .mypy_cache  https://review.openstack.org/48833208:26
*** kjackal_ has quit IRC08:27
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Write github app key to a file on disk  https://review.openstack.org/48833308:28
openstackgerritMerged openstack-infra/project-config master: Mark all jobs for python-valenceclient to voting  https://review.openstack.org/48760208:28
openstackgerritMerged openstack-infra/project-config master: networking-midonet: Remove v2-full-legacy experimental job  https://review.openstack.org/48822908:29
openstackgerritMerged openstack-infra/project-config master: networking-midonet: Fix branches for v2 jobs  https://review.openstack.org/48823108:29
openstackgerritMerged openstack-infra/project-config master: Temporarily disable voting on non-rabbit oslo.messaging jobs  https://review.openstack.org/48795108:29
openstackgerritMerged openstack-infra/project-config master: Publish python-muranoclient docs  https://review.openstack.org/48827008:29
openstackgerritMerged openstack-infra/project-config master: Publish python-solumclient docs  https://review.openstack.org/48827308:29
openstackgerritMerged openstack-infra/project-config master: jenkins: openstack-ansible: Add openSUSE 42.3 jobs  https://review.openstack.org/48660208:30
openstackgerritMerged openstack-infra/project-config master: Add gate for testing microservice ansible deployment  https://review.openstack.org/48787508:30
openstackgerritMerged openstack-infra/project-config master: Update Horizon & D_O_A jobs  https://review.openstack.org/48783408:30
*** efoley has joined #openstack-infra08:33
openstackgerritMerged openstack-infra/project-config master: Glare - Zuul/Jenkins update  https://review.openstack.org/48641808:33
*** yuanying_ has quit IRC08:35
*** cshastri has joined #openstack-infra08:35
*** dizquierdo has quit IRC08:35
*** panda is now known as panda|bbl08:35
*** yuanying has joined #openstack-infra08:35
*** yuanying has quit IRC08:35
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Fix dynamic layout with regex approval filters  https://review.openstack.org/48833108:36
*** electrofelix has joined #openstack-infra08:36
*** yuanying has joined #openstack-infra08:36
*** xarses_ has joined #openstack-infra08:37
*** ccamacho has quit IRC08:38
*** ccamacho has joined #openstack-infra08:38
*** lrossetti has quit IRC08:38
*** andreas_s has joined #openstack-infra08:41
*** xarses has quit IRC08:41
*** tuanluong has joined #openstack-infra08:45
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Update SQL reporter to store results  https://review.openstack.org/48822108:52
*** ykarel|lunch is now known as ykarel08:52
*** tosky has joined #openstack-infra09:06
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Update SQL reporter to store results  https://review.openstack.org/48822109:06
*** tosky has quit IRC09:09
*** lrossetti has joined #openstack-infra09:09
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Github - Require webhook_token  https://review.openstack.org/48824009:09
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Update docs to indicate app_key is a file  https://review.openstack.org/48833709:10
*** pgadiya has quit IRC09:13
*** shu-mutou is now known as shu-mutou-AWAY09:15
*** panda|bbl is now known as panda09:16
*** rbergero1 is now known as rbergeron09:17
*** rbergeron has joined #openstack-infra09:17
*** lrossetti_ has joined #openstack-infra09:19
*** lrossetti has quit IRC09:21
*** markvoelker has joined #openstack-infra09:23
*** slaweq has joined #openstack-infra09:26
*** kjackal_ has joined #openstack-infra09:28
*** kjackal__ has quit IRC09:29
*** kjackal___ has joined #openstack-infra09:29
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Update SQL reporter to store results  https://review.openstack.org/48822109:30
*** pgadiya has joined #openstack-infra09:30
*** kjackal_ has quit IRC09:33
*** tosky has joined #openstack-infra09:33
*** liujiong has joined #openstack-infra09:34
*** MatthewLi has quit IRC09:36
*** liujiong has quit IRC09:38
*** hashar has quit IRC09:42
openstackgerritMerged openstack-infra/tripleo-ci master: TLS-everywhere: Explicitly set overcloud_domain_name  https://review.openstack.org/48789909:44
*** liujiong has joined #openstack-infra09:47
*** gcb has quit IRC09:53
*** markvoelker has quit IRC09:57
*** prateek has joined #openstack-infra10:01
openstackgerritSlawek Kaplonski proposed openstack-infra/shade master: Add Neutron QoS dscp marking rule commands  https://review.openstack.org/48419410:02
*** thorst has joined #openstack-infra10:04
*** ykarel_ has joined #openstack-infra10:04
*** slaweq has quit IRC10:04
openstackgerritOpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/48835910:05
*** ykarel has quit IRC10:06
*** esberglu has joined #openstack-infra10:07
*** LindaWang has quit IRC10:07
*** thorst has quit IRC10:09
*** liujiong has quit IRC10:09
*** wolverineav has joined #openstack-infra10:10
*** esberglu has quit IRC10:12
*** pbourke has joined #openstack-infra10:14
pbourkehi, wondering would anyone know why https://review.openstack.org/#/c/471772/ may not be merging despite it having +workflow?10:15
pbourkeit depends on another patch that just merged about 30 mins ago10:15
*** cuongnv has quit IRC10:18
*** ccamacho is now known as ccamacho|brb10:23
*** Guest91518 has quit IRC10:27
*** sree_ has joined #openstack-infra10:28
*** sree_ is now known as Guest5180910:28
*** MatthewLi has joined #openstack-infra10:28
*** tuanluong has quit IRC10:31
*** Guest51809 has quit IRC10:32
*** MatthewLi has quit IRC10:32
*** jkilpatr has quit IRC10:34
dtantsurpbourke: it's a limitation of Depends-On. you need to add another W+1 or remove and re-add the existing one10:37
*** ykarel_ is now known as ykarel|meeting10:37
*** rossella__ has quit IRC10:38
*** arturb has quit IRC10:39
*** e0ne has joined #openstack-infra10:39
*** arturb has joined #openstack-infra10:39
*** LindaWang has joined #openstack-infra10:39
*** yamamoto_ has quit IRC10:40
*** dhajare has joined #openstack-infra10:40
*** rossella__ has joined #openstack-infra10:40
*** kornicameister has quit IRC10:45
*** kornicameister has joined #openstack-infra10:46
*** florianf has quit IRC10:46
*** rossella__ has quit IRC10:48
*** rossella__ has joined #openstack-infra10:51
*** zhurong has quit IRC10:53
*** jkilpatr has joined #openstack-infra10:53
*** markvoelker has joined #openstack-infra10:55
*** sdague has joined #openstack-infra10:55
*** dizquierdo has joined #openstack-infra10:56
*** rossella__ has quit IRC10:58
*** ccamacho|brb is now known as ccamacho10:59
*** florianf has joined #openstack-infra11:00
*** rossella__ has joined #openstack-infra11:02
*** kornicameister has quit IRC11:03
strigaziHello, when is this going to take effect on the CI? our tests are failing11:04
*** masber has joined #openstack-infra11:04
strigaziAJaeger We have a fix for the barbican client (that i didn't menyion here^^ ) but is is reverted https://review.openstack.org/#/c/487678/11:05
smcginnispbourke: Or you can do a recheck, but that will cause it to go through all of the check queue tests first before the gate queue.11:05
strigaziAJaeger We have a fix for the barbican client (that i didn't mention) but it is reverted https://review.openstack.org/#/c/487678/11:05
*** slaweq has joined #openstack-infra11:05
strigaziAJaeger what should we do to unblock our jobs?11:05
*** slaweq has quit IRC11:06
pbourkesmcginnis: dtantsur: good to know, thanks for that11:07
*** dhajare has quit IRC11:09
strigaziAJaeger it is updated a few minutes ago. You are doing a great work infra team!11:12
*** dhajare has joined #openstack-infra11:14
*** rossella__ has quit IRC11:16
*** ykarel_ has joined #openstack-infra11:16
*** pgadiya has quit IRC11:18
*** rossella__ has joined #openstack-infra11:18
*** rama_y has quit IRC11:18
*** ykarel|meeting has quit IRC11:19
*** yamamoto has joined #openstack-infra11:19
AJaegerpbourke: read https://docs.openstack.org/infra/manual/developers.html#limitations-and-caveats11:21
AJaegersmcginnis: recheck will not help with depends-on, it will not see a *new* event11:22
openstackgerritMerged openstack-infra/shade master: Add Neutron QoS dscp marking rule commands  https://review.openstack.org/48419411:22
AJaegerstrigazi: I'm lost wiht your question and assume it's solved ;)11:23
AJaegerstrigazi: changes to requirements repo are in effect by the next time a job *starts*11:23
*** kjackal_ has joined #openstack-infra11:23
strigaziAJaeger Barbican released 4.5.0 which broke a few services/componentes, magnum too.11:24
strigaziAJaeger The releases was removed from the requirements repo here: https://review.openstack.org/#/c/487678/11:24
smcginnisAJaeger: I've run it before where the depended on patch merged. Doesn't it just run the normal checks and see to move along to the gate queue?11:24
*** kjackal___ has quit IRC11:24
*** kjackal__ has joined #openstack-infra11:25
smcginnisAJaeger: At least I could have sworn I've done it.11:25
strigaziAJaeger In some nodes requirements are not synced or updated I guess11:26
AJaegersmcginnis: if those do *not* share a queue, it will not help11:26
*** rossella__ has quit IRC11:27
AJaegerstrigazi: I doubt it. I guess it's either: The job started before teh change merged or the job does ont use upper-constraints11:27
*** markvoelker has quit IRC11:27
strigaziAJaeger I guess the UPPER_CONSTRAINTS_FILE is not updated?11:27
AJaegerstrigazi: best to show me a failing test...11:28
strigaziAJaeger http://logs.openstack.org/50/488350/2/check/gate-magnum-python27-ubuntu-xenial/3141d24/ started 2017-07-28 11:1611:28
strigaziAJaeger upper constraints updated ~2017-07-27 04:4511:29
*** kjackal_ has quit IRC11:29
*** yamamoto has quit IRC11:29
*** rossella__ has joined #openstack-infra11:30
dhellmannthat spambot seems to have finally found the openstack-announce-owner address. are we still blocking @qq addresses?11:30
AJaegerstrigazi: checkout requirements file and look at the history of upper-constraints11:30
AJaegerstrigazi: another change updated the contraint, see I772c591eed6290c61e170e9027532f7f8c182530 .11:31
AJaegerstrigazi: so, you need to downgrade again ;(11:31
*** markus_z has quit IRC11:31
AJaegerfungi, can you do your magic on openstack-announce-owner, please? See dhellmann's comment above11:31
strigaziAJaeger We don't need to downgrade. we have a fix already. Should we merge it? Barbicanclient will remain to 4.5.0? We need to know to act accordingly.11:33
*** dhajare has quit IRC11:35
AJaegerstrigazi: we have an #openstack-requirements that can help you with all these, please ask them. Or talk with the release team on #openstack-release. I cannot help you with these process questions...11:35
strigaziAJaeger I thought there was a sync issue. Thanks for your help Andreas. The requirement changed 3 times in a day...11:36
*** prateek_ has joined #openstack-infra11:37
*** ldnunes has joined #openstack-infra11:37
AJaegeryeah ;(11:38
*** thorst has joined #openstack-infra11:40
*** prateek has quit IRC11:40
openstackgerritSean McGinnis proposed openstack-infra/project-config master: Remove {pipeline}-cinder-dsvm-apache-{node}{suffix}  https://review.openstack.org/47472711:43
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement a static driver for Nodepool  https://review.openstack.org/46862411:43
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an OpenContainer driver  https://review.openstack.org/46875311:43
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Move the fakeprovider module to the fake driver  https://review.openstack.org/48838311:43
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Refactor provider config to driver module  https://review.openstack.org/48838411:43
*** thorst has quit IRC11:44
*** dhajare has joined #openstack-infra11:48
openstackgerritMerged openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/48835911:48
*** yamamoto has joined #openstack-infra11:51
*** rossella__ has quit IRC11:52
*** rossella__ has joined #openstack-infra11:54
*** lucasagomes is now known as lucas-hungry11:54
*** rcernin has quit IRC11:55
*** esberglu has joined #openstack-infra11:55
*** kjackal__ has quit IRC11:59
*** esberglu has quit IRC12:00
*** dhajare has quit IRC12:01
*** Dinesh_Bhor is now known as Dinesh_Bhor|afk12:01
*** dhajare has joined #openstack-infra12:01
*** sekelso has joined #openstack-infra12:01
*** Dinesh_Bhor|afk is now known as Dinesh_Bhor12:02
*** camunoz has quit IRC12:03
*** rlandy has joined #openstack-infra12:04
*** dhajare has quit IRC12:06
*** sekelso has quit IRC12:07
*** trown|outtypewww is now known as trown12:10
*** kjackal_ has joined #openstack-infra12:10
*** rhallisey has joined #openstack-infra12:12
*** adriant has joined #openstack-infra12:12
*** dosaboy has quit IRC12:14
*** rcernin has joined #openstack-infra12:16
*** thorst has joined #openstack-infra12:16
*** camunoz has joined #openstack-infra12:19
*** strigazi is now known as strigazi_AFK12:20
*** jcoufal has joined #openstack-infra12:20
kaisers1Hey! Q for Gerrit Ninjas: Can i list changes that merged in a specific time frame in gerrit? I only find searching for things that changed in a specific time frame but i need merged specifically....12:20
*** rlandy is now known as rlandy|pto12:21
*** jaypipes has joined #openstack-infra12:23
*** jaypipes is now known as leakypipes12:23
*** markus_z has joined #openstack-infra12:23
*** wolverineav has quit IRC12:24
*** markvoelker has joined #openstack-infra12:24
*** markvoelker has quit IRC12:25
*** jangutter has quit IRC12:26
*** skelso has joined #openstack-infra12:26
*** wolverineav has joined #openstack-infra12:28
*** markvoelker has joined #openstack-infra12:29
*** hashar has joined #openstack-infra12:29
*** wolverineav has quit IRC12:32
*** wolverineav has joined #openstack-infra12:35
*** dizquierdo has quit IRC12:36
mnaserkaisers1 i believe gerrit search can do is:merged12:38
mnaserso just AND is:merged12:38
mnaser(but also.. you can look at the git log)12:38
*** wolverineav has quit IRC12:39
*** sree has joined #openstack-infra12:40
kaisers1mnaser: thanks, I'll check that!12:41
*** dhajare has joined #openstack-infra12:41
*** gordc has joined #openstack-infra12:43
*** prateek_ has quit IRC12:43
*** sree has quit IRC12:44
kaisers1mnaser: i feel stupid, git log is by far easier to use for this than gerrit.... :-D12:45
mnaserkaisers1 yeah if you're looking for merged things, and i usually like to browse on github.com commits, easier than reading it off a terminal12:45
*** dhajare has quit IRC12:47
*** slaweq has joined #openstack-infra12:49
*** flepied has quit IRC12:51
openstackgerritAmrith Kumar proposed openstack/diskimage-builder master: Allow users to specify partition type in the MBR PTE  https://review.openstack.org/48239612:54
*** dhajare has joined #openstack-infra12:55
*** esberglu has joined #openstack-infra12:55
*** esberglu has quit IRC12:55
*** esberglu has joined #openstack-infra12:56
openstackgerritRob Cresswell proposed openstack-infra/system-config master: Installation of i18n-checksite.openstack.org  https://review.openstack.org/48793612:56
*** dosaboy has joined #openstack-infra12:57
*** alexchadin has quit IRC12:57
*** wolverineav has joined #openstack-infra12:59
*** esberglu has quit IRC13:01
*** raissa_ is now known as raissa13:03
*** lucas-hungry is now known as lucasagomes13:03
*** jaosorior has quit IRC13:04
*** lbragstad has joined #openstack-infra13:04
*** yamamoto has quit IRC13:05
*** xyang1 has joined #openstack-infra13:06
*** pblaho has quit IRC13:07
*** dhajare has quit IRC13:07
*** kgiusti has joined #openstack-infra13:11
fungiAJaeger: done as of a few minutes ago13:11
AJaegerdhellmann: ^13:12
AJaegerthanks, fungi13:12
*** sree has joined #openstack-infra13:13
*** eharney has joined #openstack-infra13:14
*** esberglu has joined #openstack-infra13:16
*** links has quit IRC13:16
*** prateek has joined #openstack-infra13:16
fungikaisers1: though it is possible to use gerrit's "before" and "after" query parameters to specify a timeframe13:17
*** electrofelix has quit IRC13:18
*** dhajare has joined #openstack-infra13:20
*** akscram has quit IRC13:20
*** jcoufal_ has joined #openstack-infra13:20
*** akscram has joined #openstack-infra13:21
*** Goneri has joined #openstack-infra13:22
*** jcoufal has quit IRC13:23
*** e0ne has quit IRC13:23
*** florianf has quit IRC13:26
*** florianf has joined #openstack-infra13:26
*** rama_y has joined #openstack-infra13:27
*** mriedem has joined #openstack-infra13:28
*** yamamoto has joined #openstack-infra13:29
*** bobh has joined #openstack-infra13:33
*** dhajare has quit IRC13:33
*** baoli has joined #openstack-infra13:34
*** baoli has quit IRC13:34
*** psachin has quit IRC13:35
*** baoli has joined #openstack-infra13:35
*** prateek has quit IRC13:37
*** rcernin has quit IRC13:38
*** e0ne has joined #openstack-infra13:39
*** rossella__ has quit IRC13:42
kaisers1fungi: yeah but afaiu those filter for last update and not explicitely merge time, no? That's why i was looking for other solutions...13:42
*** jistr is now known as jistr|mtg13:45
*** dansmith is now known as superdan13:45
*** bauzas is now known as bauwser13:45
fungikaisers1: correct. for example in the openstack-infra/system-config:tools/owners.py script which we use to identify change owners whose changes merged in a given timeframe, we start with the gerrit "after" as part of the query, but then double-check the submit timestamp for merging against the specified date range and discard any which merged before the specified start time13:46
*** rossella__ has joined #openstack-infra13:46
*** andreas_s has quit IRC13:47
fungikaisers1: the down side to looking at the git log instead is that you're shown the commit dates of patch sets uploaded to gerrit, and if they were a successful fast-forward merge to the branch then there is no separate merge commit to tell you when they actually merged13:48
fungi(though if you also retrieve git notes, you can check the metadata gerrit adds to them to determine that instead)13:48
*** LindaWang has quit IRC13:48
kaisers1fungi: goog info, i already wondered about that.... :-D13:49
kaisers1*good -^13:49
kaisers1fungi: thanks13:50
fungiglad to help13:50
*** akscram has quit IRC13:52
*** weshay is now known as weshay_interview13:54
*** gouthamr has joined #openstack-infra13:54
*** prateek has joined #openstack-infra13:55
*** iyamahat has joined #openstack-infra13:57
*** skelso has quit IRC13:58
*** strigazi_AFK is now known as strigazi_OFF13:58
*** ykarel_ has quit IRC13:58
*** skelso has joined #openstack-infra13:59
*** rcernin has joined #openstack-infra14:00
*** ramishra has quit IRC14:04
*** prateek has quit IRC14:08
*** jamesmcarthur has joined #openstack-infra14:09
*** akscram has joined #openstack-infra14:11
*** yamamoto has quit IRC14:12
*** yamamoto has joined #openstack-infra14:12
*** skelso has quit IRC14:15
*** skelso has joined #openstack-infra14:15
*** slaweq has quit IRC14:17
*** gyee has joined #openstack-infra14:19
*** rbrndt has joined #openstack-infra14:20
mordredmorning mr fungi! can I interest you in a https://review.openstack.org/#/c/488333/ ?14:21
*** sbezverk has quit IRC14:22
*** larainema has joined #openstack-infra14:22
mordredfungi: also - I don't know if you caught it in the scrollback or not, but we identified a need to have a cert on the web interface for zuul v314:22
*** jamesmca_ has joined #openstack-infra14:25
*** armax has joined #openstack-infra14:26
fungimordred: will self-signed work until we get to rename it to zuul.o.o? (we currently pay for a zuul.o.o cert, i just renewed it a few weeks ago)14:26
fungii assume zuulv3.o.o is just a temporary name, anyway14:27
mordredoh - we're probably fine without until we get to zuul.o.o14:27
*** jamesmcarthur has quit IRC14:27
mordredit's a theoretical MITM attack that's super unlikley and we don't gate any github projects anyway - so it's mostly a belt/suspenders thing14:28
openstackgerritMerged openstack-infra/devstack-gate master: Do not use *_ssh ironic drivers by default  https://review.openstack.org/48062414:29
mordredfungi: (I think self-signed actually might not work, since the client of the url would be github)14:29
*** LindaWang has joined #openstack-infra14:30
*** wolverineav has quit IRC14:31
*** baoli has quit IRC14:33
dhellmannfungi : thanks14:33
*** wolverineav has joined #openstack-infra14:33
*** jcoufal has joined #openstack-infra14:33
AJaegerdo we have any meetings happening now ? If not, could an system-config core approve https://review.openstack.org/#/c/486907/ to remove one channel from meetbot, please?14:34
*** marst has joined #openstack-infra14:34
*** jcoufal_ has quit IRC14:34
fungiis that the one i already reviewed?14:34
AJaegerfungi: yes14:34
fungidarn14:34
*** waynr has joined #openstack-infra14:34
AJaeger;)14:35
*** hongbin has joined #openstack-infra14:38
fungii've been trying to keep on top of reviewing the channel additions/removals before i put together the mass removal for remaining dead channels14:39
*** wolverineav has quit IRC14:40
*** xarses has joined #openstack-infra14:42
*** jamesmca_ has quit IRC14:42
*** wolverineav has joined #openstack-infra14:43
fungimordred: if you can test and confirm that a snakeoil cert won't work for testing this, i'll buy a cert for that temporary system14:43
openstackgerritMatthew Treinish proposed openstack/os-testr master: WIP: Switch to stestr under the covers  https://review.openstack.org/48844114:45
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: DNM Random zuul type hint examples  https://review.openstack.org/48844214:45
*** jistr|mtg is now known as jistr14:46
*** Dinesh_Bhor has quit IRC14:46
*** masber has quit IRC14:46
*** annegentle has joined #openstack-infra14:48
openstackgerrityolanda.robla proposed openstack/diskimage-builder master: [DNM] larger LVM testing  https://review.openstack.org/48686414:49
*** sbezverk has joined #openstack-infra14:50
*** harlowja has joined #openstack-infra14:51
*** sdague has quit IRC14:53
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: DNM Random zuul type hint examples  https://review.openstack.org/48844214:55
*** aarefiev is now known as aarefiev_afk14:55
*** ijw has joined #openstack-infra14:57
*** liujiong has joined #openstack-infra14:58
*** dizquierdo has joined #openstack-infra14:58
openstackgerritMerged openstack/os-testr master: Fail when no test case selected  https://review.openstack.org/46649614:58
*** wolverineav has quit IRC15:01
mordredfungi: there is an option to disable cert validation15:01
*** danpawlik has quit IRC15:01
*** waynr has quit IRC15:02
*** waynr has joined #openstack-infra15:02
*** rlandy|pto has quit IRC15:06
*** ykarel_ has joined #openstack-infra15:06
mordredjeblair: https://review.openstack.org/#/c/488333/ I think should finish off yesterday's app_key fun15:07
*** claudiub has quit IRC15:09
*** e0ne has quit IRC15:10
*** egonzalez has quit IRC15:10
*** baoli has joined #openstack-infra15:11
jeblairmordred: lgtm15:11
*** wolverineav has joined #openstack-infra15:11
*** rcernin has quit IRC15:13
mordredjeblair: I just went to try to find an example of a post job run - and I sure did look at a change that hadn't merged15:15
*** weshay_interview is now known as weshay15:15
*** olaph has quit IRC15:16
openstackgerritOpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements  https://review.openstack.org/47804515:17
*** jamesmcarthur has joined #openstack-infra15:17
*** olaph has joined #openstack-infra15:20
*** Goneri has quit IRC15:20
jeblairmordred: that's a rare animal -- the pre-merge post run15:20
mordredit is indeed!15:20
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Update docs to indicate app_key is a file  https://review.openstack.org/48833715:21
*** skelso has quit IRC15:23
*** links has joined #openstack-infra15:24
openstackgerritMerged openstack/os-testr master: Updated from global requirements  https://review.openstack.org/47804515:25
*** skelso has joined #openstack-infra15:25
*** sekelso has joined #openstack-infra15:27
*** skelso has quit IRC15:30
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Ignore .mypy_cache  https://review.openstack.org/48833215:33
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix dynamic layout with regex approval filters  https://review.openstack.org/48833115:33
openstackgerritHongbin Lu proposed openstack-infra/project-config master: [Zun] Move etcd job to experiemental  https://review.openstack.org/47184015:35
openstackgerritPaul Belanger proposed openstack-infra/puppet-zuul master: Add support for github app private pem file  https://review.openstack.org/48850315:37
*** rama_y has quit IRC15:38
AJaegerpabelanger: could you approve https://review.openstack.org/#/c/486907/ to remove one channel from meetbot, please?15:39
openstackgerritPaul Belanger proposed openstack-infra/puppet-zuul master: Add support for github app private pem file  https://review.openstack.org/48850315:39
*** martinkopec has quit IRC15:39
*** Goneri has joined #openstack-infra15:42
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Move github app key to file  https://review.openstack.org/48850415:42
pabelangerAJaeger: is there any meetings going on atm?15:42
pabelangerlikely need to hold off until later today15:42
*** wolverineav has quit IRC15:48
AJaegerpabelanger: ah, release team is meeting until 16:00 UTC but that's the last meeting for today. I didn't know that there was still one, should have checked ;(15:49
*** marst_ has joined #openstack-infra15:50
*** marst has quit IRC15:50
clarkbthe drop off after milestone and freezing stuff is always crazy15:50
*** harlowja has quit IRC15:51
clarkbNow is the time to fix all the bugs we just crammed in :)15:51
fungiif people are willing to +2 any open meetbot channel removals/additions, i'm happy to approve them after the meeting15:51
fungii've been waiting for second +2s on them (pretty sure i've already reviewed them all)15:52
clarkbfungi: are tehy under a common topic?15:52
openstackgerritLin Yang proposed openstack-infra/project-config master: Add python and client libray for Rack Scale Design  https://review.openstack.org/48760115:52
fungii don't think so, but a gerrit query on file: would probably find them all15:52
pabelangerAJaeger: np, we can merge after that15:54
AJaegerclarkb: https://review.openstack.org/#/q/project:openstack-infra/system-config+is:open+file:hiera/common.yaml15:54
clarkbfungi: looks like one removal and ~6 adds? do we have enough room for that?15:54
pabelangerclarkb: mordred: jeblair: care to review: https://review.openstack.org/#/q/topic:zuulv3-github puppet updates for app private pem file15:55
funginope, we have enough for 2 adds after that removal, i think. we can approve the two earliest additions for now and the remaining four get to wait until i put up a mass removal of inactive channels15:56
mordredpabelanger: heh. I already had patch upfor that15:56
fungii just wanted to give the thread a little longer15:56
pabelangermordred: Oh, missed it15:56
mordredpabelanger: https://review.openstack.org/#/c/488333/15:56
* mordred looks at pabelanger 's15:57
*** markus_z has quit IRC15:57
clarkbfungi: https://review.openstack.org/#/c/472741/2 is one you haven't reviewed yet15:57
pabelangermordred: I've abandoned my in favor of yours15:58
mordredpabelanger: I wrote what you wrote first - then got annoyed with the hash vs. parameter15:58
clarkbfungi: and https://review.openstack.org/#/c/447125/2 is a rename so net zero channel adds15:58
fungiclarkb: thanks--i had missed a few of them after all it seems15:58
mordredpabelanger: I think we should actually refactor the whole thign so that puppet-zuul CAN be passed in github info and have it dtrt - but I think that's way more puppet than I want to write today15:59
*** Apoorva has joined #openstack-infra15:59
*** yamamoto has quit IRC15:59
*** dhajare has joined #openstack-infra15:59
*** trown is now known as trown|lunch15:59
*** yamamoto has joined #openstack-infra15:59
clarkbfungi: https://review.openstack.org/#/c/438857/4 too15:59
*** yamamoto has quit IRC15:59
*** ralonsoh has quit IRC16:01
*** LindaWang has quit IRC16:01
mordredpabelanger: also - good morning!16:01
fungi472741 says in the commit message that they wanted statusbot in the channel but they put the entry in the meetbot list instead, so i -1'd16:01
*** thorst_ has joined #openstack-infra16:02
*** dhajare_ has joined #openstack-infra16:02
funginot to mention, i wonder what the point of #openstack-uk is... maybe it's a channel for a local user group? is that common?16:02
pabelangermordred: ya, refactor of puppet-zuul makes sense. one day16:03
*** dhajare has quit IRC16:03
*** wolverineav has joined #openstack-infra16:04
clarkbfungi: ya I think it may just be confusion over what bot performs what function. As for physical region channels dunno how common that is16:04
fungi447125 is only for statusbot too16:05
*** thorst has quit IRC16:05
openstackgerritMerged openstack-infra/system-config master: Write github app key to a file on disk  https://review.openstack.org/48833316:05
fungioh, nevermind, that one's doing both16:05
*** lucasagomes is now known as lucas-afk16:06
*** dizquierdo has quit IRC16:08
*** sree has quit IRC16:09
*** sree has joined #openstack-infra16:09
clarkbwhile this channel is active, I think the gerrit 2.13 testing as gone well. There are three topic:gerrit-upgrade changes, two of which need additional review. https://review.openstack.org/#/c/487156/ is ready for merging, but I'd like to restart production gerrit as soon as that gets in so we aren't surprised if it causes us any problems on 2.11. Will other infra roots be around nowish if I approve16:10
clarkbthat change and restart gerrit this morning?16:10
fungii am around for a while, yes16:10
clarkbmordred: any cahnce you will be around as the mysql expert?16:11
clarkbfungi: sounds good maybe I should just go ahead and approve now then?16:12
fungigo for it16:12
mordredclarkb: I shall be around - I may or may not have useful things to say16:12
clarkbok approving16:12
*** sdague has joined #openstack-infra16:12
fungii'm mostly hacking on election tooling at the moment, to get the structured data for retired projects/deliverables integrated correctly16:13
clarkbI guess I can be really paranoid and do a service stop, db backup, service start too16:13
fungibut will keep an eye on the channel16:13
clarkbI'll do ^16:13
fungisounds great16:13
clarkbwill mean the restart will take a couple minutes though16:13
fungifor review-dev? i doubt anyone cares ;)16:13
clarkbno this is production16:13
clarkbshould I remove my approval?16:13
fungioh! right, production gerrit16:14
fungino, that's fine16:14
fungibut let's #status notice too?16:14
*** sree has quit IRC16:14
clarkbya we can status when we do the service stopping16:14
*** liujiong has quit IRC16:14
fungiand check in with #openstack-release before as well16:14
clarkbload on zuul at least has fallen way off since yesterday with release things16:14
clarkboh ya release may be finishing up will ping them now16:14
*** david-lyle has quit IRC16:14
*** camunoz has quit IRC16:15
*** dhajare_ has quit IRC16:15
fungitechnically feature freeze ended late yesterday, but they could have stragglers/cleanup going on16:15
*** ykarel_ has quit IRC16:15
clarkbya16:16
clarkband tagging and such16:16
*** dhajare_ has joined #openstack-infra16:16
*** david-lyle has joined #openstack-infra16:16
*** rama_y has joined #openstack-infra16:17
openstackgerritMatthew Treinish proposed openstack/os-testr master: WIP: Switch to stestr under the covers  https://review.openstack.org/48844116:19
openstackgerritMerged openstack-infra/system-config master: Move #openstack-opw to #openstack-outreachy  https://review.openstack.org/44712516:20
openstackgerritMerged openstack-infra/system-config master: Removed channel openstack-vmware-nsx  https://review.openstack.org/48690716:20
*** eroux has joined #openstack-infra16:21
*** esberglu is now known as esberglu_lunch16:21
*** armax has quit IRC16:21
clarkbthe other thing we can start thinking about is scheduling the gerrit upgrade16:22
clarkbat this point it will have to be after release I expect16:22
openstackgerritLogan V proposed openstack-infra/project-config master: func_py3 experimental check for openstack-ansible-plugins  https://review.openstack.org/48851616:22
*** david-lyle has quit IRC16:22
clarkbI'd say some time between september 4 and 8 but worry that with zuulv3 prep we won't actually have time to focus on gerrit properly16:23
*** ijw has quit IRC16:23
*** Guest41564 has quit IRC16:23
clarkbwhich probably means september 18 - 22 time frame. Or we do the really crazy thing and do upgrade next week16:23
*** e0ne has joined #openstack-infra16:23
*** ijw has joined #openstack-infra16:24
*** jascott1 has joined #openstack-infra16:27
*** armax has joined #openstack-infra16:28
dmsimardrcarrillocruz: fyi Ansible will end up keeping the old methods that ARA used to configure itself and deprecating them instead for 2.416:28
dmsimardrcarrillocruz: so it should hopefully be less work to get ara to work with 2.4 and allow it to transition to the new config manager16:28
*** e0ne has quit IRC16:28
*** jascott1 has joined #openstack-infra16:28
openstackgerritMerged openstack-infra/puppet-gerrit master: Explicitly enable utf8 on jdbc connection url  https://review.openstack.org/48715616:28
dmsimardIt turns out ARA wasn't the only plugin using those so we weren't the only ones broken by the change16:29
clarkbfungi: mordred ^ ok change is merged, now we wait for puppet to apply it and for dhellmann to give greenlight on restarting gerrit16:29
*** e0ne has joined #openstack-infra16:30
*** camunoz has joined #openstack-infra16:31
*** larainema has quit IRC16:31
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Ensure ref-updated jobs run with their ref  https://review.openstack.org/48821616:32
clarkbok new gerrit config is on place on disk16:33
fungiawesome16:33
clarkbhttps://review.openstack.org/#/c/487154/ is a review-dev only change that will allow me to enable puppet on review-dev again16:34
clarkbbut maybe we worry about that when production is sorted out16:35
jeblairtoo late16:35
clarkbha16:35
jeblairclarkb: do you want me to +0 ?16:35
clarkbjeblair: nah16:35
jeblairk16:35
clarkbits just affecting review-dev and puppet is disabled there currently16:35
clarkbso we can control when it goes in16:35
jeblairyeah seemed safe16:35
rcarrillocruzdmsimard: yah, was chatting with bcoca about that on a team meeting the other day16:37
rcarrillocruzglad your stuff will stabilize :-)16:37
*** dhajare_ has quit IRC16:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Ensure ref-updated jobs run with their ref  https://review.openstack.org/48821616:38
*** [HeOS] has quit IRC16:39
*** dhajare_ has joined #openstack-infra16:40
openstackgerritMerged openstack-infra/system-config master: Explicitly set utf8 on gerrit jdbc url  https://review.openstack.org/48715416:41
*** openstackstatus has quit IRC16:41
*** openstack has joined #openstack-infra16:43
*** openstackstatus has joined #openstack-infra16:43
*** ChanServ sets mode: +v openstackstatus16:43
*** links has quit IRC16:44
*** jbadiapa has quit IRC16:45
openstackgerritMerged openstack/os-client-config master: Updated from global requirements  https://review.openstack.org/48808416:48
*** jcoufal has quit IRC16:53
*** jcoufal has joined #openstack-infra16:53
*** efoley has quit IRC16:54
*** harlowja has joined #openstack-infra16:54
*** derekh has quit IRC16:55
*** harlowja_ has joined #openstack-infra16:55
*** iyamahat has quit IRC16:57
*** Apoorva_ has joined #openstack-infra16:57
*** dtantsur is now known as dtantsur|afk16:57
*** harlowja has quit IRC16:59
*** yamamoto has joined #openstack-infra17:00
*** Apoorva has quit IRC17:00
dhellmannclarkb , fungi : I approved a release job before I noticed your ping here. give me a few minutes and we should be good for a restart17:02
clarkbdhellmann: oh I just pinged you in -release. Thanks and just let us know when you are ready17:03
*** sekelso has quit IRC17:03
*** sambetts is now known as sambetts|afk17:03
dhellmannclarkb : yeah, I just want to make sure the manila client constraint update is submitted and its tests start before we kick gerrit17:03
openstackgerritAndreas Jaeger proposed openstack-infra/system-config master: Sort channels in hiera/common.yaml  https://review.openstack.org/48853417:04
AJaegerfungi: here's a change to sort the channels if you want it ^17:04
*** sekelso has joined #openstack-infra17:04
fungiEmilienM: are you good with https://review.openstack.org/459762 now? it's been in governance for weeks due to the missing depends-on, so we should try to get that merged as soon as is reasonable17:05
dhellmannclarkb : I think we're clear. The jobs haven't started, but zuul has them queued up17:05
clarkbdhellmann: ok17:05
*** hashar has quit IRC17:05
dhellmannclarkb : I should have asked, how long are we likely to be down?17:05
*** EricGonczer_ has joined #openstack-infra17:05
clarkbdhellmann: I expect about 5 minutes17:05
*** slaweq has joined #openstack-infra17:06
dhellmannoh, that's fine then17:06
clarkbI'll wait for the release queue jobs to finish17:06
clarkbsince they are almost done17:06
clarkbdhellmann: basically just long enough to do a database backup without gerrit running17:06
fungibasically long enough to stop the service, dump a copy of the db, then start it again... but keep in mind that after gerrit starts there tends to be a 30+ minute period where its queue is backlogged with replication tasks17:06
clarkbfungi: mordred: can you prep the status bot aloert17:06
openstackgerritPaul Belanger proposed openstack-infra/zuul feature/zuulv3: Fix GithubConnection logging name  https://review.openstack.org/48853717:07
fungiso new refs that merge will have a tendency not to be available on the mirrors for a little while after the restart17:07
fungiand we should probably avoid approving more releases until that clears up17:07
dhellmannyeah, good point17:08
fungiclarkb: how about #status notice The Gerrit service on review.openstack.org is being taken offline for roughly 5 minutes to perform a database backup and reconfiguration17:08
dhellmannthat's fine, I can queue up the branching patches that I need to create and wait a while to approve them17:08
clarkbfungi: that looks great17:09
*** slaweq has quit IRC17:09
fungiclarkb: ready to send that, or wait?17:09
clarkb`/usr/bin/mysqldump --defaults-file=/root/.gerrit_db.cnf --opt --ignore-table mysql.event --all-databases --single-transaction | gzip -9 > gerrit-mysql-20170728.sql.gz` is my db backup command, will run in roots homedir17:09
clarkbfungi: the zuul reelase queue is empty now. I think we can go for it17:10
clarkbonce you tell the bot to send that I will stop gerrit and start the db backup17:10
* clarkb starts a screen17:10
*** yamamoto has quit IRC17:10
*** mriedem is now known as mriedem_afk17:10
openstackgerritMerged openstack-infra/project-config master: Remove channel openstack-vmware-nsx  https://review.openstack.org/48688717:10
fungi#status notice The Gerrit service on review.openstack.org is being taken offline for roughly 5 minutes to perform a database backup and reconfiguration17:11
openstackstatusfungi: sending notice17:11
clarkbscreen is running on review.o.o in my giant terminal width (sorry fungi)17:11
funginp17:11
clarkbok stopping gerrit now17:11
*** ijw has quit IRC17:12
jlvillalNo!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!17:12
*** trown|lunch is now known as trown17:12
jlvillal;)17:12
*** esberglu_lunch is now known as esberglu17:12
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline for roughly 5 minutes to perform a database backup and reconfiguration17:12
clarkbbackup is running17:12
*** iyamahat has joined #openstack-infra17:13
openstackstatusfungi: finished sending notice17:14
*** junbo has quit IRC17:15
AJaegerfungi, shall I retire the inactive channels? Do you want to remove from both meetbot and statusbot?17:15
*** slaweq has joined #openstack-infra17:16
*** yamahata has joined #openstack-infra17:16
fungiAJaeger: i was going to follow up to the ml thread, though i expect we need someone to pop into those channels and link to the thread on the ml and let them know we plan to stop logging them unless they object17:16
fungibecause they may not all be reading the -dev ml (might be operator/user-focused channels, for instance)17:17
clarkbalmost done, about 60MB to go17:17
clarkber 160MB17:17
clarkbok its done17:18
clarkbfungi: anything else we should check before starting the service?17:18
AJaegerfungi, so, proposing the change, replying to mailing list, and pop into channels asking them to -1 - and then let the change sit there for a week?17:18
mordredclarkb: I think you should be good to go17:19
clarkbdouble checked config item is still in the file17:19
clarkbstarting gerrit now17:19
AJaegerI can do that now if you like - but if you want to do it yourself, go for it...17:19
AJaeger... there's no urgency here.17:19
clarkbgonna put a comment on sandbox with snowman17:19
clarkbhttps://review.openstack.org/#/c/488199/1 seemed to work17:20
fungiAJaeger: sounds good to me, i was planning to do it myself but am unlikely to get to it until mid-next week since i'm sort of vacationing right now (i'm doing a pretty terrible job of it, i know)17:21
clarkband errors all appear to be openid expired nonces or ssh exceptions17:21
clarkb(no complaints about the db)17:21
fungiclarkb: start it at your discretion17:21
clarkbI guess we should push code17:21
fungiawesome. i had something i was about to push anyway17:22
clarkbI can fetch successfully17:22
clarkbI'll let you test push17:22
AJaegerfungi, one more question: Remove from meetbot and statusbot - or only meetbot?17:22
*** gyee has quit IRC17:23
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Include retired projects/deliverables in elections  https://review.openstack.org/48854317:25
*** rbrndt has quit IRC17:25
*** gyee has joined #openstack-infra17:25
fungithat seems to have uploaded17:25
clarkbgerrit show queue is task count is falling (though slowly)17:25
openstackgerritAndreas Jaeger proposed openstack-infra/system-config master: Remove inactive meetbot channels  https://review.openstack.org/48854417:25
clarkbdhellmann: I'll let you know when it asymptotically approaches 017:26
fungiAJaeger: probably meetbot, statusbot and gerritbot if the channel is truly unused, but we can keep it in the accessbot config in case we need to reuse the channel at some point17:26
dhellmannclarkb : thanks17:26
AJaegerfungi, so meetbot and statusbot done in 488544 - gerritbot and accessbot I leave for now.17:27
fungithanks AJaeger!17:28
*** tosky has quit IRC17:31
*** openstackgerrit has quit IRC17:33
*** openstackgerrit has joined #openstack-infra17:33
openstackgerritAndreas Jaeger proposed openstack-infra/system-config master: Remove inactive meetbot channels  https://review.openstack.org/48854417:33
*** cshastri has quit IRC17:36
fungii'm approving the oldest couple of meetbot channel addition requests now (which takes us back up to 120)17:38
fungithey were still lingering from march/april17:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Provide nicer index entries for config items  https://review.openstack.org/48758017:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add zuul:value sphinx directive  https://review.openstack.org/48753017:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use default sphinx theme and index attributes  https://review.openstack.org/48723917:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove email-filter requirement  https://review.openstack.org/48760717:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use zuul config sphinx directives for pipeline  https://review.openstack.org/48760417:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Cleanup pipeline requirements  https://review.openstack.org/48761817:38
*** slaweq has quit IRC17:38
*** EricGonczer_ has quit IRC17:39
AJaegerfungi, do you want to take https://review.openstack.org/#/c/488534/ - to sort channels alphabetically?17:40
fungiAJaeger: sure, i just needed to restart my gertty17:40
AJaeger;)17:40
AJaegeris this spam protecting by freenode: "#openstack-zephyr :Targets changing too fast, message dropped" ?17:41
*** Swami has joined #openstack-infra17:42
*** dhajare_ has quit IRC17:43
openstackgerritAndreas Jaeger proposed openstack-infra/system-config master: Remove inactive meetbot channels  https://review.openstack.org/48854417:43
openstackgerritMerged openstack-infra/system-config master: Add #openstack-deployment channel to logging  https://review.openstack.org/43885717:45
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix gerrit label capitalization in docs  https://review.openstack.org/48855017:45
openstackgerritEric Kao proposed openstack-infra/project-config master: Add congress devstack option to jobs  https://review.openstack.org/48855117:45
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Include retired projects/deliverables in elections  https://review.openstack.org/48854317:46
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Invite non-member contributors  https://review.openstack.org/48855217:46
*** Goneri has quit IRC17:46
fungiAJaeger: probably17:47
openstackgerritMerged openstack-infra/system-config master: Add #openstack-helm to meetbot channels  https://review.openstack.org/45574217:47
fungiAJaeger: if we run too many accessbot check jobs in rapid succession they probably get annoyed17:47
AJaegeryeah17:48
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix gerrit label capitalization in docs  https://review.openstack.org/48855017:48
AJaegerfungi, I left a message now everywhere17:48
AJaegersome of them are really silent ;(17:48
*** prad_ has joined #openstack-infra17:50
openstackgerritAndreas Jaeger proposed openstack-infra/system-config master: Remove inactive meetbot channels  https://review.openstack.org/48854417:50
*** jcoufal_ has joined #openstack-infra17:51
*** jcoufal has quit IRC17:53
*** larainema has joined #openstack-infra17:55
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Use foundation member IDs in invitations list  https://review.openstack.org/48855517:56
fungithanks again AJaeger! that's a huge help17:56
AJaegerfungi: you're welcome - enjoy your vacation day!17:57
fungiappreciated!17:58
jeblairfungi: oh are you on vacation today?17:58
fungican't you tell? ;)17:59
*** jamesmcarthur has quit IRC17:59
fungisemi-vacation. entertaining friends and trying to hack on the electoral roll/contributor invite tooling at the same time17:59
jeblairit's... subtle17:59
jeblairfungi: are you entertaining friends *by* hacking on the electoral code?18:00
fungionly marginally, it seems18:00
jeblair"hey watch this!"18:00
clarkb"now you can vote too!"18:00
fungisince yesterday was feature freeze, i'm trying to get the summit contributor invites for non-ptg attendees knocked out now18:01
jeblairlater you can have a condorcet election for what sights to see18:01
*** e0ne has quit IRC18:01
fungiyeah, they'll be excited. we can even compare and contrast the various completion methods18:01
*** vhosakot has joined #openstack-infra18:02
*** gyee has quit IRC18:03
*** gyee has joined #openstack-infra18:06
openstackgerritMerged openstack-infra/system-config master: Sort channels in hiera/common.yaml  https://review.openstack.org/48853418:08
*** ijw has joined #openstack-infra18:08
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Use project object in _uploadPack in gerrit driver  https://review.openstack.org/48856018:09
*** esberglu has quit IRC18:11
*** openstackstatus has quit IRC18:11
*** openstack has joined #openstack-infra18:12
*** openstackstatus has joined #openstack-infra18:12
*** ChanServ sets mode: +v openstackstatus18:12
*** Apoorva_ has quit IRC18:14
*** Apoorva has joined #openstack-infra18:14
*** esberglu has quit IRC18:15
*** gyee has quit IRC18:15
*** tosky has joined #openstack-infra18:16
clarkbdhellmann: gerrit tasks are basically claered out18:25
clarkbdhellmann: current ones are only a few minutes old so no longer backlog from restart18:25
clarkbwith that I am going to shutdown the screen I started for this18:25
*** rbrndt has joined #openstack-infra18:27
*** amotoki is now known as amotoki__18:28
pabelangerclarkb: jeblair: mordred: fungi: what is the process of getting SSL / TLS cert for zuulv3.o.o ?18:28
pabelangerthink we could start the process for that?18:28
clarkbpabelanger: fungi has been purchasing them from COMODO, I think the earlier discussion was that we didn't want a proper cert for that name because it is temporary?18:29
*** ijw has quit IRC18:29
pabelangerah, so maybe just use let's encrypt for now?18:30
*** eumel8 has quit IRC18:30
clarkbfungi and mordred were talking about self signed18:30
pabelangerwould github webhooks work?18:30
mordredyes18:31
pabelangercool18:31
pabelangerself-signed then18:31
mordredpabelanger: self-signed will work fine - when we switch the gh config back to https there is a "disable verification" button18:31
pabelangerlet me read up on gearman SSL process again18:31
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Use project object in _uploadPack in gerrit driver  https://review.openstack.org/48856618:32
*** jkilpatr has quit IRC18:32
*** jamesdenton has quit IRC18:32
*** jamesdenton has joined #openstack-infra18:32
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Use default sphinx theme and index attributes  https://review.openstack.org/48723918:33
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Add zuul:value sphinx directive  https://review.openstack.org/48753018:34
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Provide nicer index entries for config items  https://review.openstack.org/48758018:34
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Use zuul config sphinx directives for pipeline  https://review.openstack.org/48760418:34
*** ijw has joined #openstack-infra18:35
*** ijw has quit IRC18:36
fungipabelanger: mordred indicated in here when we were discussing it earlier today that certificate verification could be disabled from the github side for testing with zuulv3.o.o, we already have a cert we're maintaining for production zuul.openstack.org we can use once we cut over18:36
*** ijw has joined #openstack-infra18:37
fungiah, that's what i get for replying before reading through the rest of the scrollback18:37
fungiand anyway, working github integration is probably fine to not have with zuul.openstack.org on day one of v3 cut-over, if it takes us a few days to work out?18:38
fungiit's not a regression for the openstack community ci at least18:38
*** dave-mccowan has joined #openstack-infra18:38
* AJaeger waves good night18:38
fungihave a good weekend, AJaeger!18:41
*** esberglu has joined #openstack-infra18:41
*** openstackstatus has quit IRC18:42
*** openstack has joined #openstack-infra18:43
*** jkilpatr has joined #openstack-infra18:49
dhellmannclarkb : thanks18:50
*** prad_ has quit IRC18:53
tobiashmordred: what about using let's encrypt instead of a self signed cert for zuulv3.o.o?18:54
*** prad_ has joined #openstack-infra18:55
*** blznblzn2 has joined #openstack-infra18:56
*** rtjure has quit IRC18:58
*** prad_ has quit IRC18:59
*** prad_ has joined #openstack-infra19:00
*** Douhet has quit IRC19:01
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Implement autohold  https://review.openstack.org/48669219:01
*** Douhet has joined #openstack-infra19:02
Shrewsjeblair: w00t for a passing test! ^^^  I've no idea if it's actually correct though  :)19:02
* Shrews waits for others to spot obvious bugs19:03
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Add self-signed cert for zuulv3.o.o  https://review.openstack.org/48857219:04
pabelangerfungi: clarkb: jeblair: mordred: ^ self-signed cert for zuulv3.o.o.  If I understand, I don't need a chain ssl file, because it is untrusted, is that correct?19:05
clarkbpabelanger: the server should already have a snakeoil cert if you want to make it easier19:05
*** dave-mccowan has quit IRC19:05
clarkb(then you don't have to pass around cert contents, just point at the file in apache config)19:05
pabelangerclarkb: Ah, ya. Used our CA server19:05
pabelangertrue19:06
jeblairwe have a ca server?19:07
pabelangerya, we used it for gearman SSL: https://docs.openstack.org/infra/system-config/certificate_authority.html19:07
jeblairpretty sure fungi was adamant that only be used for gearman certs19:07
clarkblooks like the zuul manifest isn't actually built to accept a file path, it only accepts file contents then uses a hardcoded path19:08
pabelangerAh, I see.19:08
clarkbso my easy mode idea isn't quite so easy mode19:08
pabelangerokay, I can revoke the certs and just use snakeoil19:08
jeblairpabelanger: ok.  revoking won't really do anything since nothing checks the revocation list.  i think you can just delete them.19:09
*** wolverineav has quit IRC19:09
jeblair(that's why fungi said we should only use it for one thing, since the only revocation method is to change the ca cert)19:09
pabelangerack19:09
jeblairclarkb, pabelanger: could stick the current snakeoil contents in hiera.  almost as easy.  :)19:09
clarkbjeblair: ya19:09
*** nicolasbock has quit IRC19:12
*** apetrich has quit IRC19:16
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Implement autohold  https://review.openstack.org/48669219:16
*** apetrich has joined #openstack-infra19:16
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Remove email-filter requirement  https://review.openstack.org/48760719:16
*** rtjure has joined #openstack-infra19:17
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Cleanup pipeline requirements  https://review.openstack.org/48761819:17
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix gerrit label capitalization in docs  https://review.openstack.org/48855019:18
*** rtjure has quit IRC19:19
*** rtjure has joined #openstack-infra19:19
*** rtjure has quit IRC19:19
*** rtjure has joined #openstack-infra19:20
*** rtjure has quit IRC19:21
*** rtjure has joined #openstack-infra19:21
*** rtjure has quit IRC19:23
*** rtjure has joined #openstack-infra19:24
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Include retired projects/deliverables in elections  https://review.openstack.org/48854319:25
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Use foundation member IDs in invitations list  https://review.openstack.org/48855519:25
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Check change owners against OSF Individual Members  https://review.openstack.org/48397419:25
*** rtjure has quit IRC19:26
*** lewo has quit IRC19:27
*** rtjure has joined #openstack-infra19:27
openstackgerritTomasz TrÄ™bski proposed openstack-infra/project-config master: Make monasca-persister tempest gate NV  https://review.openstack.org/48858319:27
*** rtjure_ has joined #openstack-infra19:27
*** rtjure has quit IRC19:29
*** rtjure has joined #openstack-infra19:29
*** rtjure_ has quit IRC19:29
*** rtjure has quit IRC19:29
*** jtomasek has quit IRC19:30
*** jtomasek has joined #openstack-infra19:30
*** kjackal_ has quit IRC19:31
*** mriedem_afk is now known as mriedem19:33
*** jtomasek has quit IRC19:34
*** prad_ has quit IRC19:35
*** lewo has joined #openstack-infra19:36
*** rama_y has quit IRC19:38
*** Apoorva has quit IRC19:39
*** EricGonczer_ has joined #openstack-infra19:40
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Implement autohold  https://review.openstack.org/48669219:40
*** Apoorva has joined #openstack-infra19:40
*** jascott1 has quit IRC19:44
*** jascott1 has joined #openstack-infra19:44
*** rama_y has joined #openstack-infra19:45
*** Apoorva has quit IRC19:45
*** liusheng has quit IRC19:45
*** mriedem has quit IRC19:46
*** liusheng has joined #openstack-infra19:46
fungithis week i finally gave into a long-time urge and compiled the latest point release of each python interpreter series from source in my homedir on my dev systems and added them to my path so that i can run tests against versions which aren't packaged in my distro19:48
fungisurprisingly easy and works very well19:48
fungiso now i have the most recent 2.7, 3.3, 3.4, 3.5 and 3.6 point releases i can test code against19:49
*** mriedem has joined #openstack-infra19:50
*** Goneri has joined #openstack-infra19:52
*** camunoz has quit IRC19:54
*** tosky has quit IRC19:54
clarkbI'm reenabling puppet on review-dev now19:56
*** EricGonczer_ has quit IRC19:56
*** EricGonczer_ has joined #openstack-infra19:57
*** rhallisey has quit IRC19:58
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Add project related type hints to gerritconnection  https://review.openstack.org/48859319:59
*** Apoorva has joined #openstack-infra20:00
*** tosky has joined #openstack-infra20:01
*** stevebaker has quit IRC20:02
clarkbpabelanger: was there ever resolution to the yum problems you were having? I don' see any more recent emails from citycloud20:04
*** kjackal_ has joined #openstack-infra20:04
pabelangerclarkb: no, just the email the fungi sent about networking issues20:08
*** e0ne has joined #openstack-infra20:09
pabelangerbut, I think multinode jobs are still disabled on sto220:09
*** e0ne has quit IRC20:14
openstackgerritJames E. Blair proposed openstack-infra/puppet-zuul master: Add support for disk_limit_per_job  https://review.openstack.org/48860020:15
clarkbya I don't think we are running jobs there again yet20:16
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Set zuulv3 executor disk limit to 500MB  https://review.openstack.org/48860220:17
*** olaph has quit IRC20:20
*** boris_42 has joined #openstack-infra20:20
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Monitor job root and kill over limit jobs  https://review.openstack.org/48590220:22
*** olaph has joined #openstack-infra20:22
*** Douhet has quit IRC20:23
*** Douhet has joined #openstack-infra20:24
clarkbreview-dev has been puppeted20:27
clarkbfungi: so that last major piece is ripping out the cla contact store stuff which you have been working on today \o/20:27
*** EricGonczer_ has quit IRC20:30
mordred\o/20:36
*** leakypipes has quit IRC20:36
*** baoli has quit IRC20:37
*** sdague has quit IRC20:37
fungiclarkb: yeah, i mean basically we can do that at any time but it should probably ideally wait until at least ptl electorate deadline to make sure new contributors get to vote, since they may not see the updated account setup documentation warning them otherwise20:38
jeblairmordred, clarkb, fungi, pabelanger: i just merged SpamapS's change to kill jobs that use lots of disk.  we should merge https://review.openstack.org/488602 and dependent soon so we don't trip over that next time we restart zuulv3.20:48
*** florianf has quit IRC20:48
jeblair(default is 250, so we're probably ok, but still i think we can do more)20:48
clarkbwhen I was paying much more attention to production per job usage we got it down to ~100MB as a realistic size for multinode jobs20:49
clarkbtripleo floats around that number now too20:49
jeblairclarkb: oh good.  this does also include some amount of space consumed by the git repos, but hard links help a lot with that.20:50
clarkbI think 200MB ~double our current high number should work well for all but jobs that upload images20:51
clarkbkolla's imge builds are like 2.5GB iirc20:51
jeblairyeah, so if they transit through the executor, we'll have a problem.  we can either bump our limit way up, or make sure those don't go through the executor.20:51
mordredjeblair: I'd like to suggest that any time we put a list or a dict into an arg list (which we do on purpose sometimes) - that we put in a comment saying it's on purpose20:51
mordredjeblair: because I just happened across an instance I'm pretty sure is a bug20:52
mordredbut I know we do it on purpose sometimes - so I'm not 100% sure20:52
jeblairmordred: we talking puppet or something else?20:52
jeblairenocontext20:52
mordredjeblair: gah. sorry - wrong channel20:52
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix broken project config in dynamic config test  https://review.openstack.org/48622620:53
*** weshay is now known as weshay_pto20:53
*** trown is now known as trown_pto20:54
pabelangerwfm20:56
clarkbeven gerrit wars are less than 60MB iirc20:57
*** kgiusti has left #openstack-infra21:01
*** marst has joined #openstack-infra21:03
*** iyamahat_ has joined #openstack-infra21:03
openstackgerritOpenStack Release Bot proposed openstack/os-client-config master: Update reno for stable/pike  https://review.openstack.org/48866321:04
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Replace singleton lists with None defaults  https://review.openstack.org/48867621:04
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add an AFS keytab to the bind-mount if it's there  https://review.openstack.org/48867721:04
*** marst_ has quit IRC21:05
*** iyamahat has quit IRC21:05
openstackgerritMerged openstack-infra/puppet-zuul master: Add support for disk_limit_per_job  https://review.openstack.org/48860021:06
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: WIP Don't ignore inexistent jobs in config  https://review.openstack.org/48875821:06
openstackgerritMerged openstack-infra/system-config master: Set zuulv3 executor disk limit to 500MB  https://review.openstack.org/48860221:07
*** david-lyle has joined #openstack-infra21:07
*** yamamoto has joined #openstack-infra21:08
openstackgerritOpenStack Release Bot proposed openstack-infra/shade master: Update reno for stable/pike  https://review.openstack.org/48883121:08
*** thorst_ has quit IRC21:09
*** sree_ has joined #openstack-infra21:10
*** sree_ is now known as Guest68821:10
*** yamamoto has quit IRC21:12
*** bobh has quit IRC21:12
*** slaweq has joined #openstack-infra21:13
*** Guest688 has quit IRC21:14
*** ldnunes has quit IRC21:15
*** yamamoto has joined #openstack-infra21:16
fungii wonder what alternatives we have for large image uploading jobs21:18
clarkbin kolla case they could use zuul secrets to upload directly to dockerhub21:19
fungii guess put creds on the worker from a secret and then have it upload directly21:19
fungiyeah, that21:19
clarkbsince we won't need to do the temp dance anymore21:19
fungiwon't that be a nice change of pace21:20
*** yamamoto has quit IRC21:20
*** gordc has quit IRC21:21
*** gouthamr has quit IRC21:24
*** david-lyle has quit IRC21:26
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Change name and document the bind_mount config paths  https://review.openstack.org/48867721:30
*** esberglu has quit IRC21:34
*** [HeOS] has joined #openstack-infra21:36
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Change name and document the bind_mount config paths  https://review.openstack.org/48867721:36
openstackgerritMonty Taylor proposed openstack-infra/puppet-zuul master: Change bind mount option names to match zuul  https://review.openstack.org/48885921:37
*** gouthamr has joined #openstack-infra21:37
openstackgerritMonty Taylor proposed openstack-infra/puppet-zuul master: Add paths bind mount option names to match zuul  https://review.openstack.org/48885921:41
openstackgerritMonty Taylor proposed openstack-infra/puppet-zuul master: Remove the old _dirs option  https://review.openstack.org/48886121:41
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Use trusted_ro_paths  https://review.openstack.org/48886221:41
mordredclarkb, jeblair: ^^21:41
mordredthatdoes the expand/contract and should pass puppet tests21:41
clarkbmordred: it complains about merge conflicts21:42
mordredsigh21:42
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Use trusted_ro_paths  https://review.openstack.org/48886221:43
mordredclarkb: thanks. fixed21:43
mordredGAH. sorry - I caught anothe piece in there21:44
clarkbmordred: where did execturo ya that21:44
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Use trusted_ro_paths  https://review.openstack.org/48886221:46
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Write AFS keytab onto zuul v3 executor nodes  https://review.openstack.org/48886821:46
mordredok. THAT should be good now21:46
* mordred goes to move hiera data21:47
*** slaweq has quit IRC21:49
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Write AFS keytab onto zuul v3 executor nodes  https://review.openstack.org/48886821:52
*** annegentle has quit IRC21:52
mordredclarkb, jeblair: ^^ I have copied the launcher-keytab into a hiera group file for that new group. once the patch has landed that puts the executors and launchers into that group, I'll remove it from the merger group so it's only in there once21:54
*** yamamoto has joined #openstack-infra21:55
clarkbmordred: can hosts be in multiple groups at the same time and have the hiera stuff work?21:55
clarkbiirc the $group var in the site.pp code determines hiera lookup paths21:56
mordredclarkb: oh. right. because of our mixed ...21:57
mordredclarkb: k. I'll do that different...21:57
clarkbmordred: I left comment on change too21:58
clarkbya that will work as far as asnible is concerned but won't make hiera do what you want21:58
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Write AFS keytab onto zuul v3 executor nodes  https://review.openstack.org/48886822:00
*** thorst has joined #openstack-infra22:01
mordredwell heck - actually - let's make that cleaner since I have to make a copy22:01
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Ensure ref-updated jobs run with their ref  https://review.openstack.org/48821622:01
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Write AFS keytab onto zuul v3 executor nodes  https://review.openstack.org/48886822:01
*** slaweq has joined #openstack-infra22:02
*** Goneri has quit IRC22:02
mordredclarkb: ok. I have put the data into the executor group vars file, have removed the zuul-worker group var file and have named the new variable as it is in that patch22:03
openstackgerritMerged openstack-infra/puppet-zuul master: Add paths bind mount option names to match zuul  https://review.openstack.org/48885922:04
*** thorst has quit IRC22:05
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Log an extra blank line to get space after each skip  https://review.openstack.org/48669822:06
*** esberglu has joined #openstack-infra22:07
*** slaweq has quit IRC22:08
*** stevebaker has joined #openstack-infra22:08
*** jcoufal has joined #openstack-infra22:09
*** xarses has quit IRC22:10
*** esberglu has quit IRC22:11
*** jcoufal_ has quit IRC22:11
*** yamamoto has quit IRC22:12
*** iyamahat_ has quit IRC22:12
*** bobh has joined #openstack-infra22:14
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Add spacer line after loops  https://review.openstack.org/48589722:16
*** sekelso has quit IRC22:18
*** jcoufal has quit IRC22:18
*** jcoufal has joined #openstack-infra22:18
*** slaweq has joined #openstack-infra22:21
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add a short name to the project in the inventory  https://review.openstack.org/48887722:22
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: WIP Add Infra AFS docs publication job  https://review.openstack.org/48887822:23
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Use project.short_name instead of jinja filter  https://review.openstack.org/48887922:23
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Log an extra blank line to get space after each skip  https://review.openstack.org/48669822:23
mordredjeblair: https://review.openstack.org/488879 is the thing that made me want https://review.openstack.org/48887722:24
*** jcoufal has quit IRC22:24
openstackgerritGoutham Pacha Ravi proposed openstack-infra/project-config master: Add Rally manila jobs to python-manilaclient  https://review.openstack.org/48888022:25
*** xarses_ has quit IRC22:25
*** bobh has quit IRC22:25
*** slaweq has quit IRC22:27
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Change name and document the bind_mount config paths  https://review.openstack.org/48867722:30
*** thorst has joined #openstack-infra22:31
jeblairmordred: cool.  i made some comments on 4888777 and 488887822:32
mordredjeblair: thanks!22:32
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Use project.short_name instead of jinja filter  https://review.openstack.org/48887922:34
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: WIP Add Infra AFS docs publication job  https://review.openstack.org/48887822:34
*** vhosakot has quit IRC22:35
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add a short name to the project in the inventory  https://review.openstack.org/48887722:35
*** pbourke has quit IRC22:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove zuul:configobject  https://review.openstack.org/48888622:48
*** pbourke has joined #openstack-infra22:48
*** jcoufal has joined #openstack-infra22:48
*** marst has quit IRC22:48
*** eroux has quit IRC22:52
*** yamahata has quit IRC22:54
*** rbrndt has quit IRC22:58
*** inc0 has quit IRC23:00
*** inc0 has joined #openstack-infra23:00
*** dangers_` is now known as dangers_away23:11
*** yamamoto has joined #openstack-infra23:13
*** jcoufal has quit IRC23:17
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add a short name to the project in the inventory  https://review.openstack.org/48887723:20
*** yamamoto has quit IRC23:21
*** slaweq has joined #openstack-infra23:24
*** tosky has quit IRC23:29
*** hongbin has quit IRC23:30
*** rtjure has joined #openstack-infra23:32
*** [HeOS] has quit IRC23:32
*** [HeOS] has joined #openstack-infra23:32
*** slaweq has quit IRC23:43
*** gagehugo has quit IRC23:50
*** gagehugo has joined #openstack-infra23:56
*** markvoelker has quit IRC23:56
*** xarses_ has joined #openstack-infra23:57

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