Wednesday, 2017-12-20

*** lukebrowning has quit IRC00:01
*** akscram1 has quit IRC00:01
*** akscram1 has joined #openstack-infra00:02
openstackgerritIan Wienand proposed openstack-infra/system-config master: Remove infra-files-ro and infra-files-rw  https://review.openstack.org/52918900:02
jeblairpabelanger, ianw: the nodepool fixes are failing check.  can one of you start working on manually reverting to a20d81c32c7caeb330bbc6b7373ddb975c533e20 on the launchers and restarting?00:02
ianwpabelanger: there is now :)00:02
*** lukebrowning has joined #openstack-infra00:02
jeblairi have to run right now, but can check back in a bit00:03
ianware they emergency-ed?00:03
jeblairianw: no00:03
ianwpabelanger: i can ... just let me know so we don't overwrite each other00:03
jeblair(remember to pip3 install nodepool -- not pip install)00:03
pabelangerianw: +200:03
pabelangerianw: go for it00:04
ianwok, first thing i'll put them in emergency00:04
*** salv-orlando has quit IRC00:05
*** salv-orlando has joined #openstack-infra00:05
*** lukebrowning has quit IRC00:07
*** edmondsw has joined #openstack-infra00:08
*** wolverineav has quit IRC00:08
ianwa20d81c32c7caeb330bbc6b7373ddb975c533e20 is the feature/zuulv3 branch, right?00:09
ianwtot00:09
pabelangerianw: yah, feature/zuulv300:10
*** salv-orlando has quit IRC00:11
ianwis that not just what puppet installs?00:11
pabelangeryah, currently00:12
ianwoh right, jeblair installed some changes manually00:12
pabelangeroh, I see what you are saying00:13
*** edmondsw has quit IRC00:13
ianwok, i've re-installed 3e20 on nl01, will restart the launcher now00:13
pabelangerk00:14
ianwok, that's up, moving on to nl0200:14
ianw... done.  so both nl01 & nl02 are running nodepool-launcher from feature/zuulv3 TOT (...3e20)00:16
pabelangerkk00:16
pabelangershould be safe to remove from emergency00:17
ianwdo we want to roll back further though, prior to quota changes?00:17
pabelangerI think it is safe, we might just not have rax00:18
pabelangerwhich, should be okay until tomorrow00:18
pabelangeror patches land00:18
pabelangerodd, I see node_failure on status.o.o00:18
pabelangermight be first time I have seen that00:19
ianwpabelanger: ok, both removed from emergency since we're deploying from the branch00:19
*** lukebrowning has joined #openstack-infra00:19
pabelangerwfm00:20
*** Vi has joined #openstack-infra00:20
ianwi'm guessing we could revert back to https://git.openstack.org/cgit/openstack-infra/nodepool/commit/?h=feature/zuulv3&id=afcb56e0fb887a090dbf1380217ebcc06ef6b66b00:21
*** Vi is now known as Guest8023300:21
ianw(just based on merge times from https://git.openstack.org/cgit/openstack-infra/nodepool/log/?h=feature/zuulv3)00:21
ianwbut i will just hold tight until further instructions :)00:22
pabelangeryah00:23
*** Guest80233 has quit IRC00:23
*** lukebrowning has quit IRC00:24
*** lukebrowning has joined #openstack-infra00:26
*** threestrands has joined #openstack-infra00:26
*** threestrands has quit IRC00:26
*** threestrands has joined #openstack-infra00:26
*** lukebrowning has quit IRC00:30
*** xarses_ has quit IRC00:31
*** bobh has joined #openstack-infra00:32
*** lukebrowning has joined #openstack-infra00:32
jeblairoh shoot00:33
jeblairi gave you the wrong hash00:33
jeblairshould have been afcb56e0fb887a090dbf1380217ebcc06ef6b66b00:33
jeblairbecause, indeed, the point was not to run the branch tip which is broken00:33
jeblairi'll revert back to that00:34
*** dave-mccowan has quit IRC00:35
jeblairdone00:36
*** lukebrowning has quit IRC00:37
jeblair#status log nl01 nl02 manually downgraded to afcb56e0fb887a090dbf1380217ebcc06ef6b66b due to broken quota handling on branch tip00:37
openstackstatusjeblair: finished logging00:37
*** esberglu has quit IRC00:38
ianwjeblair: heh, i *thought* that was what you might have meant :)00:38
*** lukebrowning has joined #openstack-infra00:38
ianwi'll put them in the emergency file now, to avoid puppet updating that?00:39
*** wolverineav has joined #openstack-infra00:39
jeblairianw: i lean slightly toward leaving it alone, so that config changes still happen; they won't automatically restart or anything, so it's okay as long as they aren't rebooted.  and we'll hopefully get the fix in by tomorrow anyway00:41
jeblairianw: but if you would rather add to emergency, that's fine.  i could go either way.   :)00:41
ianwi won't be around tomorrow, so let's leave it!00:42
*** lukebrowning has quit IRC00:43
*** wolverineav has quit IRC00:44
*** jamesmcarthur has joined #openstack-infra00:44
*** lukebrowning has joined #openstack-infra00:45
*** caphrim007_ has joined #openstack-infra00:47
*** kjackal has quit IRC00:47
*** kjackal has joined #openstack-infra00:48
*** lukebrowning has quit IRC00:49
*** caphrim007 has quit IRC00:50
*** lukebrowning has joined #openstack-infra00:51
*** caphrim007_ has quit IRC00:52
*** lukebrowning has quit IRC00:56
*** bobh has quit IRC00:56
*** kiennt26 has joined #openstack-infra00:56
*** lukebrowning has joined #openstack-infra00:57
openstackgerritJames E. Blair proposed openstack-infra/nodepool feature/zuulv3: Log provider names with quota  https://review.openstack.org/52917801:00
openstackgerritJames E. Blair proposed openstack-infra/nodepool feature/zuulv3: WIP: Fail on quota-exceeded (partial revert)  https://review.openstack.org/52917301:00
openstackgerritJames E. Blair proposed openstack-infra/nodepool feature/zuulv3: Assume a quota limit of -1 means unlimited  https://review.openstack.org/52918001:00
*** jamesmcarthur has quit IRC01:01
*** lukebrowning has quit IRC01:01
*** lukebrowning has joined #openstack-infra01:03
fungicatching up on quota excitement now01:04
*** salv-orlando has joined #openstack-infra01:06
openstackgerritMerged openstack-infra/system-config master: Remove jenkins related puppet manifests  https://review.openstack.org/52914501:07
openstackgerritMerged openstack-infra/system-config master: Remove additional dead jenkins related code  https://review.openstack.org/52915001:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Use yarn and webpack to manage zuul-web javascript  https://review.openstack.org/48753801:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add babel transpiling enabling use of ES6 features  https://review.openstack.org/52829501:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add StandardJS linting and analysis  https://review.openstack.org/52829601:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Fix source_url handling for jobs view  https://review.openstack.org/52837301:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Fix StandardJS warnings and turn them to errors  https://review.openstack.org/52829701:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add bundle analysis to the lint target  https://review.openstack.org/52829801:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Make bundle of build web content  https://review.openstack.org/52837401:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Remove use strict  https://review.openstack.org/52843701:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Inject url endpoint information  https://review.openstack.org/52919301:07
*** lukebrowning has quit IRC01:08
jeblairanyone understand time.monotonic and why that's different in the gate vs on my machine?01:10
*** wolverineav has joined #openstack-infra01:10
mordredugh - I feel like I did understand time.monotonic at one point01:10
*** salv-orlando has quit IRC01:11
*** dhill_ has quit IRC01:11
mordredjeblair: I'm assuming you've seen "The reference point of the returned value is undefined, so that only the difference between the results of consecutive calls is valid." already01:11
jeblairmordred: yeah, i think that's the thing at issue01:11
*** dave-mccowan has joined #openstack-infra01:11
jeblairit's probably not worth understanding why the reference time is always low in the gate and high locally, and instead just rework the whole thing.01:12
*** cuongnv has joined #openstack-infra01:12
mordredjeblair: I wonder if that reference time has something to do with clocks in vms01:12
mordredjeblair: like - I agree, it's probably more better to rework the thing, although I'd also find it intellectually interesting to understand what was the cause of different reference points01:13
jeblairmordred: yeah, i would probably have agreed a couple of hours ago :)01:14
jeblairgood news -- this test is now capable of exposing 2 bugs01:15
mordredjeblair: woot!01:15
mordredjeblair: that makes it an extra good test01:15
openstackgerritJames E. Blair proposed openstack-infra/nodepool feature/zuulv3: Fail on quota-exceeded (partial revert)  https://review.openstack.org/52917301:17
openstackgerritJames E. Blair proposed openstack-infra/nodepool feature/zuulv3: Assume a quota limit of -1 means unlimited  https://review.openstack.org/52918001:17
jeblairokay, i think those have a chance of passing now01:17
*** wolverineav has quit IRC01:17
*** asettle has quit IRC01:19
*** lukebrowning has joined #openstack-infra01:19
clarkbso the < comparison wasnt working with now being monotonic?01:20
*** yamamoto has joined #openstack-infra01:21
*** yamamoto has quit IRC01:21
jeblairclarkb: yeah, in the gate we were getting values like "243" for now.  243 < 0 + 300 so we used the invalid cache.01:21
*** asettle has joined #openstack-infra01:22
clarkbok I reapproved the two that lost their +As and +2'd the one with monotonic01:23
*** annp has joined #openstack-infra01:23
*** psachin has joined #openstack-infra01:23
fungii'll take a look01:24
*** lukebrowning has quit IRC01:24
fungitc office hour is pretty dead tonight01:24
*** jamesmcarthur has joined #openstack-infra01:25
*** lukebrowning has joined #openstack-infra01:26
*** cshastri has joined #openstack-infra01:26
*** Apoorva_ has joined #openstack-infra01:26
*** psachin has quit IRC01:28
*** liujiong has joined #openstack-infra01:29
*** Apoorva has quit IRC01:29
*** lukebrowning has quit IRC01:30
*** Apoorva_ has quit IRC01:31
openstackgerritMerged openstack-infra/project-config master: Add accessbot for #openstack-vpnaas  https://review.openstack.org/52786601:32
*** psachin has joined #openstack-infra01:32
*** lukebrowning has joined #openstack-infra01:32
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Inject url endpoint information  https://review.openstack.org/52919301:34
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Make bundle of build web content  https://review.openstack.org/52837401:34
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Remove use strict  https://review.openstack.org/52843701:34
*** lukebrowning has quit IRC01:37
*** yamahata has quit IRC01:38
*** lukebrowning has joined #openstack-infra01:38
*** yamahata has joined #openstack-infra01:39
openstackgerritMerged openstack-infra/nodepool feature/zuulv3: Log provider names with quota  https://review.openstack.org/52917801:41
*** lukebrowning has quit IRC01:43
*** makowals has quit IRC01:43
*** andreykurilin has quit IRC01:43
*** makowals has joined #openstack-infra01:44
*** lukebrowning has joined #openstack-infra01:44
*** jd_ has quit IRC01:45
*** _ari_ is now known as ari|pto01:46
*** andreykurilin has joined #openstack-infra01:46
*** jd_ has joined #openstack-infra01:47
*** rmcall has quit IRC01:47
*** lukebrowning has quit IRC01:49
*** lukebrowning has joined #openstack-infra01:51
*** claudiub|2 has quit IRC01:53
*** lukebrowning has quit IRC01:55
*** liujiong has quit IRC01:56
*** liujiong has joined #openstack-infra01:56
*** jamesmcarthur has quit IRC01:57
*** lukebrowning has joined #openstack-infra01:57
*** yamamoto has joined #openstack-infra01:57
*** felipemonteiro has quit IRC01:57
*** caphrim007 has joined #openstack-infra01:59
*** lukebrowning has quit IRC02:02
*** iyamahat has quit IRC02:03
*** hoangcx has joined #openstack-infra02:03
*** iyamahat has joined #openstack-infra02:03
*** lukebrowning has joined #openstack-infra02:04
openstackgerritMerged openstack-infra/nodepool feature/zuulv3: Fail on quota-exceeded (partial revert)  https://review.openstack.org/52917302:04
*** caphrim007 has quit IRC02:04
*** dprince has quit IRC02:06
*** salv-orlando has joined #openstack-infra02:07
*** lukebrowning has quit IRC02:08
*** salv-orlando has quit IRC02:11
*** lukebrowning has joined #openstack-infra02:19
*** r-daneel has quit IRC02:20
hoangcxHi team,02:20
hoangcxI have a small question: the following patch has been merged recently https://review.openstack.org/#/c/527255/02:21
hoangcxBut we don't it publish to https://governance.openstack.org/tc/reference/projects/neutron.html02:22
hoangcxAfter looking we found that it seems post job failure: http://logs.openstack.org/24/24e11953f30926e2ce7eedfd0a6c2f62b3ace451/post/publish-governance-tc/2164f19/job-output.txt02:22
hoangcxCould you please help us to check it? Thanks a lot02:23
*** lukebrowning has quit IRC02:24
hoangcxs/But we don't/But we don't see02:24
openstackgerritYAMAMOTO Takashi proposed openstack-infra/openstack-zuul-jobs master: networking-midonet: Remove some of legacy jobs  https://review.openstack.org/52247102:25
*** lukebrowning has joined #openstack-infra02:25
mordredhoangcx: there was an issue earlier today - it should be fixed by https://review.openstack.org/#/c/529155/02:27
mordredhoangcx: unfortunately, that landed about 30 minutes after the patch you mention02:27
mordredhoangcx: the next patch that lands to governance should update the docs02:27
*** lukebrowning has quit IRC02:30
*** Goneri has quit IRC02:30
hoangcxmordred: Ah, I see the problem now. Thanks a lot for your head up :)02:31
*** lukebrowning has joined #openstack-infra02:32
mordredhoangcx: sure thing - sorry for the issue!02:32
*** bobh has joined #openstack-infra02:34
*** lukebrowning has quit IRC02:36
*** catintheroof has joined #openstack-infra02:38
*** lukebrowning has joined #openstack-infra02:44
*** lukebrowning has quit IRC02:49
*** calebb has quit IRC02:49
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Fix build-javascript-content success-url  https://review.openstack.org/52920302:50
*** lukebrowning has joined #openstack-infra02:50
*** bobh has quit IRC02:51
*** bobh has joined #openstack-infra02:52
*** catintheroof has quit IRC02:53
*** lukebrowning has quit IRC02:55
*** gmann has quit IRC02:55
*** calebb has joined #openstack-infra02:56
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Don't install npm dependencies in /var/log  https://review.openstack.org/52862502:56
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file  https://review.openstack.org/52862602:56
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Update default version of nodejs  https://review.openstack.org/52815602:56
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test  https://review.openstack.org/52813002:56
*** ijw has quit IRC02:58
*** rmcall has joined #openstack-infra02:59
*** lukebrowning has joined #openstack-infra03:01
*** rlandy|rover has quit IRC03:01
*** threestrands has quit IRC03:03
*** threestrands has joined #openstack-infra03:04
*** threestrands has quit IRC03:05
*** threestrands has joined #openstack-infra03:06
*** bobh has quit IRC03:06
*** lukebrowning has quit IRC03:06
*** threestrands has quit IRC03:07
*** threestrands has joined #openstack-infra03:07
*** salv-orlando has joined #openstack-infra03:07
*** lukebrowning has joined #openstack-infra03:07
*** lin_yang has quit IRC03:08
*** yamahata has quit IRC03:11
*** iyamahat has quit IRC03:12
*** lukebrowning has quit IRC03:12
*** salv-orlando has quit IRC03:12
*** bobh has joined #openstack-infra03:13
*** dave-mccowan has quit IRC03:13
*** lukebrowning has joined #openstack-infra03:14
*** vivsoni has quit IRC03:15
*** lukebrowning has quit IRC03:18
*** lukebrowning has joined #openstack-infra03:20
*** lukebrowning has quit IRC03:25
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Don't install npm dependencies in /var/log  https://review.openstack.org/52862503:25
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file  https://review.openstack.org/52862603:25
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Update default version of nodejs  https://review.openstack.org/52815603:25
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test  https://review.openstack.org/52813003:25
*** lukebrowning has joined #openstack-infra03:26
*** andreas_s has joined #openstack-infra03:29
*** tinwood has quit IRC03:30
*** tinwood has joined #openstack-infra03:31
*** lukebrowning has quit IRC03:31
*** lukebrowning has joined #openstack-infra03:33
*** andreas_s has quit IRC03:33
*** lukebrowning has quit IRC03:37
*** caphrim007 has joined #openstack-infra03:38
*** edmondsw has joined #openstack-infra03:44
*** Apoorva has joined #openstack-infra03:44
*** gongysh has joined #openstack-infra03:48
*** edmondsw has quit IRC03:49
*** slaweq has joined #openstack-infra03:49
*** ramishra has joined #openstack-infra03:50
*** gyee has quit IRC03:51
*** slaweq has quit IRC03:54
*** owalsh_ has joined #openstack-infra03:55
*** udesale has joined #openstack-infra03:57
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fix on Speaker PUT  https://review.openstack.org/52921303:57
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Don't install npm dependencies in /var/log  https://review.openstack.org/52862503:57
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Xenial: add a service file  https://review.openstack.org/52862603:57
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Update default version of nodejs  https://review.openstack.org/52815603:57
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test  https://review.openstack.org/52813003:57
ianwclarkb: arrgh, why'd you have to go and point out perfectly valid problems! :)03:57
*** threestrands_ has joined #openstack-infra03:57
*** threestrands has quit IRC03:57
clarkbianw sorry03:58
*** threestrands_ has quit IRC03:58
*** zhurong has joined #openstack-infra03:58
*** threestrands_ has joined #openstack-infra03:59
*** owalsh has quit IRC03:59
openstackgerritMerged openstack-infra/openstackid-resources master: Fix on Speaker PUT  https://review.openstack.org/52921304:00
*** lukebrowning has joined #openstack-infra04:02
*** lukebrowning has quit IRC04:07
*** yamamoto_ has joined #openstack-infra04:08
*** salv-orlando has joined #openstack-infra04:08
*** yamamoto has quit IRC04:11
*** lukebrowning has joined #openstack-infra04:13
*** salv-orlando has quit IRC04:13
*** namnh has joined #openstack-infra04:14
*** lukebrowning has quit IRC04:18
mordredianw: clarkb has talent in those areas04:19
*** lukebrowning has joined #openstack-infra04:19
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test  https://review.openstack.org/52813004:19
*** shu-mutou has joined #openstack-infra04:20
mordredianw, clarkb: any reason to not approve https://review.openstack.org/#/c/526978/3 ?04:24
*** lukebrowning has quit IRC04:24
ianwmordred: i don't think so, but i'm not really going to be around to check on it04:24
*** iyamahat has joined #openstack-infra04:25
*** lukebrowning has joined #openstack-infra04:25
ianwi'm still working on the rspec test that fully validates the etherpad patches04:25
*** ykarel has joined #openstack-infra04:27
*** pgadiya has joined #openstack-infra04:29
*** lukebrowning has quit IRC04:30
*** lukebrowning has joined #openstack-infra04:32
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fixed assets sync mapping  https://review.openstack.org/52921904:33
openstackgerritMerged openstack-infra/openstackid-resources master: Fixed assets sync mapping  https://review.openstack.org/52921904:34
*** iyamahat has quit IRC04:35
*** lukebrowning has quit IRC04:36
*** lukebrowning has joined #openstack-infra04:38
*** iyamahat has joined #openstack-infra04:39
*** pgadiya has quit IRC04:39
SpamapShmmm weird thing with glean04:41
SpamapSI have nodes that have fqdn's in the hostname field but are not getting an fqdn hostname set.04:42
*** lukebrowning has quit IRC04:42
SpamapSOh that's kind of annoying...04:43
SpamapSglean uses 'name'04:43
SpamapSnot 'hostname'04:43
openstackgerritMerged openstack-infra/puppet-etherpad_lite master: Fix up log rotation  https://review.openstack.org/52862404:43
*** armax has quit IRC04:44
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/glean master: Use hostname from config drive if set  https://review.openstack.org/52922104:47
*** bobh has quit IRC04:48
*** lukebrowning has joined #openstack-infra04:50
*** armax has joined #openstack-infra04:51
*** armax has quit IRC04:51
*** lukebrowning has quit IRC04:55
*** pgadiya has joined #openstack-infra04:56
*** lukebrowning has joined #openstack-infra04:57
openstackgerritIan Wienand proposed openstack-infra/puppet-etherpad_lite master: Add basic rspec test  https://review.openstack.org/52813004:57
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/glean master: Use hostname from config drive if set  https://review.openstack.org/52922104:57
*** ianw is now known as ianw_pto04:57
*** links has joined #openstack-infra04:57
*** armaan has quit IRC05:02
*** felipemonteiro has joined #openstack-infra05:03
openstackgerritIan Wienand proposed openstack-infra/system-config master: Fix codesearch stderr capturing  https://review.openstack.org/52918805:03
*** lukebrowning has quit IRC05:06
*** xarses_ has joined #openstack-infra05:08
*** lukebrowning has joined #openstack-infra05:08
*** salv-orlando has joined #openstack-infra05:09
*** yamamoto_ has quit IRC05:10
*** lukebrowning has quit IRC05:13
*** lukebrowning has joined #openstack-infra05:14
*** salv-orlando has quit IRC05:15
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Assume a quota limit of -1 means unlimited  https://review.openstack.org/52918005:15
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fix on Admin Speaker Serializer  https://review.openstack.org/52922305:16
openstackgerritzhufl proposed openstack-infra/elastic-recheck master: Remove "Initializing SSH connection to" message from query  https://review.openstack.org/51954605:16
openstackgerritMerged openstack-infra/openstackid-resources master: Fix on Admin Speaker Serializer  https://review.openstack.org/52922305:17
*** namnh has quit IRC05:19
*** lukebrowning has quit IRC05:19
*** coolsvap has joined #openstack-infra05:20
*** lukebrowning has joined #openstack-infra05:21
*** lukebrowning has quit IRC05:25
*** lukebrowning has joined #openstack-infra05:27
*** pgadiya has quit IRC05:30
*** pgadiya has joined #openstack-infra05:31
*** lukebrowning has quit IRC05:32
*** edmondsw has joined #openstack-infra05:32
*** yamamoto has joined #openstack-infra05:32
*** edmondsw has quit IRC05:37
*** lukebrowning has joined #openstack-infra05:39
*** bhavik has joined #openstack-infra05:42
openstackgerritYushiro FURUKAWA proposed openstack-infra/project-config master: Add python-pqclient project  https://review.openstack.org/52761205:42
*** bhavik has quit IRC05:44
*** lukebrowning has quit IRC05:44
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fix on Admin Speaker Serializer  https://review.openstack.org/52922405:44
*** bhavik has joined #openstack-infra05:45
*** jascott1 has quit IRC05:45
*** xarses_ has quit IRC05:46
*** lukebrowning has joined #openstack-infra05:46
openstackgerritMerged openstack-infra/openstackid-resources master: Fix on Admin Speaker Serializer  https://review.openstack.org/52922405:46
*** jascott1 has joined #openstack-infra05:46
*** armaan has joined #openstack-infra05:48
*** Apoorva has quit IRC05:50
*** lukebrowning has quit IRC05:50
*** jascott1 has quit IRC05:50
*** lukebrowning has joined #openstack-infra05:52
*** armaan has quit IRC05:52
*** bhavik has quit IRC05:54
*** lukebrowning has quit IRC05:57
*** lukebrowning has joined #openstack-infra05:58
*** lukebrowning has quit IRC06:03
openstackgerritMerged openstack-infra/nodepool feature/zuulv3: Assume a quota limit of -1 means unlimited  https://review.openstack.org/52918006:03
*** lukebrowning has joined #openstack-infra06:05
*** lukebrowning has quit IRC06:09
*** salv-orlando has joined #openstack-infra06:10
*** lukebrowning has joined #openstack-infra06:14
*** salv-orlando has quit IRC06:15
*** d0ugal has quit IRC06:17
*** lukebrowning has quit IRC06:18
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Test that -1 works for infinite quota  https://review.openstack.org/52923106:19
*** rmcall has quit IRC06:21
*** d0ugal has joined #openstack-infra06:23
*** lukebrowning has joined #openstack-infra06:25
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Make AFS marker roles honor sphinx variables  https://review.openstack.org/52573506:26
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Migrate special docs jobs to build-openstack-sphinx-docs  https://review.openstack.org/52573606:26
prometheanfireSamYaple: didn't think it'd get heated so fast lol06:26
*** salv-orlando has joined #openstack-infra06:28
*** lukebrowning has quit IRC06:30
*** lukebrowning has joined #openstack-infra06:31
*** felipemonteiro has quit IRC06:38
*** felipemonteiro has joined #openstack-infra06:39
*** lukebrowning has quit IRC06:41
*** ianychoi has quit IRC06:44
*** lukebrowning has joined #openstack-infra06:45
*** e0ne has joined #openstack-infra06:47
*** yamamoto has quit IRC06:49
*** janki has joined #openstack-infra06:49
*** jascott1 has joined #openstack-infra06:50
*** felipemonteiro has quit IRC06:50
*** lukebrowning has quit IRC06:50
*** lukebrowning has joined #openstack-infra06:52
SamYapleprometheanfire: i know right. i was literally just looking for bullet points. "no and heres why"06:53
SamYapleim a big boy. i can handle it06:53
SamYaplehonestly stuff like that is kinda why i hate posting to the ML06:56
*** andreas_s has joined #openstack-infra06:56
*** lukebrowning has quit IRC06:57
*** threestrands_ has quit IRC06:57
*** pahuang has quit IRC06:58
*** lukebrowning has joined #openstack-infra06:58
*** lukebrowning has quit IRC07:03
*** yamamoto has joined #openstack-infra07:04
*** lukebrowning has joined #openstack-infra07:04
*** `Kevin has quit IRC07:04
*** `Kevin has joined #openstack-infra07:05
*** rcernin has quit IRC07:08
prometheanfire:P07:08
*** yamamoto has quit IRC07:09
*** lukebrowning has quit IRC07:09
*** e0ne has quit IRC07:12
*** lukebrowning has joined #openstack-infra07:13
*** gongysh has quit IRC07:17
*** lukebrowning has quit IRC07:18
*** claudiub|2 has joined #openstack-infra07:19
*** e0ne has joined #openstack-infra07:19
*** yamamoto has joined #openstack-infra07:20
*** lukebrowning has joined #openstack-infra07:20
*** edmondsw has joined #openstack-infra07:21
*** yamamoto has quit IRC07:24
*** lukebrowning has quit IRC07:24
*** edmondsw has quit IRC07:25
*** e0ne has quit IRC07:25
*** lukebrowning has joined #openstack-infra07:26
*** kjackal has quit IRC07:28
*** shardy has joined #openstack-infra07:29
*** lukebrowning has quit IRC07:31
*** lukebrowning has joined #openstack-infra07:32
*** kiennt26 has quit IRC07:35
openstackgerritDirk Mueller proposed openstack-dev/hacking master: Transition to flake8 2.6.x  https://review.openstack.org/51493407:37
*** lukebrowning has quit IRC07:37
*** lukebrowning has joined #openstack-infra07:38
*** iyamahat has quit IRC07:42
*** gongysh has joined #openstack-infra07:42
*** lukebrowning has quit IRC07:43
*** slaweq has joined #openstack-infra07:45
*** lukebrowning has joined #openstack-infra07:45
ttxAJaeger: thanks for your help getting everything back to normal yesterday !07:45
*** HeOS has joined #openstack-infra07:46
*** iyamahat has joined #openstack-infra07:49
*** lukebrowning has quit IRC07:50
*** iyamahat has quit IRC07:50
*** lukebrowning has joined #openstack-infra07:51
*** lukebrowning has quit IRC07:56
*** iyamahat has joined #openstack-infra07:57
*** lukebrowning has joined #openstack-infra07:57
*** mkoderer__ has joined #openstack-infra08:01
*** lukebrowning has quit IRC08:02
*** lukebrowning has joined #openstack-infra08:04
*** yamamoto has joined #openstack-infra08:04
*** iyamahat has quit IRC08:08
*** lukebrowning has quit IRC08:09
*** yamamoto has quit IRC08:09
*** raissa has quit IRC08:10
*** hoangcx has quit IRC08:12
*** hoangcx has joined #openstack-infra08:13
*** lukebrowning has joined #openstack-infra08:16
*** zhurong has quit IRC08:17
*** daidv has quit IRC08:18
*** daidv has joined #openstack-infra08:19
*** lukebrowning has quit IRC08:21
*** salv-orlando has quit IRC08:22
*** lukebrowning has joined #openstack-infra08:23
AJaegerttx: sorry for helping to break it ;)08:23
*** iyamahat has joined #openstack-infra08:23
AJaegerttx, you need to merge a single patch for governance to publish again - we only fixed the post job once all the changes were in ;808:24
AJaegerttx, do you have other repos that need python3 for docs publishing that we should fix?08:24
*** alexchadin has joined #openstack-infra08:26
*** cshastri has quit IRC08:26
*** ralonsoh has joined #openstack-infra08:27
*** jtomasek has joined #openstack-infra08:27
*** ccamacho has joined #openstack-infra08:27
*** hashar has joined #openstack-infra08:27
*** lukebrowning has quit IRC08:27
*** lukebrowning has joined #openstack-infra08:29
oansonHello.08:31
oansonI get the following error when trying to ssh to review.openstack.org: Too many concurrent connections (64) - max. allowed: 6408:31
oansonI don't have any open connections on my end.08:31
*** jtomasek has quit IRC08:31
oansonCan someone please help?08:32
*** jtomasek has joined #openstack-infra08:32
*** lukebrowning has quit IRC08:34
*** yamamoto has joined #openstack-infra08:34
*** amoralej|off is now known as amoralej08:35
*** lukebrowning has joined #openstack-infra08:35
*** cshastri has joined #openstack-infra08:39
*** yamamoto has quit IRC08:39
*** lukebrowning has quit IRC08:40
frickleroanson: are you using a shared gateway so that others in your company or some CI might be causing those connections?08:41
*** kmalloc has quit IRC08:41
oansonI'm behind a company NAT, but we're about 4-5 people working on OpenStack at the moment08:41
*** lukebrowning has joined #openstack-infra08:42
*** jpena|off is now known as jpena08:44
oansonfrickler, also, my colleague manages to connect behind the same NAT with her username08:44
*** lukebrowning has quit IRC08:46
*** lukebrowning has joined #openstack-infra08:48
*** markmcd has joined #openstack-infra08:49
*** claudiub has joined #openstack-infra08:49
*** slaweq_ has joined #openstack-infra08:49
*** yamamoto has joined #openstack-infra08:50
*** pgadiya has quit IRC08:50
*** claudiub|2 has quit IRC08:52
*** lukebrowning has quit IRC08:53
*** stakeda has quit IRC08:53
*** yamamoto has quit IRC08:54
*** slaweq_ has quit IRC08:54
*** lukebrowning has joined #openstack-infra08:54
frickleroanson: sadly I seem to be lacking permissions to actually view the connections, will have to wait for some other infra-root to help08:55
*** yamamoto has joined #openstack-infra08:56
*** yamamoto has quit IRC08:56
*** salv-orlando has joined #openstack-infra08:57
*** claudiub|2 has joined #openstack-infra08:58
oansonfrickler, I see. Thanks anyway!08:59
*** lukebrowning has quit IRC08:59
*** lukebrowning has joined #openstack-infra09:00
*** claudiub has quit IRC09:01
*** sbezverk has quit IRC09:02
*** alexchadin has quit IRC09:04
*** pgadiya has joined #openstack-infra09:04
*** alexchadin has joined #openstack-infra09:04
*** yamamoto has joined #openstack-infra09:05
*** lukebrowning has quit IRC09:05
*** vivsoni has joined #openstack-infra09:06
*** lukebrowning has joined #openstack-infra09:07
*** lukebrowning has quit IRC09:12
*** rossella_s has joined #openstack-infra09:13
chandankumarfrickler: hello09:14
chandankumarfrickler: please have a look at this review https://review.openstack.org/#/c/528672/09:14
*** owalsh_ is now known as owalsh09:17
AJaegerconfig-core, please review https://review.openstack.org/529111 and https://review.openstack.org/#/c/528518/ to fix translations.09:18
*** ralonsoh has quit IRC09:19
*** lukebrowning has joined #openstack-infra09:19
*** slaweq_ has joined #openstack-infra09:20
*** lukebrowning has quit IRC09:24
*** slaweq_ has quit IRC09:25
*** e0ne has joined #openstack-infra09:26
*** dsariel has joined #openstack-infra09:29
*** lukebrowning has joined #openstack-infra09:30
*** lucas-afk is now known as lucasagomes09:34
*** lukebrowning has quit IRC09:34
*** lukebrowning has joined #openstack-infra09:36
*** iyamahat has quit IRC09:40
*** lukebrowning has quit IRC09:41
*** lukebrowning has joined #openstack-infra09:42
*** derekh has joined #openstack-infra09:44
*** ralonsoh has joined #openstack-infra09:45
*** lukebrowning has quit IRC09:47
*** lukebrowning has joined #openstack-infra09:49
openstackgerritMasayuki Igawa proposed openstack/os-testr master: Fix to enable --color option of ostestr command  https://review.openstack.org/52928109:50
*** lukebrowning has quit IRC09:53
oansonHello.09:53
oansonI get the following error when trying to ssh to review.openstack.org: Too many concurrent connections (64) - max. allowed: 6409:53
oansonI don't have any open connections on my end.09:53
oansonCan someone please help?09:53
openstackgerritCao Xuan Hoang proposed openstack-infra/project-config master: Add neutron-vpnaas docs and release notes pub job to zuul layout  https://review.openstack.org/52269509:54
*** claudiub has joined #openstack-infra09:54
oansonAJaeger, chandankumar, do you have the permissions to clear out my (dead) connections?09:54
*** lukebrowning has joined #openstack-infra09:55
*** claudiub|2 has quit IRC09:56
*** florianf has joined #openstack-infra09:58
*** lukebrowning has quit IRC10:00
*** psachin has quit IRC10:01
*** lukebrowning has joined #openstack-infra10:01
*** kjackal has joined #openstack-infra10:03
*** jascott1 has quit IRC10:05
*** jascott1 has joined #openstack-infra10:05
*** tosky has joined #openstack-infra10:05
*** lukebrowning has quit IRC10:06
*** liujiong has quit IRC10:06
dulekHi guys! Anyone have a clue why kuryr-kubernetes sphinx jobs started to fail yesterday with:10:06
dulekhttp://logs.openstack.org/92/529092/2/check/build-openstack-sphinx-docs/83eac6e/job-output.txt.gz#_2017-12-20_09_22_39_28552310:06
*** rossella_s has quit IRC10:06
*** cuongnv has quit IRC10:08
*** e0ne_ has joined #openstack-infra10:08
*** rossella_s has joined #openstack-infra10:09
ttxAJaeger: the governance-uc, governance-website and governance-sigs likely need the same fix10:10
*** psachin has joined #openstack-infra10:10
*** jascott1 has quit IRC10:10
AJaegeroanson: no - you need to wait for the US to wake up...10:11
*** e0ne has quit IRC10:11
AJaegerttx, looking at their tox.ini - I don't think so. Let me recheck a couple of changes to double check10:12
*** Qiming has quit IRC10:14
*** Qiming has joined #openstack-infra10:14
ttxAJaeger: re single patch to publish again... which one is it ?10:15
*** jeblair has quit IRC10:15
*** jeblair has joined #openstack-infra10:16
oansonAJaeger, thanks. Is there a specific person I will have to contact?10:17
*** rpittau has joined #openstack-infra10:19
*** psachin has quit IRC10:20
*** shardy has quit IRC10:24
AJaegerttx, I mean: Your next change to merge will publish. So, if you need to udpate the webpage now, create a trivial change and merge ;)10:24
*** lukebrowning has joined #openstack-infra10:24
AJaegeroanson: just ask here again after 15:00 UTC10:25
oansonSure. Thanks!10:25
*** rossella_s has quit IRC10:26
AJaegerttx, governance-* don't use python3 - I rechecked one governance-uc change now to be sure10:27
*** sambetts|afk is now known as sambetts10:27
AJaegerdulek: see http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html10:27
AJaegerdulek: Let me check...10:27
*** rossella_s has joined #openstack-infra10:27
dulekAJaeger: I think I need to put neutron into doc/requirements.txt?10:29
AJaegerdulek: no - let me first understand your repo10:29
*** lukebrowning has quit IRC10:29
AJaegerdulek: are you kuryr core as well? Please help merging https://review.openstack.org/52102610:30
*** aarefiev has joined #openstack-infra10:30
*** Qiming has quit IRC10:30
dulekAJaeger: Not a core, but I can raise that on the channel.10:31
AJaegerdulek: thanks!10:31
AJaegerdulek: please do - you need that one to fix releasenotes build - a related problem to yours here10:31
*** Qiming has joined #openstack-infra10:36
AJaegerdulek: what's your channel? Let me join...10:36
dulekAJaeger: #openstack-kuryr.10:37
dulekWithout dot. ;)10:37
*** anupn has quit IRC10:38
*** aviau has quit IRC10:39
ttxAJaeger: I'll find one to approve. Thanks!10:40
*** aviau has joined #openstack-infra10:40
AJaeger;)10:40
*** alexchadin has quit IRC10:41
*** alexchadin has joined #openstack-infra10:42
*** gongysh has quit IRC10:42
*** anupn has joined #openstack-infra10:46
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Set remote url on every getRepo in merger  https://review.openstack.org/52929310:48
*** ldnunes has joined #openstack-infra10:53
*** annp has quit IRC10:53
*** krtaylor has quit IRC10:53
*** hashar is now known as hasharAway10:54
*** edmondsw has joined #openstack-infra10:57
*** cshastri has quit IRC10:57
*** edmondsw has quit IRC11:01
*** dtantsur|afk is now known as dtantsur11:02
*** yolanda__ has joined #openstack-infra11:05
*** lukebrowning has joined #openstack-infra11:05
*** krtaylor has joined #openstack-infra11:06
*** rossella_s has quit IRC11:06
*** rossella_s has joined #openstack-infra11:07
*** yamamoto_ has joined #openstack-infra11:08
*** yolanda has quit IRC11:08
*** markmcd has quit IRC11:08
AJaegerinfra-root, I got some gerrit timeouts. Is it time to restart gerrit?11:10
*** cshastri has joined #openstack-infra11:10
*** lukebrowning has quit IRC11:10
*** markmcd has joined #openstack-infra11:10
*** yamamoto has quit IRC11:11
*** gcb has quit IRC11:12
fricklerAJaeger: looks to me like it has been stable for an astonishinly long time already: http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=27&rra_id=all11:12
fricklerI'd suggest to wait a couple of minutes to see if it recovers11:13
AJaegerfrickler: I have problems since an hour or two and just heard others asking...11:14
AJaegerfrickler: but yeah, we can wait a bit...11:14
*** aviau has quit IRC11:15
*** lukebrowning has joined #openstack-infra11:16
*** aviau has joined #openstack-infra11:17
odyssey4meargh, gerrit's slow and timing out from time to time11:20
*** lukebrowning has quit IRC11:21
*** lukebrowning has joined #openstack-infra11:23
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/zuul feature/zuulv3: Add run_ansible_setup_on_start flag for executor  https://review.openstack.org/52930011:24
*** priteau has joined #openstack-infra11:24
*** lukebrowning has quit IRC11:27
*** cshastri has quit IRC11:29
*** rossella_s has quit IRC11:29
*** lukebrowning has joined #openstack-infra11:29
jamespagehmm I'm seeing gerrit slowness and timing out as well11:30
*** andreas__ has joined #openstack-infra11:30
*** vivsoni has quit IRC11:31
*** neiljerram has joined #openstack-infra11:31
*** andreas__ has quit IRC11:31
neiljerramMorning all!11:32
*** rossella_s has joined #openstack-infra11:32
jamespagehi neiljerram11:32
sshnaidmsee gerrit problems too11:33
*** andreas_s has quit IRC11:33
neiljerramA couple days ago, I got a notification about "Removed reviewer Jenkins with the following votes" from several of my standing changes in Gerrit.  I'm not sure what that means - does someone here know?11:33
*** alexchadin has quit IRC11:33
neiljerramjamespage, hi11:34
*** lukebrowning has quit IRC11:34
cmurphyneiljerram: see http://lists.openstack.org/pipermail/openstack-dev/2017-December/125684.html nothing to worry about11:35
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/zuul feature/zuulv3: Add run_ansible_setup_on_start flag for executor  https://review.openstack.org/52930011:35
*** lukebrowning has joined #openstack-infra11:35
neiljerramcmurphy, many thanks!11:35
*** szaher has quit IRC11:36
*** markmcd has quit IRC11:36
*** alexchadin has joined #openstack-infra11:39
*** psachin has joined #openstack-infra11:39
*** lukebrowning has quit IRC11:40
*** lukebrowning has joined #openstack-infra11:41
*** lukebrowning has quit IRC11:46
*** lukebrowning has joined #openstack-infra11:48
*** pbourke has quit IRC11:48
*** alexchadin has quit IRC11:49
*** pbourke has joined #openstack-infra11:49
*** lukebrowning has quit IRC11:53
*** lukebrowning has joined #openstack-infra11:54
*** rossella_s has quit IRC11:55
*** rossella_s has joined #openstack-infra11:58
*** lukebrowning has quit IRC11:59
*** salv-orl_ has joined #openstack-infra11:59
*** lukebrowning has joined #openstack-infra12:00
*** smatzek has joined #openstack-infra12:01
*** dizquierdo has joined #openstack-infra12:02
*** priteau has quit IRC12:03
*** salv-orlando has quit IRC12:03
*** priteau has joined #openstack-infra12:03
*** claudiub has quit IRC12:03
*** rossella_s has quit IRC12:04
*** lukebrowning has quit IRC12:05
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/zuul feature/zuulv3: Add run_ansible_setup_on_start flag for executor  https://review.openstack.org/52930012:06
*** rossella_s has joined #openstack-infra12:06
*** jascott1 has joined #openstack-infra12:06
*** andreas_s has joined #openstack-infra12:06
*** lukebrowning has joined #openstack-infra12:07
*** priteau has quit IRC12:08
*** andreas_s has quit IRC12:11
*** jascott1 has quit IRC12:11
*** lukebrowning has quit IRC12:11
*** andreas_s has joined #openstack-infra12:11
*** dave-mccowan has joined #openstack-infra12:14
*** alexchadin has joined #openstack-infra12:15
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/zuul feature/zuulv3: Add run_ansible_setup_on_start flag for executor  https://review.openstack.org/52930012:16
tobiashSpamapS, jeblair, mordred: change 527298 (drop local fork of gitpython for 2.1.8 release) breaks my local test suite12:18
tobiashdidn't find out yet why12:18
tobiashups, zuul channel would be better for that12:21
*** lucasagomes is now known as lucas-hungry12:26
*** rossella_s has quit IRC12:33
*** aarefiev has quit IRC12:33
*** rossella_s has joined #openstack-infra12:35
*** armaan has joined #openstack-infra12:36
*** gcb has joined #openstack-infra12:39
*** cdent has joined #openstack-infra12:41
cdentgerrit unhappy, or is it just meâ„¢12:41
tobiashSpamapS, jeblair, mordred: oh, gitpython is now processing my globally defined gerrit commit message hook which breaks string matching in the tests12:43
tobiashremoved that locally made the tests run again :)12:43
tobiashcdent: I also experienced slow requests to gerrit12:43
cdentI was almost hoping it was just me.12:43
*** liuyulong has joined #openstack-infra12:44
*** lukebrowning has joined #openstack-infra12:45
*** edmondsw has joined #openstack-infra12:45
*** armaan has quit IRC12:45
*** bobh has joined #openstack-infra12:45
tobiashit runs normally for me nwo12:46
*** felipemonteiro has joined #openstack-infra12:46
tobiashhrm, I thought that, but now experienced another almost timeout12:46
*** gcb has quit IRC12:47
*** lukebrowning has quit IRC12:49
*** edmondsw has quit IRC12:49
*** felipemonteiro has quit IRC12:50
*** kiennt26 has joined #openstack-infra12:53
*** pgadiya has quit IRC12:53
*** armaan has joined #openstack-infra12:53
*** yolanda__ is now known as yolanda12:53
*** lukebrowning has joined #openstack-infra12:55
*** rossella_s has quit IRC12:57
*** jpena is now known as jpena|lunch12:58
*** lukebrowning has quit IRC13:00
*** lukebrowning has joined #openstack-infra13:01
mordredtobiash: when you say globally defined gerrit commit message hook, that's defined in your homedir right? we could add self.useFixture(fixtures.TempHomeDir()) to ZuulTestCase.setUp and that *should* prevent such leakage13:02
tobiashmordred: yes13:02
tobiashmordred: I'll check that13:03
tobiashquick fix was to delete the hook (as it will be invalid with github anyway)13:03
mordredgood point :)13:03
*** rossella_s has joined #openstack-infra13:03
*** dmellado has quit IRC13:05
*** coolsvap has quit IRC13:05
*** lukebrowning has quit IRC13:06
fricklermordred: I think gerrit is ripe for a restart by now, can you confirm that I only need to do a "service gerrit stop/start" on review.o.o?13:07
ttxslow13:07
oansonHello.13:07
oansonI get the following error when trying to ssh to review.openstack.org: Too many concurrent connections (64) - max. allowed: 6413:07
oansonI don't have any open connections on my end.13:07
oansonCan someone please help?13:07
*** lukebrowning has joined #openstack-infra13:07
*** dprince has joined #openstack-infra13:08
ttxoanson: yes, known issue, restart being planned13:08
oansonttx, great. Thanks!13:08
liuyulongreview.o.o: get errors: "Reason: Error reading from remote server, Apache/2.4.7 (Ubuntu) Server at review.openstack.org Port 44313:08
mnaseri think it would be good to #status or #notice the gerrit issues13:10
fricklerinfra-root: seems gerrit is getting unuseable by now, I'm going to perform the stop/start13:10
*** e0ne has joined #openstack-infra13:10
mnaserfrickler: i would "#status notice <heads up>"13:11
mnaserhttps://docs.openstack.org/infra/system-config/irc.html#statusbot13:12
*** lukebrowning has quit IRC13:12
jianghuaw_gerrit is down now? suddenly got error: "503 Service Unavailable"13:12
ttxjianghuaw_: restart in progress13:12
frickler#status notice gerrit is being restarted due to extreme slowness13:12
openstackstatusfrickler: sending notice13:12
jianghuaw_ttx, got it. thanks.13:13
*** e0ne_ has quit IRC13:13
*** openstackgerrit has quit IRC13:13
-openstackstatus- NOTICE: gerrit is being restarted due to extreme slowness13:14
mnaserfrickler: from 2015 it seems that service gerrit restart does the trick13:14
*** dmellado has joined #openstack-infra13:15
fricklermnaser: thx, it said "Starting Gerrit Code Review: FAILED" but seems to have succeeded a bit later anyhow13:15
mnaserfrickler: hmm, i wonder why it would fail (maybe stale pid file?)13:15
openstackstatusfrickler: finished sending notice13:15
*** amoralej is now known as amoralej|lunch13:16
mnaserfrickler: im not infra root so i cant dive in but at least we have it back up, but probably good to check with other infra roots when they wake up to double heck13:16
*** stevebaker has quit IRC13:18
*** lukebrowning has joined #openstack-infra13:18
*** dmellado has quit IRC13:19
*** dmellado has joined #openstack-infra13:21
frickleroanson: the restart should have solved your issue implicitly for now, please let us know if/when it repeats13:21
oansonfrickler, thanks. Appears to work. Will do.13:22
*** lukebrowning has quit IRC13:23
*** rlandy has joined #openstack-infra13:23
*** rlandy is now known as rlandy|ruck13:24
*** lukebrowning has joined #openstack-infra13:25
AJaegermordred: fun debugging of the day: kuryr-kubernetes had a tools/tox_install.sh that was not used at all - now the updated jobs used it and failed ;)13:26
*** openstackgerrit has joined #openstack-infra13:27
openstackgerritMerged openstack-infra/project-config master: Add neutron-vpnaas/vpnaas-dashboard to gerritbot  https://review.openstack.org/52786713:27
openstackgerritMerged openstack-infra/project-config master: Drop django_openstack_auth master branch jobs  https://review.openstack.org/52908013:27
*** alexchadin has quit IRC13:27
*** lukebrowning_ has joined #openstack-infra13:28
*** stevebaker has joined #openstack-infra13:28
*** lucas-hungry is now known as lucasagomes13:28
*** lukebrowning has quit IRC13:29
openstackgerritMerged openstack-infra/project-config master: Use openstack-python-jobs for tacker-horizon  https://review.openstack.org/52850313:31
*** bobh has quit IRC13:31
*** lukebrowning_ has quit IRC13:33
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove obsolete dashboard tox jobs  https://review.openstack.org/52858113:33
mordredAJaeger: hahahahahahahahaha13:34
*** lukebrowning has joined #openstack-infra13:34
*** yamamoto_ has quit IRC13:34
AJaegermordred: another problem: RHEL and CentOS7 use rsync 3.0 apparently that does not support "--ignore-missing-args", see  http://logs.openstack.org/30/529230/2/experimental/swift-tox-func-centos-7/f5ab6e1/job-output.txt.gz#_2017-12-20_11_11_35_50068513:36
AJaegerwonder why we didn't notice that elsewhere...13:36
AJaegermordred: so, this breaks fetch-stestr-output and gives a post job failure13:37
*** armaan has quit IRC13:38
*** cdent has left #openstack-infra13:38
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Rework translation templates  https://review.openstack.org/52911113:38
*** lukebrowning has quit IRC13:39
*** lukebrowning has joined #openstack-infra13:40
*** rhallisey has quit IRC13:40
mordredAJaeger: oh - well, that may also explain another issuetoo - andreykurilin was having an issue with synchronize: not supporting --ignore-missing-args and I was confused13:41
AJaegerif it was on centos7...13:42
andreykurilinI suppose it was centos713:42
AJaegerinfra-root, we just merged 528581 but it's not in http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/13:43
AJaegeranything broken with gerrit/git?13:43
*** jcoufal has joined #openstack-infra13:43
*** rhallisey has joined #openstack-infra13:43
*** lukebrowning has quit IRC13:45
*** gongysh has joined #openstack-infra13:45
mordredAJaeger: looking in the logs, I think gerrit is still working on starting replication13:46
*** kiennt26 has quit IRC13:46
*** lukebrowning has joined #openstack-infra13:46
AJaegerinteresting. Looking at gerrit showqueue, I see "18be545f waiting .... 13:14:21.963 git01/stackforge/graffiti" - so, many stackforge repos ;(13:47
AJaegercan't we disable that somehow?13:47
AJaegerthanks, mordred13:47
AJaegerqueue is getting shorter - I'll wait some more.13:47
AJaegermordred: should I open a story for the rsync --ignore-missing-args?13:48
mordredAJaeger: yah, I think so - we use it in a few places, so solving it is going to be important13:48
*** trown|outtypewww is now known as trown13:48
dmsimardAJaeger, mordred: I remember approving a patch about missing args, what about it ?13:49
*** gongysh has quit IRC13:49
mordreddmsimard: apparently it's not supported on centos/rhel13:49
*** psachin has quit IRC13:49
*** eharney has joined #openstack-infra13:49
dmsimardthat's right so the assumption was that a particular role needed to run exclusively on ubuntu/debian which wasn't necessary if only for that argument13:49
* dmsimard searches patch13:49
*** amoralej|lunch is now known as amoralej13:50
*** goutham has joined #openstack-infra13:50
AJaegerdmsimard: that was for translations - but we talk about unittests here13:50
AJaegerconfig-core, please review https://review.openstack.org/#/c/528518/ to fix translations. And a refactor: https://review.openstack.org/#/c/529047/13:51
*** lukebrowning has quit IRC13:51
dmsimardAJaeger, mordred: https://review.openstack.org/#/c/525510/13:51
openstackgerritMohammed Naser proposed openstack-infra/project-config master: Remove publish-openstack-python-branch-tarball from steth  https://review.openstack.org/52931713:51
openstackgerritMohammed Naser proposed openstack-infra/project-config master: Added check to ensure template usage  https://review.openstack.org/52931813:51
*** rlandy|ruck is now known as rlandy|rover13:52
AJaegerdmsimard: ah, so we need to do this in the other role as well...13:52
*** mriedem1 has joined #openstack-infra13:52
* AJaeger prepares a change13:52
dmsimardIt's okay that OpenStack "only" runs tox on Ubuntu but tox also runs on other distros you know :P13:52
*** catintheroof has joined #openstack-infra13:52
*** trown is now known as trown|ruck13:52
mordreddmsimard: :)13:53
dmsimardso what's the fix then ?13:53
dmsimardI'm not super familiar with that arg, I had to look it up13:53
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Set remote url on every getRepo in merger  https://review.openstack.org/52929313:53
*** lukebrowning has joined #openstack-infra13:53
mordreddunno. we're going to have to get more cleverer in constructing synchronize commands13:54
dmsimardsynchronize is one of those modules which kind of suck tbh :(13:54
mordredyah- but in this case it's actually just an rsync issue anyway13:54
*** jpena|lunch is now known as jpena13:54
mordredwe've only got two uses of it in zuul-jobs ...13:55
dmsimardjust being able to understand which direction the transfer is going to be in (pull or push, remember it's on a remote target) requires my brain to make efforts.13:55
*** mriedem has quit IRC13:55
mordredtotes13:55
*** claudiub has joined #openstack-infra13:55
tobiashdmsimard: better (faster) than copy ;)13:56
openstackgerritAndreas Jaeger proposed openstack-infra/zuul-jobs master: Do not use --ignore-missing-args for rsync  https://review.openstack.org/52932013:56
tobiashdmsimard: but totally agree...13:56
AJaegermordred, dmsimard, tobiash: Fixed ^13:56
dmsimardtobiash: yeah but copy doesn't scale if you have a large number of files13:56
*** mriedem1 is now known as mriedem13:56
tobiashdmsimard: yes, and large begins at 50...13:56
mordredAJaeger: yes. ++13:56
tobiashthat makes copy almost useless13:56
AJaegerthose were the only two occurences in our config repos13:57
dmsimardAJaeger: the impact of removing --ignore-missing-args is still not 100% clear to me. Is there a use case that fails if we remove that ?13:57
mordreddmsimard: it's basically to allow us to just have the sync command for the files that might be there without having to stat for them first13:57
AJaegerdmsimard: and I set "ignore: errors" instead, so that should result in same behaviour13:57
dmsimardoh, so rsync foo bar wouldn't fail if foo didn't exist13:57
mordredyah13:57
dmsimardAJaeger: either that or failed_when: false13:58
openstackgerritAkihiro Motoki proposed openstack-infra/project-config master: Add noop job for django_openstack_auth master branch  https://review.openstack.org/52932113:58
AJaegerdmsimard: copied the construct from the patch you showed me ;)13:58
dmsimardfailed_when and ignore_errors are slightly different13:58
*** lukebrowning has quit IRC13:58
AJaegerdo we need failed_when here? I don't see a use case yet13:59
tobiashAJaeger: depends on what we want13:59
AJaeger;)14:00
tobiashignore_errors still shows the task as failed14:00
*** lukebrowning has joined #openstack-infra14:00
mordredah - so we probably want failed_when: false14:00
tobiashwhere failed_when defined when the task really is failed14:00
dmsimardthe difference is that if you put "failed_when: false", the task will never fail whereas ignore_errors: yes will properly have the task fail but won't fail the playbook14:00
AJaegerso, what do we want, ansible experts?14:00
AJaegerI can updated my change and also change the place in 525510...14:00
dmsimard¯\_(ツ)_/¯14:01
tobiashI like playbooks to not include any *justified* failures14:01
dmsimardfwiw ara will properly display those tasks as "IGNORED" if there is a failure14:01
tobiasharg the other way round, I like playbooks to only include justified failures14:02
tobiash;)14:02
dmsimardI don't think we want a task failure here if the expectation is that it is going to fail semi regularly14:02
mordredSO ... looking at these roles, this makes me want to rework both of them slightly14:02
AJaegermordred: do you want to take over my change - or shall I do the change first?14:02
tobiashdmsimard: ++14:02
mordredAJaeger: go ahead with yours, then I'll do mine on top of it14:02
*** catintheroof has quit IRC14:02
mordredAJaeger: mine might take slightly longer, and yours should be good enough for now14:03
dmsimardAJaeger: feel free to change that other one I linked to14:03
dmsimards/to/too/14:03
*** catintheroof has joined #openstack-infra14:04
*** jamesmcarthur has joined #openstack-infra14:04
AJaegerdmsimard: already done, just pushing ...14:05
*** lukebrowning has quit IRC14:05
openstackgerritAndreas Jaeger proposed openstack-infra/zuul-jobs master: Do not use --ignore-missing-args for rsync  https://review.openstack.org/52932014:05
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Set remote url on every getRepo in merger  https://review.openstack.org/52929314:05
AJaegermordred, dmsimard, could you review  https://review.openstack.org/#/c/528518 and https://review.openstack.org/#/c/529047/ as well, please?14:05
*** lukebrowning has joined #openstack-infra14:06
*** jascott1 has joined #openstack-infra14:08
AJaegermordred: i think we should continue with updated-pti. Let's ask for reviews on https://review.openstack.org/#/c/525735/ and  https://review.openstack.org/#/c/525718/ (Zuul)14:08
AJaegerconfig-core ^14:08
*** logan- has quit IRC14:08
*** logan- has joined #openstack-infra14:10
*** yamamoto has joined #openstack-infra14:10
*** lukebrowning has quit IRC14:11
*** kgiusti has joined #openstack-infra14:12
*** andreas_s has quit IRC14:12
*** lukebrowning has joined #openstack-infra14:12
*** jascott1 has quit IRC14:13
AJaegerconfig-core, openstack-zuul-jobs job removal change for review, please https://review.openstack.org/#/c/522471/14:15
*** andreas_s has joined #openstack-infra14:17
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy novaclient jobs  https://review.openstack.org/52210214:17
*** lukebrowning has quit IRC14:17
*** lukebrowning has joined #openstack-infra14:19
*** catinthe_ has joined #openstack-infra14:22
*** yamamoto has quit IRC14:23
*** lukebrowning has quit IRC14:24
*** lukebrowning has joined #openstack-infra14:25
*** catintheroof has quit IRC14:26
*** andreas_s has quit IRC14:26
*** smatzek has quit IRC14:28
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in storlets  https://review.openstack.org/51295814:28
*** smatzek has joined #openstack-infra14:29
*** smatzek has quit IRC14:29
*** lukebrowning has quit IRC14:30
*** lukebrowning has joined #openstack-infra14:31
*** andreas_s has joined #openstack-infra14:31
*** openstackgerrit has quit IRC14:33
*** goutham has left #openstack-infra14:33
*** openstackgerrit has joined #openstack-infra14:33
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in Murano  https://review.openstack.org/51143814:33
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add consolidated role for processing subunit  https://review.openstack.org/52933914:34
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Remove testr and stestr specific roles  https://review.openstack.org/52934014:34
*** lukebrowning has quit IRC14:36
*** lukebrowning has joined #openstack-infra14:38
*** trown|ruck is now known as trown|brb14:39
*** shardy has joined #openstack-infra14:41
mordredAJaeger, dmsimard: ^^14:41
*** andreas_s has quit IRC14:41
*** lukebrowning has quit IRC14:42
AJaegermordred: "should do the right thing"? Should we test it with a DNM change?14:42
*** ykarel has quit IRC14:43
*** andreas_s has joined #openstack-infra14:45
AJaegermordred: nice!14:46
*** trown|brb is now known as trown14:47
*** sshnaidm has quit IRC14:50
*** lukebrowning has joined #openstack-infra14:50
*** sshnaidm has joined #openstack-infra14:50
mordredAJaeger: well ... it should be at least partially self-testing - since we run zuul unit tests on zuul-jobs changes14:51
AJaegerAh, good!14:51
*** trown is now known as trown|ruck14:52
mordredAJaeger: and also the tox-linters job on zuul-jobs should get us testing of running the role when there is no testr output14:52
AJaegerhow nice for a change ;)14:53
AJaegermordred: you shouldn't have said "should do the right thing" - that triggered something...14:53
mordredAJaeger: hehe. well, out of habit, I *never* want to claim anything will *actually* work14:55
*** lukebrowning has quit IRC14:55
sambettsHi, I'm seeing a failure on my "build-openstack-sphinx-docs" job here: http://logs.openstack.org/21/529121/2/check/build-openstack-sphinx-docs/329628e/job-output.txt.gz#_2017-12-20_13_47_21_082540 and on other patches, it seems to be related to https://github.com/openstack-infra/openstack-zuul-jobs/commit/e1a793b8a1f093c24264c4d0c19480607c4e1c8314:55
AJaegeryeah, made that mistake myself often enough but I'm not that careful yet14:55
AJaegersambetts: did you see mordred's email on openstack-dev?14:56
sambettsAJaeger: ah no I must have missed it14:56
*** lukebrowning has joined #openstack-infra14:57
AJaegersambetts: merge https://review.openstack.org/522315 t ofix it - waiting since a few weeks for your review ;(14:57
AJaegerwait, it fails - mordred ^14:57
*** mriedem is now known as mriedem_away14:57
AJaegersambetts: but that's the first step in fixing your problem14:58
AJaegersambetts: http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html14:58
sambettsah ok... got it, I wondered if that was the case, I left a recheck on it this morning to try to push it thourhg14:59
AJaegermordred, sambetts : http://logs.openstack.org/15/522315/3/check/build-openstack-sphinx-docs/c7bd264/job-output.txt.gz#_2017-12-20_11_46_13_258957 - import problem with neutron?14:59
andreafmordred: about fetch-subunit-output / https://review.openstack.org/#/c/529339/1 - did you consider the process-test-results role https://github.com/openstack-infra/zuul-jobs/tree/master/roles/process-test-results ?14:59
openstackgerritMerged openstack-infra/zuul-jobs master: Do not use --ignore-missing-args for rsync  https://review.openstack.org/52932015:00
*** bobh has joined #openstack-infra15:01
*** lukebrowning has quit IRC15:01
sambettsAJaeger: thats failing because we're missing an "override-checkout: stable/pike" on that job in zuul.d/project.yaml15:01
*** catinthe_ has quit IRC15:02
*** ykarel has joined #openstack-infra15:02
AJaegersambetts, is that an official project that has a post job?15:02
*** catintheroof has joined #openstack-infra15:03
openstackgerritThierry Carrez proposed openstack/ptgbot master: Specify and load event schedule  https://review.openstack.org/52516815:03
openstackgerritThierry Carrez proposed openstack/ptgbot master: Add scheduled / additional rooms  https://review.openstack.org/52934515:03
openstackgerritThierry Carrez proposed openstack/ptgbot master: Display scheduled and additional rooms in web page  https://review.openstack.org/52934615:03
*** lukebrowning has joined #openstack-infra15:03
AJaegersambetts: it's not - so, could you send a patch for project-config, please?15:03
sambettsAJaeger: yeah, writing it nowe15:03
mordredandreaf: no! I didn't - lemme rework that15:04
sambettsAJaeger: before the change our tox_install.sh logic and tox.ini set the right branch for that job, now it doesn't use tox.ini the branch is wrong :(15:04
mordredandreaf: also - while I've got you ... I'm about to start pushing on the base job rework to make the log fetching/publishing stuff more better15:04
mordredandreaf: re: your comment: https://review.openstack.org/#/c/511843/2/roles/fetch-output/tasks/main.yaml@9 ...15:05
mordredandreaf: my concern is that for simple jobs, like tox, we wind up with an extra directory that's never useful15:06
mordredandreaf: what if we just had that fetch role always just do the sync of the logs dir and never do anything with adding hostnames15:06
*** marst has joined #openstack-infra15:06
mordredandreaf: and if jobs are written that are multinode, they can write logs to {{ zuul_output_dir }}/logs/{{ inventory_hostname }}15:07
*** makowals has quit IRC15:08
*** catintheroof has quit IRC15:08
*** lukebrowning has quit IRC15:08
mordredandreaf: that way the behavior can be tied to a type of job content (and job content such as the devstack base job can just always write to the multi-node format)15:09
sambettsAJaeger: does the sphinx docs job continue to call tox_install.sh?15:09
AJaegersambetts: if it exists - it does.15:09
*** lukebrowning has joined #openstack-infra15:09
AJaegersambetts: but with *specific* parameters. That's why you need mordred's change15:09
sambettsAJaeger: ok, I think I have a better idea of how to fix it instead of adding more checkout overrides to project-config, I'm going to change the default branch in our tox_install.sh to "stable/pike"15:10
andreafmordred: uhm... yeah I guess it makes sense, so you always have  any(logs / docs / artifacts) in the root15:10
andreafmordred: and under logs you can have multiple folders, one per host, if applicable15:11
sambettsso then when tox_install.sh is called it will install the stable/pike version of neutron as we expected for that job, instead of installing master15:11
andreafdocs and artifacts won't ever be multinode I guess15:11
*** shardy has quit IRC15:11
mordredandreaf: yah - I mean, if you do a multinode docs job, good luck to you :)15:11
AJaegersambetts: ok, give it a try ;)15:11
andreaf:D15:11
andreafmordred: a log file that is indexed in elastic search might need one extra path - one for the single node case, one for multinode controller case15:13
andreafmordred: and potentially one for the subnode as well but that's going to be the case anyways15:13
*** lukebrowning has quit IRC15:13
*** liuyulong has quit IRC15:14
*** rbrndt has joined #openstack-infra15:15
*** sshnaidm is now known as sshnaidm|afk15:16
openstackgerritFabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant config loading in case of config issue  https://review.openstack.org/52906015:19
*** esberglu has joined #openstack-infra15:20
*** myoung is now known as myoung|bbl15:20
*** Swimingly has quit IRC15:21
*** yamamoto has joined #openstack-infra15:23
*** Swimingly has joined #openstack-infra15:24
*** shardy has joined #openstack-infra15:24
openstackgerritAndrea Frittoli proposed openstack-infra/openstack-zuul-jobs master: Use tempest-full in the integrated gate  https://review.openstack.org/52935315:24
andreafAJaeger, mordred: what do you think? ^^^15:24
*** Apoorva has joined #openstack-infra15:29
mordredandreaf: I think it's a great idea - maybe let's move the integrated-gate project-template into the normal project-templates.yaml file since it's a template we're aiming to keep?15:29
mordredandreaf: although, ignore me - let's do that as a different patch so it's clear what this one does :)15:30
mordredandreaf: in any case, I'm +2 on that change if you're satisfied that the new job is ready - I assume we have added elasticsearch paths for the controller/ path already?15:32
*** yamamoto has quit IRC15:35
andreafmordred: I have to check the e-r config for that15:36
fricklerinfra-root: we have quite a bit of backlog in the integrated gate, looks like there is a significant amount of jobs running into timeouts, if anyone could take a look15:38
jeblairfrickler: is that something a root needs to do?15:39
andreafmordred: according to http://git.openstack.org/cgit/openstack-infra/project-config/tree/roles/submit-logstash-jobs/defaults/main.yaml subfolders are searched automatically already15:39
AJaegerandreaf: tell us once you're ready to merge it...15:39
fungii only see two failing changes in the integrated gate at the moment15:40
jeblairfrickler: jobs running into timeouts should be something anyone can investigate i'd think15:40
*** lukebrowning has joined #openstack-infra15:41
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Remove github caching  https://review.openstack.org/52936115:42
andreafAJaeger, mordred: the job is ready from my pov, one thing that I'm wondering now is what is the impact in terms of files / irrelevant-files pov15:42
openstackgerritRicardo Carrillo Cruz proposed openstack-infra/zuul feature/zuulv3: Use connection type supplied from nodepool  https://review.openstack.org/50197615:42
AJaegerandreaf: you can iterate on that in tempest ;)15:42
*** slaweq has quit IRC15:43
AJaegerfrickler: looking at http://logs.openstack.org/30/528630/1/gate/openstack-tox-py35/df0d181/job-output.txt.gz I see so many errors in the log file, that I wonder it didn't fail earlier15:43
andreafAJaeger: if any projects has such settings on the legacy job today, they won't apply until there's changed to the new job name... but I guess it should be fine to fix that iteratively15:44
*** armax has joined #openstack-infra15:44
fungifrickler: that failed job which kicked out the change currently at the top of the gate seems to have errored out on a tempest test. what timeouts are you finding?15:44
andreafAJaeger: so I think the patch is ready to be merged15:44
AJaegerandreaf: agreed, you can iterate on it.15:44
*** iyamahat has joined #openstack-infra15:44
jeblairandreaf: it may also cause them to run both jobs15:44
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: DNM: Add github debugging script  https://review.openstack.org/52936215:45
*** dizquierdo has quit IRC15:45
*** iyamahat has quit IRC15:45
*** iyamahat has joined #openstack-infra15:45
*** lukebrowning has quit IRC15:45
andreafjeblair: well if the job is added to the pipeline both via template as well as directly, yeah15:45
fricklerfungi: I saw some earlier already, didn't keep a reference15:46
*** lukebrowning has joined #openstack-infra15:47
*** shu-mutou has quit IRC15:47
fungifrickler: ahh, maybe whatever the problem was, it's since subsided. what sort of "timeouts" are you talking about" job timeouts? network connection timeouts? some other sort of timeouts?15:47
AJaegerconfig-core, could you review a few of the open changes, please? I grouped them https://etherpad.openstack.org/p/Nvt3ovbn5x and ordered by prio - I'd like at least the translation and PTI jobs to go in today if you find some time for reviews. thanks!15:47
*** eharney has quit IRC15:49
fricklerfungi: jobs stuck in the middle of their stream output for hours. not sure whether these are network issues or nodes getting exponentially slow sometimes15:49
fricklerfungi: different question: can you confirm that what I did for gerrit issues earlier (service gerrit stop/start) was correct? is it normal for the start to claim "FAILED" but succeed anyway? maybe increase the timeout for the init job?15:51
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Add github debugging template  https://review.openstack.org/52936215:51
*** pcaruana has joined #openstack-infra15:51
*** lukebrowning has quit IRC15:52
jeblairAJaeger: i jumped into 528819 to review and don't understand (even after reading the commit message) why that's happening.  is there a description?15:52
AJaegerfrickler: that's not normal. Was it a specific cloud?15:52
AJaegerjeblair: that's mordred's change, let's ask him...15:52
fungifrickler: yeah, we've talked about how to solve that, the service takes too long to start and so the initscript gives up waiting... that may just go away under xenial/systemd15:52
jeblairfrickler: there's only one gerrit process and it has the right start time, so that's good.  :)15:53
fricklerAJaeger: difficult to tell while the job is still running15:53
*** coolsvap has joined #openstack-infra15:53
*** lukebrowning has joined #openstack-infra15:53
*** janki has quit IRC15:53
AJaegerjeblair: it's basically a dance: enable using new jobs, update proejct-config to use new job, cleanup15:53
jeblairAJaeger: that i understand -- the commit message says that.  it doesn't say why we're making the change at all.15:54
*** felipemonteiro has joined #openstack-infra15:54
jeblairbasically it says "we're changing jobs"  it doesn't say "this is why we're changing jobs"15:54
fungifrickler: we generally first check javamelody at https://review.openstack.org/monitoring to see whether the java garbage collection graph shows it going haywire (known behavior) or something new, and then restart it. but based on your comment in scrollback about being unable to check the connections list i expect you're not in the gerrit administrators group yet so can't get to the javamelody15:55
fungiinterface either15:55
AJaegerjeblair: I did https://review.openstack.org/#/c/528553/ to import the tox jobs and then mordred wanted to simplify - he basically said there's no need for those custom jobs, let's use the "normal" ones.15:55
*** smatzek has joined #openstack-infra15:55
andreafmordred, jeblair: is this variant issue (in zuul?) fixed now? https://github.com/openstack-infra/project-config/blob/master/zuul.d/projects.yaml#L3115:55
fricklerfungi: ah, yes, that's a different thing I wanted to ask, can I get admin rights for gerrit?15:56
fricklerfungi: also related to omer's issue earlier15:57
*** xarses_ has joined #openstack-infra15:57
jeblairAJaeger, mordred: got it, that makes sense.  :)  it's hard to keep up with all the irc chat between you -- you have a lot of things going on.  extra background in commit messages helps. :)15:57
andreafmordred: also, I just realised that integrated-gate is not used yet - its jobs are included one by one for all relevant repos - and I was wondering why - is it related to the variant issue as well?15:57
fungifrickler: luckily jm keeps some history through restarts, so looking at the gc graph in it right now i can see it was definitely getting bad right up to the time you restarted15:57
*** jamesmcarthur has quit IRC15:57
AJaegerjeblair: sorry ;( Too much balls juggling currently ;) That's why I tried to sort the changes to help with. Let me update the etherpad at least...15:57
clarkbfrickler: Gerrit's init script has a timeout for it to see whatever it is that says Gerrit is up happen but Gerrit takes so long yo start now that it misses that timeout15:58
*** jamesmcarthur has joined #openstack-infra15:58
clarkbbut the service ends up starting ok amyways15:58
*** lukebrowning has quit IRC15:58
*** caphrim007 has quit IRC15:58
jeblairandreaf, mordred: i don't know what that comment was referring to, however, all known bugs regarding variants are fixed and in production.15:58
*** caphrim007 has joined #openstack-infra15:59
fricklerclarkb: yes, if we know it takes like 180 secs instead of the default 90, we could maybe increase it. but if we migrate that to xenial anyway soon, then meh ;)15:59
fungiinfra-root: i've added frickler to the "Administrators" group in gerrit on review.openstack.org15:59
*** rossella_s has quit IRC15:59
clarkbfungi: thanks! I guess that was missed when we did initial Gerrit group updates16:00
*** lukebrowning has joined #openstack-infra16:00
AJaegerfungi, do you want to add any of the other new infra-roots as well?16:00
AJaegeror were those in already?16:00
fungifrickler: note that the gerrit ui may now show you some additional administrative options even when doing seemingly normal tasks such as code review, so please do be careful16:00
fungiAJaeger: i figure we have a general consensus that they can be added when they feel they're ready16:00
fricklerfungi: ack, thx16:00
AJaegerfungi: I see16:01
fungii did take the opportunity to clear out the launchpadsync account (wow that's been unused for ages) and several former/retired infra-core members16:01
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Use tempest-full in the integrated gate  https://review.openstack.org/52935316:01
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Make AFS marker roles honor sphinx variables  https://review.openstack.org/52573516:01
*** shu-mutou has joined #openstack-infra16:02
*** jamesmca_ has joined #openstack-infra16:02
fungiAJaeger: we've tended not to automatically add people to the gerrit administrators group until they're prepared for the fact that the ui will start showing some new and potentially dangerous options, so wouldn't want to surprise them with it16:02
*** jamesmcarthur has quit IRC16:02
AJaegerfungi: thanks for the background...16:03
clarkbalso probably want a short braindump on correct way to force merge something if necessary16:03
fungie.g., the ability to delete votes from anyone on any change16:03
fungiclarkb: we have it documented now, right?16:03
*** rossella_s has joined #openstack-infra16:03
* AJaeger would love that feature ;)16:03
clarkbfungi: we might? I sent an email a couple months back that could be adapted to docs if nothing else16:04
fungiAJaeger: with great responsibility comes great annoyance, or something like that16:04
*** caphrim007 has quit IRC16:04
AJaegerfungi: ;)16:04
clarkband now we are in a spiderman comic16:04
fricklerwhile folks are reviewing devstack-gate changes, this one is to add a working neutron-api job https://review.openstack.org/52900016:04
*** lukebrowning has quit IRC16:04
*** jcoufal has quit IRC16:04
fungiclarkb: frickler: https://docs.openstack.org/infra/system-config/sysadmin.html#force-merging-a-change16:05
*** lukebrowning has joined #openstack-infra16:06
dmsimardfungi: was I there? just wondering16:06
dmsimardin the gerrit administrator group16:06
fungidmsimard: nope, do you want to be?16:07
*** jamesmca_ has quit IRC16:07
dmsimardoh, I hadn't caught up with the backlog just yet16:07
fungihappy to add you now16:07
*** ldnunes has quit IRC16:07
dmsimardsure, I have some amount of experience managing gerrit from review.rdoproject.org :)16:07
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Use python3 for docs publication  https://review.openstack.org/52571816:07
fungidmsimard: sold! you're added now16:08
*** mriedem_away is now known as mriedem16:09
*** ykarel has quit IRC16:09
*** ykarel has joined #openstack-infra16:09
*** jamesmcarthur has joined #openstack-infra16:09
*** lukebrowning has quit IRC16:10
AJaegerclarkb: could you help reviewing the translation changes mentioned in https://etherpad.openstack.org/p/Nvt3ovbn5x , please?16:11
frickleroh, removing a vote/reviewer is indeed pretty easy to do accidentally. at least the action shows up in the log so folks can blame me when I manage that :D16:11
*** ykarel has quit IRC16:11
*** lukebrowning has joined #openstack-infra16:12
andreafclarkb, tonyb: how much longer do we plan on testing newton in the gate? is it because of the discussion on LTS that tests are still running?16:12
AJaegerandreaf: aren't we waiting for nova?16:13
fungifrickler: yup, i can attest that it shows up in the review comments given i just did more than 4k of them over the weekend16:13
andreafAJaeger: maybe - not sure16:13
fungi(granted, i used the api for that, not the webui)16:13
AJaegerandreaf: there was some discussion on the openstack-dev mailing list about nova and newton recently16:14
AJaegerandreaf: but yes, best ask others what else is needing newton16:15
andreafAJaeger: it's just that I started fixing the nnet integrated gate template usage, only to realise that projects are not running the n-net job as they should on the newton branch16:15
andreafAJaeger: and before I add back the job I thought maybe we should remove it instead from tempest and devstack-gate16:16
*** lukebrowning has quit IRC16:17
AJaegerandreaf: most projects have no newton branch anymore and thus don't need the job.16:17
fricklerfungi: I tested it on a patch of mine I was about to abandon, wanted to see whether there is any confirmation first, but the answer is no16:17
*** andreas_s has quit IRC16:18
*** ldnunes has joined #openstack-infra16:19
*** lukebrowning has joined #openstack-infra16:19
andreafAJaeger: well atm only devstack, tempest and devstack-gate have a nnet job running16:20
*** edmondsw has joined #openstack-infra16:21
*** felipemonteiro_ has joined #openstack-infra16:22
*** andreas_s has joined #openstack-infra16:22
andreafAJaeger: anyways according to the email thread netwon EOL is going to happen soon-ish so I guess I won't bother changing things now16:23
andreafAJaeger: only after nova newton is EOL16:23
*** iyamahat has quit IRC16:24
*** felipemonteiro_ has quit IRC16:24
*** lukebrowning has quit IRC16:24
*** felipemonteiro_ has joined #openstack-infra16:24
*** jcoufal has joined #openstack-infra16:25
*** edmondsw has quit IRC16:26
*** lukebrowning has joined #openstack-infra16:26
dmsimardtonyb is in new zealand so his reply might lag a bit :)16:26
*** felipemonteiro has quit IRC16:26
dmsimardNew Zealand or Australia, I forget16:26
fungiau16:26
fungiunless he's on holiday16:26
*** lukebrowning has quit IRC16:31
clarkbit is that time of year16:31
clarkbAJaeger: ya I'll take a look shortly at translation thinvs16:31
fungidmsimard: deep links working great with firefox in new ara. thanks!16:32
*** andreas_s has quit IRC16:32
*** lukebrowning has joined #openstack-infra16:32
dmsimardYay16:32
dmsimardmnaser should be credited with the fix actually16:32
fungithanks mnaser!16:33
mnaseri dont even remember how we fixed it16:33
fungi#thanks dmsimard and mnaser for getting deep-linking in ara working for firefox16:33
mnaserr maybe i do16:33
openstackstatusfungi: Added thanks to Thanks page16:33
mnaserbut yay! :P16:33
* fungi must remember to make use of #thanks more often16:34
*** rmcall has joined #openstack-infra16:34
*** lukebrowning has quit IRC16:37
*** Goneri has joined #openstack-infra16:37
*** lukebrowning has joined #openstack-infra16:38
*** rmcall has quit IRC16:38
fungiour of my depth on this ansiblelint error: http://logs.openstack.org/40/526140/5/check/openstack-zuul-jobs-linters/991b3ca/job-output.txt.gz#_2017-12-19_01_11_30_29063416:40
fungier, out of my depth16:40
fungiit seems to be complaining about a missing action in a task in another repo for a role merely referenced in this change16:40
fungiCouldn't parse task at /home/zuul/src/git.openstack.org/openstack-infra/zuul-jobs/roles/prepare-workspace/tasks/main.yaml:2 (no action detected in task. This often indicates a misspelled module name, or incorrect module path.)16:41
fungi{'name': 'Start zuul_console daemon.', 'zuul_console': None}16:41
fungisame error in both playbooks where i attempt to add the prepare-workspace role16:42
*** lukebrowning has quit IRC16:43
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184316:43
fricklerfungi: I seem to remember something about zuul needing to be installed into the linters env16:44
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185316:44
ssbarneacan anyone give some hints regarding use of boolean variables with jenkins-job-builder? somehow it seems to be permanent source of problems16:44
clarkb"This web page is slowing down your browser"16:44
*** links has quit IRC16:44
*** lukebrowning has joined #openstack-infra16:44
clarkbssbarnea: JJB largely deals with strings iirc16:45
ssbarneathese bool vars  are expanded inside groovy or bash, but jjb does expand it as True or False (note the case), which happens to be invalid bool value in groovy which is using true and false.16:45
ssbarneaclarkb: not really, if someone does not add quotes, YAML parser will make them bools.16:45
clarkbah in that case you are using a yaml boolean which gets pulled in as True/False in python which then gets stringified as True/False16:46
ssbarneaclarkb: is not JJB itself which does that, i think is a yaml "feature".16:46
ssbarneaclarkb: indeed.16:46
clarkbyes its yaml being helpful16:46
clarkbcan you just supply string values instead so that you get 'true' or 'false' in the xml?16:46
ssbarneaclarkb: now the big question is how to make this fool proof, considering that I have >50 people writing jjb definitions. is very easy to miss to add quotes around a "yes/no" value.16:47
clarkbotherwise you'll likely need to update whatever jjb modules are writing boolean values to stringify True to true instead of True16:47
clarkbalso there is #openstack-jjb now I think?16:47
*** dangers_away has quit IRC16:48
ssbarneaclarkb: how to do that? ahh, forgot about that.16:48
ssbarneait grew enough to have its own channel.16:48
clarkbssbarnea: and we stopped using it16:48
*** lucasagomes is now known as lucas-afk16:48
pabelangerokay, I've powered off kdc02.o.o, kd04.o.o is not our secondary kerberos server16:48
ssbarneaclarkb: lucky you!16:48
clarkbssbarnea: so those interested in continuing to use it collaborate in that space16:48
andreafclarkb, jeblair, mordred, AJaeger: I'm looking at projects.yaml in project-config, putting the integrated-gate template into use - it's a bit frustrating since every project has to re-include the jobs from the template in both check and gate to define irrelevant-files16:49
ssbarneaclarkb: not sure if "interested" is the best way to describe it, is more of "still forced by other reasons" ;)16:49
*** e0ne has quit IRC16:49
*** lukebrowning has quit IRC16:49
andreafso one side it feels like the template is for nothing, on the other side it's a lot of repeating the irrelevant-files bit over and over again16:49
*** gibi is now known as gibi_away16:50
*** andreas_s has joined #openstack-infra16:50
andreafis there any way to factor the irrelevant-files bit at project level into a default so we can specify it only where there is a variance?16:50
openstackgerritFabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant config loading in case of config issue  https://review.openstack.org/52906016:50
clarkbandreaf: if there are common sets of irrelevant files we can collapse them down into a yaml anchor16:50
clarkbandreaf: you still have to specify the anchor everywhere you use it but its a lot less verbose16:51
clarkb(I think smcginnis was going to look into doing some of that)16:51
*** lukebrowning has joined #openstack-infra16:51
andreafclarkb: yeah that would improve readability - but it won't be as good as having a default set16:51
openstackgerritSayali Lunkad proposed openstack/diskimage-builder master: Adding mapping for SUSE package  https://review.openstack.org/52937016:51
andreafclarkb: or perhaps can we attach relevant files to a job template in the project?16:51
smcginnisclarkb: I tried it out, but you can anchor/merge dict values but not arrays.16:52
pabelangerjeblair: when we do https://docs.openstack.org/infra/system-config/kerberos.html#no-service-outage-server-maintenance on kdc01.o.o, does that process sync ALL data to the slave? Trying to see if delete kdc01 if there is any other data we'd need to migrate when we bring kdc03.o.o (replacement)16:52
pabelangeronline16:52
clarkbandreaf: you can attach them to the more base version of the job16:52
smcginnisIt ends up being an array where one of the items is the array you've anchored.16:53
clarkbsmcginnis: wouldn't you just anchor irrelevant-files: some_anchor_here and if you want a different set of files you don't use that anchor?16:53
*** cshastri has joined #openstack-infra16:53
smcginnisclarkb: True, that's one approach. It may help get rid of some duplication at least.16:54
*** jtomasek has quit IRC16:54
smcginnisclarkb: I was really hoping we could have an anchor for the common set, then be able to add additional items as needed.16:54
*** aviau has quit IRC16:54
*** aviau has joined #openstack-infra16:54
*** andreas_s has quit IRC16:55
andreafclarkb: well the thing is the list tends to be repo specific, so attach it to the job definition would not be good16:55
*** dangers has joined #openstack-infra16:55
clarkbandreaf: ya I think thats why the anchor idea came up16:55
clarkbsince there is the nova exclude list or whatever that can be applied multiple times16:55
*** lukebrowning has quit IRC16:55
andreafclarkb: unless we can include a variable in the irrelevant file list and have project specify a value for that16:55
andreafclarkb: well still it forces you to spell out the list of jobs which means if you make a change to a template you have to go and change every place where the template is used16:56
andreafclarkb: which kind of defeats the purpose of the template itself16:56
clarkbyes I think you are finding two features which don't work together because they stand in opposition16:58
clarkbone is specifically to say "this is common lets do it the same everywhere" and the other ends up being "we are all special and need our own stuff"16:59
pabelangermordred: mind looking at https://review.openstack.org/526708/ and adding the usage of serial to base-test (eventually base) for shared ansible_host in zuulv3?16:59
*** dizquierdo has joined #openstack-infra16:59
*** ijw has joined #openstack-infra16:59
clarkbandreaf: I think both are valid use cases, its just that I'm not sure they will always work well together16:59
andreafclarkb: well kinda - the file list could really mostly be done at project level but we're forced to do it at job level17:00
andreafclarkb: so a job-template and a file list at project level would work well together17:00
*** ykarel has joined #openstack-infra17:01
*** hasharAway is now known as hashar17:01
andreafclarkb: if you look at projects.yaml it doesn't look like they'll ever work well together17:01
*** dangers is now known as dangers_away17:01
*** gyee has joined #openstack-infra17:01
clarkbandreaf: they could if we also standardized project structure more17:02
clarkbandreaf: so that you could rely on common sets of irrelevant files for jobs17:02
clarkbbut we've not done that17:02
andreafclarkb: yeah that's kind of hard to fix now :D17:03
*** priteau has joined #openstack-infra17:04
clarkbfungi: do you know if the nodepool changes we needed got in?17:04
* clarkb is finishing up reviews for AJaeger but then I think we should consider another nodepool launcher restart if they have17:04
andreafclarkb17:05
andreafclarkb: irrelevant-files are proper regex right?17:05
AJaegerclarkb: thanks17:05
AJaegerandreaf: yes, they can be17:05
andreafclarkb, AJaeger: something like  - ^neutron/tests/unit/.*$ can be made into a regex that matches for all projects17:05
AJaegerclarkb: indeed, that is fine for all projects - even if it matches only one ;)17:06
AJaegerandreaf: and you can use a *list* of regexes.17:06
AJaegerclarkb: sorry, meant andreaf17:06
andreafclarkb, AJaeger: that's what I see as the most common exception for integration tests to job level file exclusions17:06
AJaegercalbers: tahnks17:06
* AJaeger cannot type anymore ;(17:06
AJaegercalbers: sorry, wanted to say:17:07
clarkbAJaeger: its ok I was never able to type17:07
AJaegerclarkb: Thanks17:07
andreafAJaeger, clarkb: well I'd rather do something like ^[a-z_\-]*/tests/'17:07
calbersAJaeger: np :D17:07
AJaegerandreaf: feel free to start with something, and then iterate as you get suggestions.17:08
AJaegerandreaf: I'm happy to review a change...17:08
*** priteau has quit IRC17:08
mriedemclarkb: where is the multiline handling for indexed logs again? system-config?17:09
mriedemtrying to figure out why i can't do multiline queries on the placement-api logs17:09
clarkbmriedem: openstack-infra/logstash-filters17:10
mriedemthanks17:10
fungiclarkb: the nodepool changes from last night (-1 unlimited quota and bug fixes) merged as far as i know17:11
pabelangernice17:11
*** pcichy has joined #openstack-infra17:13
openstackgerritMerged openstack-infra/project-config master: Prevent duplicate pull_from_zanata on the same repo  https://review.openstack.org/52851817:14
*** iyamahat has joined #openstack-infra17:14
sambettsAJaeger: changing the default branch in the tox_install.sh worked: https://review.openstack.org/#/c/522315/17:14
*** smatzek has quit IRC17:16
openstackgerritTytus Kurek proposed openstack-infra/project-config master: Add charm-interface-designate project  https://review.openstack.org/51733617:17
openstackgerritJavier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support  https://review.openstack.org/52937617:17
*** udesale has quit IRC17:18
mriedemclarkb: oh i suppose it doesn't work because we run placement-api under wsgi and so it doesn't match the syslog_program entry here17:19
mriedemhttp://git.openstack.org/cgit/openstack-infra/logstash-filters/tree/filters/openstack-filters.conf#n4617:19
openstackgerritTytus Kurek proposed openstack-infra/project-config master: Add charm-interface-designate project  https://review.openstack.org/51733617:19
mriedemmatch => { "message" => "(?m)^(%{TIMESTAMP_ISO8601:logdate}|%{SYSLOGTIMESTAMP:logdate})%{SPACE}(%{DATA:syslog_program}(?:\[%{POSINT:syslog_pid}\])?:|%{NUMBER:pid})?%{SPACE}?(?<loglevel>AUDIT|CRITICAL|DEBUG|INFO|TRACE|WARNING|ERROR) \[?\b%{NOTSPACE:module}\b\]?%{SPACE}?%{GREEDYDATA:logmessage}?" }17:19
*** florianf has quit IRC17:19
*** cshastri has quit IRC17:20
mriedemDec 20 15:15:20.666337 ubuntu-xenial-citycloud-lon1-0001533915 devstack@placement-api.service[15195]: ERROR nova.api.openstack.placement.handler17:20
mriedemcompared to17:20
mriedemDec 20 15:15:20.747237 ubuntu-xenial-citycloud-lon1-0001533915 nova-conductor[21763]: ERROR nova.conductor.tasks.migrate17:20
*** ykarel has quit IRC17:21
*** andreas_s has joined #openstack-infra17:22
clarkbmriedem: that could be it there is a logstash grok filter checking website that makes testing easy17:22
*** caphrim007 has joined #openstack-infra17:22
mriedemyeah i'm poking through that17:23
*** dprince has quit IRC17:23
fricklerfungi: like this maybe https://github.com/openstack-infra/project-config/blob/master/tox.ini#L1417:23
*** d0ugal has quit IRC17:23
fricklerbbiab17:24
openstackgerritJavier Peña proposed openstack-infra/puppet-openstackci master: Add AFS mirror support for RHEL/CentOS  https://review.openstack.org/52873917:24
openstackgerritTytus Kurek proposed openstack-infra/project-config master: Add charm-interface-designate project  https://review.openstack.org/52937917:24
*** trown|ruck is now known as trown|lunch17:25
fungifrickler: thanks, i'll see if that helps17:25
*** sbezverk has joined #openstack-infra17:25
mriedemhmm actually the grok pattern should work here17:25
openstackgerritTytus Kurek proposed openstack-infra/project-config master: Add charm-interface-designate project  https://review.openstack.org/51733617:25
mriedemyeah i don't get it, the grok patterns match17:26
AJaegersambetts: yeah!17:29
*** Apoorva has quit IRC17:30
*** dtantsur is now known as dtantsur|afk17:34
*** d0ugal has joined #openstack-infra17:35
AJaegermordred: comment on https://review.openstack.org/#/c/525736 - we should handle doc/requirements.txt now, shouldn't we?17:36
AJaegermordred: I'm testing that change with https://review.openstack.org/529382 now17:39
*** andreas_s has quit IRC17:40
*** lukebrowning has joined #openstack-infra17:40
*** neiljerram has quit IRC17:42
*** neiljerram has joined #openstack-infra17:44
*** jamesmcarthur has quit IRC17:44
*** andreas_s has joined #openstack-infra17:45
*** iyamahat has quit IRC17:45
*** lukebrowning has quit IRC17:45
*** fultonj has quit IRC17:47
clarkbjeblair: are you around? looks like on nl01 at least all three of your changes have merged and the tip of master nodepool is installed. I think we are ready to do a service restart17:47
clarkb(happy to do it but want to makle sure there isn't anything else we need to do first as I missed a good chunk of the debugging yesterday)17:47
*** lukebrowning has joined #openstack-infra17:48
*** mihalis68 has quit IRC17:48
openstackgerritJeremy Stanley proposed openstack-infra/zuul-base-jobs master: Add generic base and base-test jobs/playbooks  https://review.openstack.org/52614017:49
jeblairclarkb: yep, i think we can restart and try again.  please go for it.17:49
*** andreas_s has quit IRC17:49
clarkbok starting with nl01 now17:49
fungii'm around for a bit if something goes sideways17:49
*** rmcall has joined #openstack-infra17:50
clarkbwill watch for exceptions in its debug log before doing 0217:50
clarkbjeblair: tobiash http://paste.openstack.org/show/629485/ seems to be happening a fair bit17:52
*** derekh has quit IRC17:52
*** lukebrowning has quit IRC17:52
tobiashclarkb: that doesn't look good17:52
*** xarses_ has quit IRC17:53
clarkbself.provider.pools.get(node.pool) is where pool comes from17:53
clarkbso guessing its not in that dict17:53
*** xarses_ has joined #openstack-infra17:53
*** smatzek has joined #openstack-infra17:53
*** sambetts is now known as sambetts|afk17:53
tobiashclarkb: do you have pools in the config without labels?17:54
*** lukebrowning has joined #openstack-infra17:54
tobiashclarkb: ah, I think I have a clue17:54
clarkbtobiash: no the pool there (belonging to mtl01) has labels17:54
clarkband there is only one pool attached to that provider17:55
tobiashclarkb: you do sharding of the providers with two launchers right?17:55
clarkbtobiash: correct17:55
tobiashso launcher 1 has provider/pool a and launcher 2 provider/pool b17:55
clarkbyes17:56
clarkbin this case launcher 1 is showing the exception and it has config for provider/pool a that is failing17:56
clarkbnot we shard on providers so providers in launcher 1 are not in launcher 217:56
tobiashhm, that crashes my theory17:56
*** hemna_ has quit IRC17:56
clarkbalso it may have stopped? I'm going to run a grep to check17:57
clarkbya it happened 3 times and hasn't happened since (but that could be due to demand I supposed)17:57
clarkbtobiash: is it possible that is due to the restart itself? the zookeper node wasn't fully updated with pool information so when it started up again it had a mismatch between node.pool and pool.name?17:58
*** lukebrowning has quit IRC17:59
tobiashclarkb: is there maybe a stale vm in this cloud which states it's from a different provider?17:59
clarkblooks like its checking node.provider and node.pool and if we don't update them atomically I could see something like this happening17:59
tobiashclarkb: yes that could be possible18:00
tobiashwe should add a check to none there to not crash the calculation18:00
*** lukebrowning has joined #openstack-infra18:00
*** iyamahat has joined #openstack-infra18:00
*** jascott1 has joined #openstack-infra18:00
clarkbas for a stale vm you mean one as listed by zookeeper? or by the cloud?18:00
tobiashclarkb: ah wait18:01
tobiashit's quota needed for node type18:01
tobiashcould it be that this handler tried to create a node for a label it's not configured for?18:01
clarkbI'm not sure how that would happen18:01
clarkb(also I don't think we log enough info to know18:01
tobiashme neither18:02
jeblairtobiash: it looks like this is calculated the quota used by existing nodes18:02
clarkbthat provider has a pool with all of our labels in it. I don't see any missing labels18:02
clarkbIn the config that is18:02
jeblairyeah, it would be nice to know what the value of node.pool is18:03
*** dprince has joined #openstack-infra18:03
clarkbjeblair: ya I'm wondering if this is stale state left over from the restart18:03
clarkbbecause its checking multiple values that aren't necessarily updated atomically18:03
clarkbgrep says it has still only happened the three times18:04
jeblairclarkb: especially if we don't see this again for a while, i think that's a good assumption.  maybe we should just add the handler tobiash suggested, maybe with an extra bit of logging so we know what node triggered the error18:04
jeblairthat would let us go back through the logs and confirm that node was in an expected-bad state (and cleaned up by another thread likely)18:04
clarkbThat seems like a good idea. Do we awnt to get that in before restarting 02?18:04
tobiashclarkb, jeblair: looking at the code it at least matched already the provider so this might be a stale node entry in zk18:05
*** lukebrowning has quit IRC18:05
jeblairclarkb: nah, it'll take an hour to merge; let's restart so we can continue to observe for any other new behaviors18:05
*** fultonj has joined #openstack-infra18:05
AJaegerfungi: rebase https://review.openstack.org/#/c/525271/18:05
clarkbok I'm moving on to 02 then and if no one has a patch up get I can work on a patch too18:05
*** Apoorva has joined #openstack-infra18:06
*** Apoorva has quit IRC18:06
clarkbthat was quick 02 is done18:06
*** lukebrowning has joined #openstack-infra18:07
clarkbI'm working on a patch to do more logging now18:07
*** Apoorva has joined #openstack-infra18:07
fungiAJaeger: oh, thanks! gertty wasn't making it obvious18:08
AJaegerfungi: the webui did18:08
fungiyep, i expect i got used to the old merge-check comments from zuul18:08
*** neiljerram has quit IRC18:09
fungistrangely it doesn't say it needs a rebase, just that a rebase is possible18:09
*** edmondsw has joined #openstack-infra18:09
AJaegeryeah, this is confusing.18:09
*** claudiub has quit IRC18:09
AJaegerMy role of thumb for teh webui: It does not merge, orange circle at parent -> Let's rebase18:09
AJaegernot intuitive ;(18:09
AJaegerBut first i check for dependencies - stack and depends-on18:10
fungii'll let zuul finish just in case we get more details out of it this time18:10
AJaegersure18:10
fungii expect it's getting an error back when asking gerrit to merge the commit, and if so i can hopefully at least find that in the scheduler log18:11
*** makowals has joined #openstack-infra18:11
*** lukebrowning has quit IRC18:11
AJaegerand if we can pass that back to gerrit for display, it would be really great18:11
fungiworst case, if the logs aren't helping, i can test pushing the submit button for it in gerrit and see what gerrit tells me then18:11
clarkbon nl02 I'm seeing timeouts for ssh access and the citycloud quota issue but both are "normal" I think18:12
*** lukebrowning has joined #openstack-infra18:13
*** edmondsw has quit IRC18:13
*** Apoorva has quit IRC18:13
fungifrickler: your suggestion worked, thanks! now 526140 is all green and ready for review18:13
openstackgerritClark Boylan proposed openstack-infra/nodepool feature/zuulv3: Log unknown providers during quota calculation  https://review.openstack.org/52939118:15
clarkbtobiash: jeblair ^ something like that maybe?18:15
jeblairlgtm18:16
*** e0ne has joined #openstack-infra18:16
clarkbNode type even has a __repr__ (checked for that to amek sure log would stringify properly18:17
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Fix github caching  https://review.openstack.org/52939218:17
jeblairya18:17
AJaegerfungi, let me add those two to my review etherpad at https://etherpad.openstack.org/p/Nvt3ovbn5x18:17
*** yamahata has joined #openstack-infra18:17
AJaegerhope others go over that and see it...18:18
*** lukebrowning has quit IRC18:18
tobiashclarkb: lgtm, thanks18:19
*** Apoorva has joined #openstack-infra18:19
*** Apoorva has quit IRC18:24
*** rbrndt has quit IRC18:25
*** jamesmcarthur has joined #openstack-infra18:28
*** raissa has joined #openstack-infra18:30
*** rhallisey has quit IRC18:31
*** Douhet has quit IRC18:31
*** jamesmcarthur has quit IRC18:32
*** Apoorva has joined #openstack-infra18:34
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Fix github caching  https://review.openstack.org/52939218:34
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Use connection type supplied from nodepool  https://review.openstack.org/50197618:35
jlvillalAre there any other values allowed for 'logfile' in http://zuulv3.openstack.org/stream.html?uuid=a9610cf1fcfc4a4588545c077511c856&logfile=console.log  ?18:37
clarkbjlvillal: not currently18:38
clarkbjlvillal: on the roadmap is support for being able to stream arbitrary files but getting feature parity (console log streaming) with old zuul is/was focus18:38
jlvillalclarkb, Thanks :) I was hoping to see what was happening during the devstack run part. I'll wait.18:39
fungiAJaeger: so the zuul log indicates there was an error returned by gerrit when it tried to report on that change, but not what the error was: http://paste.openstack.org/show/62948818:39
funginext step, trying the submit button myself18:39
*** jtomasek has joined #openstack-infra18:40
jeblairfungi: there may be something in gerrit's sshd log18:40
fungi"Code Review - Error... Failed to submit 1 change due to the following problems: Change 525271: depends on change that was not submitted"18:41
fungijeblair: good point, i'll take a peek in there too18:41
*** jascott1 has quit IRC18:41
fungiAJaeger: that at least suggests you're onto something with the stale parent revision18:41
*** jascott1 has joined #openstack-infra18:41
fungii wonder if there's at least a way to pipe an error message from a gerrit api call back up through that exception18:42
*** lukebrowning has joined #openstack-infra18:43
*** jascott1 has quit IRC18:43
*** jascott1 has joined #openstack-infra18:45
fungijeblair: i can definitely see the api call in the gerrit ssh log, but no resulting error. this is all it has:18:45
fungi[2017-12-20 18:32:51,967 +0000] ed1dcee1 ufcg-oneview-ci a/17998 gerrit.query.--format.json.--all-approvals.--comments.--commit-message.--current-patch-set.--dependencies.--files.--patch-sets.--submit-records.525271 2ms 14ms 018:45
fungid'oh, wait, that's the wrong account18:46
fungisomeone has some seriously misconfigured zuul there18:46
fungioh, i misread it, that's just a query18:46
*** jamesmcarthur has joined #openstack-infra18:46
fungi[2017-12-20 18:32:35,125 +0000] ede14926 zuul a/22348 gerrit.review.--project.openstack-infra/project-config.--message.Build succeeded (gate pipeline).18:47
fungi- openstack-zuul-jobs-linters http://logs.openstack.org/71/525271/2/gate/openstack-zuul-jobs-linters/5217e97/ : SUCCESS in 5m 08s18:47
fungi.--submit.--label.Verified=2.525271,2 1ms 819ms 118:47
fungibut no error18:47
*** lukebrowning has quit IRC18:48
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Add github debugging template  https://review.openstack.org/52936218:49
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add ns2.openstack.org  https://review.openstack.org/52939818:49
jeblairfungi: bummer :(18:49
jeblairfungi: i thought zuul logged output from command errors if it got them18:50
jeblairfungi, clarkb: can you approve 529398 real quick?18:50
* clarkb looks18:50
*** lukebrowning has joined #openstack-infra18:51
fungiapproved, going to find lunch now18:52
*** ijw has quit IRC18:52
*** lukebrowning has quit IRC18:55
*** jamesmcarthur has quit IRC18:57
*** lukebrowning has joined #openstack-infra18:57
*** coolsvap has quit IRC19:00
*** dizquierdo has quit IRC19:00
*** myoung|bbl is now known as myoung19:01
*** lukebrowning has quit IRC19:01
*** lukebrowning has joined #openstack-infra19:03
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fixed Speakers API issues  https://review.openstack.org/52940019:06
openstackgerritMerged openstack-infra/openstackid-resources master: Fixed Speakers API issues  https://review.openstack.org/52940019:06
*** trown|lunch is now known as trown|brb19:08
*** trown|brb is now known as trown19:08
*** lukebrowning has quit IRC19:08
*** trown is now known as trown|ruck19:08
*** jamesmcarthur has joined #openstack-infra19:09
*** lukebrowning has joined #openstack-infra19:09
*** salv-orl_ has quit IRC19:14
*** lukebrowning has quit IRC19:14
*** salv-orlando has joined #openstack-infra19:14
*** lukebrowning has joined #openstack-infra19:15
*** makowals has quit IRC19:15
*** jamesmcarthur has quit IRC19:16
pabelangerclarkb: fungi: mind a +3 on https://review.openstack.org/529189/ will fix up all-clouds.yaml, which we talked about yesterday19:18
*** salv-orlando has quit IRC19:19
clarkbdone19:19
*** lukebrowning has quit IRC19:20
*** bobh has quit IRC19:20
clarkbI've got to convert my office to a guest room so will be afk for lunch longer than usual. I haven't noticed any problems iwth nodepool launcher other than the thing identified earlier19:21
jeblairclarkb, tobiash: yeah, graphs look okay so far19:22
tobiash:)19:22
*** shardy has quit IRC19:23
*** lukebrowning has joined #openstack-infra19:24
pabelangernice! so that means we can remove max-servers now that nodepool knows the quota?19:24
fungiclarkb: i know the feeling. all my breaks have been occupied by prepping guest accommodations, wrapping gifts, preparing any food which will keep until people arrive, wrapping gifts. i'm already ready for this to be over :/19:25
*** rbrndt has joined #openstack-infra19:25
fungiat least smoking meats will give me an opportunity to take long breaks outside in the cold away from the visiting hordes19:25
*** jamesmcarthur has joined #openstack-infra19:26
* bkero takes pity on other family and just flees the continent for the holidays.19:26
clarkbpabelanger: maybe? we might want to track down the thing I identified once extra logging is in just to make sure ist not some major problem first19:26
clarkbpabelanger: but also we don't need to remove that until we start adding different sized labels19:26
pabelangerclarkb: okay, cool. Which issue is that?19:27
clarkbpabelanger: http://paste.openstack.org/show/629485/ I think its mostly a non issue but https://review.openstack.org/#/c/529391/ should help confirm that19:28
*** felipemonteiro__ has joined #openstack-infra19:28
*** CrayZee has joined #openstack-infra19:28
pabelangerah19:28
*** bobh has joined #openstack-infra19:28
*** pcaruana has quit IRC19:29
pabelangerI see some infracloud SSH timeout errors, wonder if we are having an issue with a compute node19:29
pabelangerwill look in a moment19:29
*** lukebrowning has quit IRC19:29
clarkbpabelanger: also is toady a good day to talk about gerrit things?19:29
clarkbI can do that now or after lunch if it works19:29
pabelangerclarkb: yah, less do after lunch. Just getting out of a meeting myself and need some food19:30
clarkbok19:30
*** lukebrowning has joined #openstack-infra19:30
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Migrate special docs jobs to build-openstack-sphinx-docs  https://review.openstack.org/52573619:31
AJaegermordred: updated files list ^19:31
*** felipemonteiro_ has quit IRC19:32
*** ijw has joined #openstack-infra19:32
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Migrate special docs jobs to build-openstack-sphinx-docs  https://review.openstack.org/52573619:33
*** bobh has quit IRC19:33
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Update get speaker query  https://review.openstack.org/52940219:33
*** bobh has joined #openstack-infra19:33
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Set remote url on every getRepo in merger  https://review.openstack.org/52929319:34
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: DNM: shall fail  https://review.openstack.org/52940319:34
openstackgerritMerged openstack-infra/elastic-recheck master: Query for 1719469  https://review.openstack.org/50729419:34
openstackgerritMerged openstack-infra/openstackid-resources master: Update get speaker query  https://review.openstack.org/52940219:34
mordredfungi, pabelanger: mind +A'ing this real quick: https://review.openstack.org/#/c/529203/19:34
AJaegerconfig-core, I updated https://etherpad.openstack.org/p/Nvt3ovbn5x with list of reviews I'd like to see go in. could you grab a few, please?19:35
mordredAJaeger: lgtm19:35
AJaegermordred: that one is also on my etherpad ;)19:35
*** lukebrowning has quit IRC19:35
AJaegermordred: thanks. The test change worked fine, so I'm ready to take the next steps19:35
pabelangerHmm, I think we might have a bad HDD in infracloud-vanilla: http://paste.openstack.org/raw/629492/19:35
pabelangerlooking at compute026 now19:36
*** dsariel has quit IRC19:36
pabelangerYah, not looking healthy19:37
AJaegermordred: did you see my comment on https://review.openstack.org/#/c/525734/4/zuul.d/jobs.yaml19:37
*** lukebrowning has joined #openstack-infra19:37
pabelanger[Tue Nov  7 12:44:54 2017] EXT4-fs error (device sda1) in ext4_writepages:2618: IO failure19:37
*** jpena is now known as jpena|off19:37
pabelangerI'm going to take compute026.vanilla.ic.o.o offline on the controller19:38
*** bobh has quit IRC19:38
* AJaeger waves good night19:38
*** rlandy|rover is now known as rlandy|rover|brb19:39
jlvillalIs there an easy way to run jobs for different branches in a project? We have our ironic-tempest-plugin which is branchless (only master). We want to test changes to it against ironic branches: master (working), stable/pike, and stable/ocata19:39
jlvillalI'm unsure how to do that :(19:39
mordredAJaeger: woot! looking at comment19:40
*** jascott1 has quit IRC19:41
openstackgerritMerged openstack-infra/system-config master: Add ns2.openstack.org  https://review.openstack.org/52939819:41
openstackgerritMerged openstack-infra/system-config master: Remove infra-files-ro and infra-files-rw  https://review.openstack.org/52918919:41
*** jascott1 has joined #openstack-infra19:41
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fix on speakers query  https://review.openstack.org/52940419:41
*** lukebrowning has quit IRC19:42
openstackgerritMerged openstack-infra/openstackid-resources master: Fix on speakers query  https://review.openstack.org/52940419:42
*** lukebrowning has joined #openstack-infra19:43
*** ijw has quit IRC19:43
*** jascott1 has quit IRC19:43
*** ijw has joined #openstack-infra19:44
*** bobh has joined #openstack-infra19:44
*** pcaruana has joined #openstack-infra19:48
*** bobh has quit IRC19:49
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: DNM: shall fail  https://review.openstack.org/52940319:49
openstackgerritMerged openstack-infra/zuul-jobs master: Fix build-javascript-content success-url  https://review.openstack.org/52920319:50
*** jtomasek has quit IRC19:50
*** lukebrowning has quit IRC19:51
tobiashis something broken with log streaming?19:52
tobiashit seems to randomly work and randomly not19:52
*** lukebrowning has joined #openstack-infra19:52
*** pcaruana has quit IRC19:54
*** bobh has joined #openstack-infra19:55
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Switch remaining docs publications jobs to sphinx  https://review.openstack.org/52573419:56
fungitobiash: randomly works/fails for the same build, or works at random for some builds and not others?19:57
*** edmondsw has joined #openstack-infra19:57
*** lukebrowning has quit IRC19:57
fungithe latter usually happens if we crash zuul-web on an executor (there was trouble for a while where memory pressure from concurrent jobs was causing the oom killer to snipe zuul-web processes)19:58
tobiashfungi: some work, others not19:58
*** ijw has quit IRC19:59
pabelanger#status log Disabled compute026.vanilla.ic.o.o due to hard disk being in read-only mode19:59
openstackstatuspabelanger: finished logging19:59
*** lukebrowning has joined #openstack-infra19:59
*** bobh has quit IRC19:59
*** mkoderer__ has quit IRC20:00
openstackgerritMerged openstack-infra/project-config master: Small refactor of translation scripts  https://review.openstack.org/52904720:01
*** dprince has quit IRC20:01
*** ijw has joined #openstack-infra20:01
mordredfungi, pabelanger: got a sec for an easy +3? https://review.openstack.org/#/c/528649 - https://review.openstack.org/#/c/528500 is also tiny and easy20:01
*** edmondsw has quit IRC20:02
fungitobiash: looks like we still have the oom killer situation going on... one happened today on ze02 (i'm still looking through dmesg on the other executors for similar occurrences)20:03
*** lukebrowning has quit IRC20:04
tobiashah, ok20:04
*** eharney has joined #openstack-infra20:04
fungiso if the zuul-executor child handling the web socket ended up getting killed, then any jobs run by that executor won't have a working web stream20:05
smcginnisWhat does that mean when a job shows it's half way in zuulv3.o.o but if I try to look at the console it shows "End of Stream". Is it actually running?20:05
*** lukebrowning has joined #openstack-infra20:05
fungismcginnis: that's what we're talking about20:05
fungitobiash and i20:05
*** ijw has quit IRC20:05
*** bobh has joined #openstack-infra20:05
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix github caching  https://review.openstack.org/52939220:05
smcginnisfungi: Ah OK. Need to read scrollback. Does that mean my patch is doomed again?20:06
*** pcaruana has joined #openstack-infra20:06
*** slaweq_ has joined #openstack-infra20:06
funginot necessarily, no20:07
mordredfungi: fwiw - I'm about half done with a reworking of the mechanics of that ... but I doubt it'll be ready today20:07
fungiyeah, so for example ze02 has had oom incidents since the last time zuul-executor was started there a couple weeks ago20:08
*** lukebrowning has quit IRC20:10
*** bobh has quit IRC20:10
*** jamesmcarthur has quit IRC20:11
*** lukebrowning has joined #openstack-infra20:11
*** smatzek has quit IRC20:12
*** pcaruana has quit IRC20:13
*** rlandy|rover|brb is now known as rlandy|rover20:14
*** bobh has joined #openstack-infra20:14
*** ldnunes has quit IRC20:15
*** lukebrowning has quit IRC20:16
*** amoralej is now known as amoralej|off20:17
*** lukebrowning has joined #openstack-infra20:19
*** bobh has quit IRC20:19
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Set remote url on every getRepo in merger  https://review.openstack.org/52929320:19
*** jamesmcarthur has joined #openstack-infra20:23
*** lukebrowning has quit IRC20:23
*** bobh has joined #openstack-infra20:24
*** jamesmcarthur has quit IRC20:27
*** bobh has quit IRC20:28
jeblairmordred: mechanics of what?20:31
jeblairjlvillal: did you get your question answered?20:31
jlvillaljeblair, No, I didn't20:32
jeblairfungi, tobiash, mordred: i think we need a ram governor in the executor, not just load average.20:32
jlvillaljeblair, I'm trying this, but not totally sure if it works or not: https://review.openstack.org/#/c/529386/20:32
tobiashjeblair: weren't there patches already for this?20:33
openstackgerritMatt Riedemann proposed openstack-infra/openstack-zuul-jobs master: Run the CachingScheduler in the nova-next job  https://review.openstack.org/52941020:33
tobiashor am I mistaken there?20:33
jeblairtobiash: maybe? i think SpamapS was going to look into it... i'm not sure the status20:33
tobiashjeblair: https://review.openstack.org/#/c/508960/20:33
fungijeblair: may help, but governors based on trailing indicators still have a tendency to get slammed by spikes if it's picking up jobs fast enough20:34
fungiwe saw that with the load average governor when there was memory pressure on the scheduler resulting in atypical batching20:35
jeblairfungi: indeed, though i haven't heard a better idea20:35
fungiby the time it measured the load average and noticed it was too high, it was several times higher than the upper bound we configure20:35
fungii wonder whether there's a way to mark some processes as higher priority so the oom killer will attempt to preserve them20:36
jeblairjlvillal: even the legacy devstack playbooks honor the checked out versions of repos, so you should be able to set 'override-checkout' in the job20:36
jlvillaljeblair, Okay. I haven't heard of that before. I'll search for an example of it. Thanks.20:37
jeblairjlvillal: https://docs.openstack.org/infra/zuul/feature/zuulv3/user/config.html#attr-job.override-checkout20:37
jlvillalThanks!20:37
jeblairjlvillal: so maybe try a change like the one you linked, but without the depends-on, and using that instead of the var20:37
jeblairjlvillal: (that's basically the zuulv3 implementation of the v2/devstack-gate thing you were poking at)20:38
jlvillaljeblair, Thanks. Will do.20:38
tobiashfungi: for ram I think this could at least work better than load as the measurement ram usage doesn't have a delay of several minutes20:38
fungitobiash: i agree, i think it's at least a piece of the solution20:38
jeblairthen next is probably tuning both of them -- i think we're pretty good on the load average normally.20:39
fungilooks like it's possible to override oom priorities via writing a priority integer into /proc/NNNN/oom_adj20:39
fungihaving the oom killer kill the ansible children is way less of a concern than having it kill other executor processes20:39
jeblairfungi: i wonder if there's a way to set that for a pgroup20:40
fungilooking20:40
jeblairor i guess we could leave it normal for ansible procs, and just lower the priority for the 2 executor procs20:40
jeblairor is it raise?  "be more important" whichever that direction is.  :)20:41
fungidefault is 0 i think. higher values make the processes more disposable20:41
*** links has joined #openstack-infra20:41
funginegative values make it try to preserve them20:42
jeblairok, so we could just set it to -1 on executor procs20:42
jeblairdoes that require root or a capability?20:42
funginot sure yet. still digging in since i've never done oom-killer tuning before20:43
jeblair-rw-r--r-- 1 root root 0 Dec 20 20:43 oom_adj20:43
fungi" Any particular process leader may be immunized against the oom killer if the value of its /proc/<pid>/oomadj is set to the constant OOM_DISABLE (currently defined as -17)."20:43
fungithat suggests to me that it's inherited by the pgroup for the leader20:44
fungihttps://linux-mm.org/OOM_Killer20:44
jeblairzuul@ze01:/proc/10839$ echo "-1" > oom_adj20:45
jeblair-su: oom_adj: Permission denied20:45
fungiapparently just nicing the ansible children might have the intended effect?20:45
fungi"'Niced' processes have their scores increased, and long running processes have their scores decreased"20:45
jeblair'long running processes"20:45
jeblair?20:45
fungis = int_sqrt(int_sqrt(run_time));20:45
fungiwhich leads me to wonder why we'd kill the web socket listener children20:46
*** links has quit IRC20:47
jeblairfungi: i believe they are threads now, so all the streaming is in one proc20:47
fungilooks like it divides the score by the sqrt of cpu_time and fourth root of run_time20:47
jeblair0/X = 0 ?20:48
*** lukebrowning has joined #openstack-infra20:48
jeblairso perhaps none of that matters for these since they start with a 0 score?20:48
fungidifferent score here20:48
fungii don't think these are the oom_adj value20:48
fungiit's seeded by points = p->mm->total_vm;20:49
fungiso these are basically rules for comparing virtual memory consumed20:49
SpamapSDo I need to pick the RAM governor back up?20:49
SpamapSI think I used the wrong stat to calc available RAM20:50
SpamapSIt's worth noting btw..20:50
SpamapSThat we're duplicating a lot of what Kubernetes does. ;)20:50
jeblairSpamapS: that would be great, thanks :)20:51
* SpamapS blows the dust off the old patch20:51
*** felipemonteiro_ has joined #openstack-infra20:53
*** felipemonteiro__ has quit IRC20:53
*** jcoufal has quit IRC20:53
*** lukebrowning has quit IRC20:53
*** catintheroof has joined #openstack-infra20:54
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck master: Add query for volume backup bug 1739482  https://review.openstack.org/52941120:58
openstackbug 1739482 in Cinder "test_snapshot_backup fails to build backup due to MessagingTimeout" [Undecided,New] https://launchpad.net/bugs/173948220:58
*** dprince has joined #openstack-infra21:00
*** dsariel has joined #openstack-infra21:00
*** lukebrowning has joined #openstack-infra21:00
*** jascott1 has joined #openstack-infra21:01
fungijeblair: so the web socket listener is a thread of the main daemon process? it looks like child process memory also gets counted for parent processes21:01
fungias for the adjustment knob, that's used to bitshift the total score21:01
*** jascott1 has quit IRC21:01
SpamapShm... did the load governor get reverted?21:01
jeblairfungi: it's a child process -- but it has no children itself, only threads.21:01
* SpamapS is merging and.. confused21:01
jeblairSpamapS: not to my knowledge21:01
*** jascott1 has joined #openstack-infra21:01
*** ijw has joined #openstack-infra21:01
*** ijw has quit IRC21:02
*** kgiusti has left #openstack-infra21:02
*** ijw has joined #openstack-infra21:02
fungiso the adjustment knob is basically a log base 2 scale21:02
*** kgiusti has joined #openstack-infra21:03
*** priteau has joined #openstack-infra21:04
*** lukebrowning has quit IRC21:05
openstackgerritMerged openstack-infra/zuul-base-jobs master: Initial boilerplate, packaging and testing  https://review.openstack.org/52613921:06
*** lukebrowning has joined #openstack-infra21:06
SpamapSoh hah no, the merge was just so perturbed it moved manageLoad 900 lines21:07
*** smatzek has joined #openstack-infra21:07
* SpamapS tries a different merge method for his rebase21:07
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck master: Add query for VolumeTypes.project lazy-load bug 1739484  https://review.openstack.org/52941321:08
openstackbug 1739484 in Cinder "DetachedInstanceError trying to lazy-load VolumeTypes.projects" [Undecided,Confirmed] https://launchpad.net/bugs/173948421:08
*** priteau has quit IRC21:09
*** smatzek has quit IRC21:09
*** jamesmcarthur has joined #openstack-infra21:10
openstackgerritMerged openstack-infra/nodepool feature/zuulv3: Log unknown providers during quota calculation  https://review.openstack.org/52939121:10
pabelangerclarkb: ^maybe restart to debug more?21:12
*** lukebrowning has quit IRC21:12
clarkbpabelanger: ya its happened more on nl0221:12
clarkb(but not on 01)21:12
pabelangerk, still need for puppet to run21:12
*** lukebrowning has joined #openstack-infra21:13
*** bobh has joined #openstack-infra21:15
*** salv-orlando has joined #openstack-infra21:16
jlvillaljeblair: Question on the docs: https://docs.openstack.org/infra/zuul/feature/zuulv3/user/config.html#attr-job.run21:17
jlvillaljeblair: So if I supply a job.run, it uses mine and also the parents. Correct?21:17
jlvillalThe docs make it sounds like it doesn't. Or at least unclear.21:17
*** CrayZee has left #openstack-infra21:17
*** lukebrowning has quit IRC21:17
jeblairjlvillal: there's only one run, so yours -- not the parent's21:18
SamYapleweekly (monthly?) ask for bindep patch review https://review.openstack.org/#/c/517105/21:18
jlvillaljeblair: Oh. Okay. Not what I was expecting. Thanks21:18
SamYaplealso, im happy to be a maintainer of the bindep project, though i think it falls under infra group scope right now21:18
*** caphrim007_ has joined #openstack-infra21:20
*** bobh has quit IRC21:20
*** salv-orlando has quit IRC21:20
*** lukebrowning has joined #openstack-infra21:22
*** slaweq_ has quit IRC21:23
*** catinthe_ has joined #openstack-infra21:23
*** caphrim007 has quit IRC21:23
*** catintheroof has quit IRC21:25
openstackgerritMerged openstack-infra/elastic-recheck master: Add query for volume backup bug 1739482  https://review.openstack.org/52941121:25
openstackbug 1739482 in Cinder "test_snapshot_backup fails to build backup due to MessagingTimeout" [Undecided,New] https://launchpad.net/bugs/173948221:25
openstackgerritMerged openstack-infra/elastic-recheck master: Add query for VolumeTypes.project lazy-load bug 1739484  https://review.openstack.org/52941321:25
openstackbug 1739484 in Cinder "DetachedInstanceError trying to lazy-load VolumeTypes.projects" [Undecided,Confirmed] https://launchpad.net/bugs/173948421:25
*** lukebrowning has quit IRC21:27
*** lukebrowning has joined #openstack-infra21:29
*** lukebrowning has quit IRC21:33
*** bobh has joined #openstack-infra21:34
*** catinthe_ has quit IRC21:35
*** lukebrowning has joined #openstack-infra21:35
*** catintheroof has joined #openstack-infra21:35
*** bnemec has quit IRC21:36
*** threestrands_ has joined #openstack-infra21:36
openstackgerritAndrea Frittoli proposed openstack-infra/project-config master: Use the integration-gate job-template  https://review.openstack.org/52941721:37
andreafclarkb, mordred, AJaeger, jeblair: this is the second part needed to start using the tempest-full job in the integrated gate ^^^21:37
*** rmcall has quit IRC21:38
*** bobh has quit IRC21:38
*** catintheroof has quit IRC21:40
*** lukebrowning has quit IRC21:40
clarkbandreaf: that removes comments from mordred about restoring nova net testing?21:40
clarkbandreaf: shoudl we keep that TODO in place?21:40
clarkbor does this also fix nova net testing?21:40
*** lukebrowning has joined #openstack-infra21:41
andreafclarkb: well as per chat earlier that template is not going to be used - most repos do not have a newton branch anymore and nova is only a matter of one last backport before it's closed21:41
*** bobh has joined #openstack-infra21:42
andreafclarkb: so I though I'd just leave there the correct integration gates...21:42
andreafclarkb: but having it in the same change is a bit of a stretch, I can split that into a different one if you prefer so21:42
clarkbandreaf: gotcha, thanks for the clarification21:42
clarkbmeh21:43
*** kgiusti has left #openstack-infra21:44
*** bobh has quit IRC21:47
clarkbpabelanger: I'm here now if you are ready to talk gerrit21:48
*** ijw has quit IRC21:50
*** trown|ruck is now known as trown|outtypewww21:50
*** priteau has joined #openstack-infra21:51
*** lukebrowning has quit IRC21:51
*** lukebrowning has joined #openstack-infra21:52
*** bobh has joined #openstack-infra21:53
jlvillaljeblair: I don't think that worked. Seems like devstack-gate checks out master21:53
jlvillaljeblair: http://logs.openstack.org/86/529386/5/check/ironic-dsvm-standalone-pike/ef9a0f2/logs/devstack-gate-setup-workspace-new.txt.gz21:53
jlvillaljeblair: I can see it checking out stable/pike at the start of the job: http://logs.openstack.org/86/529386/5/check/ironic-dsvm-standalone-pike/ef9a0f2/job-output.txt.gz21:54
jlvillalBut then I think it gets over-written later21:54
*** priteau has quit IRC21:56
*** lukebrowning has quit IRC21:58
*** bobh has quit IRC21:58
*** ralonsoh has quit IRC21:58
*** lukebrowning has joined #openstack-infra21:59
jeblairjlvillal: hrm, i guess i was wrong about that.  you'll probably need to do something like your earlier patch then, if you want to stick with the legacy devstack stuff.  or you could convert to new v3 native.21:59
jeblairjlvillal: sorry about that21:59
jlvillaljeblair: Thanks for the help. Yeah we need to switch over at some point.21:59
*** ramishra has quit IRC21:59
*** Apoorva_ has joined #openstack-infra22:01
pabelangerclarkb: any objection if we push it to tomorrow morning?22:02
*** jascott1 has quit IRC22:03
*** jascott1 has joined #openstack-infra22:03
*** lukebrowning has quit IRC22:04
*** Apoorva has quit IRC22:04
*** kjackal has quit IRC22:05
*** lukebrowning has joined #openstack-infra22:05
*** jascott1 has quit IRC22:05
clarkbpabelanger: nope22:07
*** Goneri has quit IRC22:07
clarkbpabelanger: jeblair may logging chagne to nodepool is installed now. I'm going to restart the launchers again22:09
clarkbstarting with 01 now22:09
*** lukebrowning has quit IRC22:10
pabelangerkk22:10
*** apetrich has quit IRC22:11
*** apetrich has joined #openstack-infra22:11
*** lukebrowning has joined #openstack-infra22:11
*** jascott1 has joined #openstack-infra22:15
*** lukebrowning has quit IRC22:16
clarkbpabelanger: jeblair tobiash http://paste.openstack.org/show/629505/ seems like the nodes tripping over that are in a deleting state22:16
*** fultonj has quit IRC22:16
clarkbso I think its probably semi safe to ignore them for the sake of quota calculations.22:16
*** salv-orlando has joined #openstack-infra22:17
clarkbI've not seen any issues on 01 going to restart 02 now22:17
fungidiablo_rojo noticed that storyboard-dev is giving a 500 (isr) on logging in. going to see if i can figure out what that's all about22:17
*** lukebrowning has joined #openstack-infra22:18
fungioof, uptime 582 days!22:18
fungithat's probably a record in rax22:18
*** jascott1 has quit IRC22:19
pabelangerclarkb: we do have some stuck nodes in infracloud in delete22:19
pabelangerwonder if citycloud once were deleted too22:20
*** felipemonteiro__ has joined #openstack-infra22:20
clarkbpabelanger: those are both clouds on the nl02 side but will be interesting if they show up in similar log messages22:20
*** jascott1 has joined #openstack-infra22:21
*** salv-orlando has quit IRC22:22
*** lukebrowning has quit IRC22:23
*** e0ne has quit IRC22:23
*** Goneri has joined #openstack-infra22:23
*** felipemonteiro_ has quit IRC22:24
*** bobh has joined #openstack-infra22:24
*** david-lyle has quit IRC22:24
*** lukebrowning has joined #openstack-infra22:24
*** jascott1 has quit IRC22:26
*** e0ne has joined #openstack-infra22:26
*** e0ne has quit IRC22:26
*** e0ne has joined #openstack-infra22:27
*** jascott1 has joined #openstack-infra22:27
*** e0ne has quit IRC22:28
*** e0ne has joined #openstack-infra22:28
*** bobh has quit IRC22:28
*** e0ne has quit IRC22:28
*** lukebrowning has quit IRC22:29
*** jascott1 has quit IRC22:29
openstackgerritMerged openstack-infra/elastic-recheck master: SearchEngine search only supports ints for days  https://review.openstack.org/32098822:29
openstackgerritMerged openstack-infra/elastic-recheck master: Add query for Ansible privilege escalation timeout  https://review.openstack.org/50523322:29
*** jascott1 has joined #openstack-infra22:30
*** lukebrowning has joined #openstack-infra22:30
*** rcernin has joined #openstack-infra22:32
*** jascott1 has quit IRC22:33
*** jascott1 has joined #openstack-infra22:34
*** jappleii__ has joined #openstack-infra22:35
*** lukebrowning has quit IRC22:35
dmsimardI don't know what byobu is22:36
SpamapSnor do you want to ;)22:36
dmsimardbut it's snuck its way in lol22:36
*** threestrands_ has quit IRC22:36
*** lukebrowning has joined #openstack-infra22:36
dmsimardhttp://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/manifests/server.pp#n22822:37
dmsimardhttp://paste.openstack.org/raw/629509/22:37
*** jascott1 has quit IRC22:37
dmsimardit was once 98 but NOT ANYMORE22:37
dmsimard"Byobu is a GPLv3 open source text-based window manager and terminal multiplexer" so.. like screen and tmux ?22:38
fungidid we recently merge a change which would have caused "no check-requirements job specified for openstack/django_openstack_auth" failures to start up for the requirements-tox-validate-projects job on all openstack/requirements changes?22:38
*** jascott1 has joined #openstack-infra22:40
*** ijw has joined #openstack-infra22:40
SpamapSdmsimard: it's just a config for screen or tmux22:40
fungiaha... https://review.openstack.org/529080 Drop django_openstack_auth master branch jobs22:40
fungiwe should have also removed it from projects.txt in the reqs repo i guess22:41
*** mixos has joined #openstack-infra22:41
*** lukebrowning has quit IRC22:42
dmsimardSpamapS: wow, Ubuntu doesn't like if you want to remove that package... http://paste.openstack.org/raw/629510/22:42
dmsimardyou want to remove that? Let me remove... ubuntu-server22:42
*** lukebrowning has joined #openstack-infra22:43
dmsimardfungi: does it need to be removed elsewhere ?22:43
dmsimardit sounds like we're missing a few steps if the intent is to retire that repo22:43
*** jascott1 has quit IRC22:44
fungidmsimard: smcginnis says there's a change doing that... looking for it now22:44
*** aviau has quit IRC22:44
smcginnishttps://review.openstack.org/#/c/529067/22:45
dmsimardfungi: that's projects.txt -- but I expected other patches like the gerrit ACL and so on.22:47
fungidmsimard: it's not a complete retirement because the lib is still used by stable branches of some projects22:47
dmsimardoh, makes sense22:47
*** jascott1 has joined #openstack-infra22:48
fungiwe realized that our repo retirement process has a big hole regarding retiring projects still used by stable branches of other active projects22:48
*** lukebrowning has quit IRC22:48
fungiat least where it's being used from source and not just released packages22:48
fungi#status log enqueued 529067,1 into the gate pipeline and promoted in order to unblock requirements and release changes22:49
openstackstatusfungi: finished logging22:49
*** lukebrowning has joined #openstack-infra22:49
*** caphrim007_ has quit IRC22:50
*** caphrim007 has joined #openstack-infra22:50
*** jascott1 has quit IRC22:52
*** ijw has quit IRC22:53
*** lukebrowning has quit IRC22:54
*** dsariel has quit IRC22:54
*** bobh has joined #openstack-infra22:54
*** lukebrowning has joined #openstack-infra22:55
*** mixos has quit IRC22:56
*** david-lyle has joined #openstack-infra22:56
*** bobh has quit IRC22:59
*** lukebrowning has quit IRC23:00
*** lukebrowning has joined #openstack-infra23:02
SpamapSugh.. executor server changed so much git is too confused to rebase my mem governor23:02
* SpamapS has to hand merge23:02
fungiyears ago i used to hand-merge all manner of patchfiles with no revision control. i've almost forgotten what that was like23:03
SpamapSvimdiff and a stiff drink23:03
fungishoot-me-now.c.rej23:04
clarkbfungi: not even rcs?23:04
fungisadly, no23:04
fungido versioned tarballs count as revision control? ;)23:05
fungii never really used rcs, didn't learn about revision control until cvs days23:05
clarkball this bind zone file stuff has remidned me we managed all public DNS for a particular corporation using zone files in rcs23:05
clarkbthen ran a script to sync the updates to all the servers and reload the service23:05
fungiclarkb: ooh, for that i had written a fancy visudo sort of wrapper which used nslint and rejected edits and kicked you back into the temporary editing session. those were the days23:06
*** jascott1 has joined #openstack-infra23:06
fungialso had a bunch of custom linting checks23:07
*** lukebrowning has quit IRC23:07
fungiautoincremented the zone serial23:07
fungikicked ndc on commit, et cetera23:07
clarkbnice23:07
clarkbone neat thing about this was there was like 20 yaers or more of rcs history you could dig through23:07
fungihad a mode to insert/remove/alter zone entries from the bind config and restart bind too23:08
clarkbyou can kinda see a marketing history of a company through that23:08
*** lukebrowning has joined #openstack-infra23:08
fungiindeed23:08
fungibut we kept hiring greener and greener people for hosting ops, until i was eventually pressured to put in a webui for dns management23:09
clarkbwe used something off the shelf for ipam and dns for internal dns23:09
fungitext editors were hard for that generation of sysadmins to figure out, apparently23:09
*** rmcall has joined #openstack-infra23:09
clarkbit aws terrible because you couldn't see the history of edits as easily (and I'm comparing to rcs in 2012)23:10
clarkbbut you could clikc a button and search without grep23:10
*** rmcall has quit IRC23:10
*** dprince has quit IRC23:10
*** rmcall has joined #openstack-infra23:10
fungijust remembered another detail. my script would pause at the end and collect bind logs, then filter for errors and display them to you. that was also super handy23:11
fungiso you got to have the "oh nuts!" moment more or less immediately, instead of waiting for your pager to go off and tell you you'd made a serious typo23:12
fungior, better still, the help desk calling to ask you if they could bridge you into an incident call with some angry customer and a few division heads23:12
*** lukebrowning has quit IRC23:13
fungithose were by far my favorite23:13
*** bobh has joined #openstack-infra23:14
*** lukebrowning has joined #openstack-infra23:14
fungi#status log updated storyboard-dev openids from login.launchpad.net to login.ubuntu.com to solve DBDuplicateEntry exceptions on login23:15
openstackstatusfungi: finished logging23:15
fungimordred: ^ maybe that's what you were seeing a while back?23:15
*** dave-mccowan has quit IRC23:17
*** salv-orlando has joined #openstack-infra23:17
*** bobh has quit IRC23:18
*** lukebrowning has quit IRC23:19
*** salv-orlando has quit IRC23:21
*** lukebrowning has joined #openstack-infra23:24
*** Apoorva_ has quit IRC23:24
*** felipemonteiro__ has quit IRC23:24
*** felipemonteiro__ has joined #openstack-infra23:24
*** bobh has joined #openstack-infra23:25
*** Apoorva has joined #openstack-infra23:25
*** xarses_ has quit IRC23:29
*** lukebrowning has quit IRC23:29
*** bobh has quit IRC23:29
*** dhill_ has joined #openstack-infra23:30
*** lukebrowning has joined #openstack-infra23:30
*** lukebrowning_ has joined #openstack-infra23:32
*** hashar has quit IRC23:33
*** bobh has joined #openstack-infra23:33
*** edmondsw has joined #openstack-infra23:33
*** lukebrowning has quit IRC23:34
openstackgerritDavid Moreau Simard proposed openstack-infra/system-config master: Remove byobu profile configuration  https://review.openstack.org/52942623:36
*** bobh has quit IRC23:37
*** lukebrowning_ has quit IRC23:37
*** edmondsw has quit IRC23:38
*** lukebrowning has joined #openstack-infra23:38
*** felipemonteiro__ has quit IRC23:41
*** rbrndt has quit IRC23:43
*** rbrndt has joined #openstack-infra23:43
*** rbrndt has quit IRC23:43
*** bmjen has joined #openstack-infra23:43
*** Hunner_ has joined #openstack-infra23:44
*** Hunner_ has quit IRC23:45
*** Hunner_ has joined #openstack-infra23:45
*** jamesmcarthur has quit IRC23:46
*** felipemonteiro__ has joined #openstack-infra23:46
*** rlandy|rover is now known as rlandy|bbl23:47
*** lukebrowning has quit IRC23:48
openstackgerritDavid Moreau Simard proposed openstack-infra/system-config master: Setup bash history to contain standard timestamps (RFC3339)  https://review.openstack.org/52942823:52
dmsimardinfra-root: some of you might find this last patch useful ^23:52
*** ijw has joined #openstack-infra23:53
jeblairi'm going to poke at the zuul apache rewrite rules for a bit; hopefully i won't break them, but i might23:56
johnsom404: Not Found on the zuulv3 dashboard23:58
*** ijw has quit IRC23:59
jeblairor, rather, i might break them briefly :)23:59
johnsomlol23:59
johnsomHonest, I went to the dashboard link first....23:59

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