Friday, 2019-01-04

*** jamesmcarthur has quit IRC00:02
corvusre-enqueue is done00:04
corvusit no longer appears that we are reconfiguring everytime someone does something to ansible :)00:10
*** sshnaidm_ has joined #openstack-infra00:12
*** sshnaidm__ has quit IRC00:15
*** rascasoft has joined #openstack-infra00:18
clarkbyay00:19
*** rascasoft has quit IRC00:23
openstackgerritMerged openstack-infra/system-config master: Turn on the future parser for all mirrors  https://review.openstack.org/61599300:28
openstackgerritMerged openstack-infra/system-config master: Turn on the future parser for files.openstack.org  https://review.openstack.org/61599400:28
*** slaweq has joined #openstack-infra00:39
*** yamamoto has quit IRC00:42
*** slaweq has quit IRC00:43
*** gyee has quit IRC00:45
*** rascasoft has joined #openstack-infra00:52
*** markvoelker has quit IRC00:54
*** markvoelker has joined #openstack-infra00:54
*** rascasoft has quit IRC00:58
*** lbragstad has quit IRC00:59
*** markvoelker has quit IRC00:59
*** yamamoto has joined #openstack-infra01:01
openstackgerritMerged openstack-infra/system-config master: Turn on future parser for one zookeeper instance  https://review.openstack.org/61599501:04
*** bobh_ has joined #openstack-infra01:05
*** rlandy|rover has quit IRC01:06
clarkbfor the prioritization setup that cinder et al are using couldn't we set a range of -1 to +1 then set noop function and copyminscore and copymaxscore01:08
*** bobh has quit IRC01:08
clarkbThen you end up with -1 "this shouldn't be reviewed", 0 "default priority", 1 a priority. Values copy as patchsets are pushed but don't affect your ability to merge or not01:09
*** lbragstad has joined #openstack-infra01:09
clarkbsmcginnis: jungleboyj: fungi ^01:09
clarkbit doesn't give you fine grained control over levels of priority but I think it gets everything else?01:09
*** lbragstad has quit IRC01:09
*** bobh has joined #openstack-infra01:15
*** bobh_ has quit IRC01:16
*** tosky has quit IRC01:17
*** bobh has quit IRC01:20
*** AJaeger has quit IRC01:23
*** AJaeger has joined #openstack-infra01:29
*** jamesmcarthur has joined #openstack-infra01:32
fungithat's along the lines of what i suggested in #swift a little while ago01:38
clarkbI responded ont he mailing list with that01:39
clarkbI think it gets 99% of the way there, you just don't get multiple levels of positive priority01:39
clarkbwhich if you have a real emergency of "please only review this one change" you can remove existing priority from changes to unprioritize them01:40
*** bobh has joined #openstack-infra01:41
*** dave-mccowan has quit IRC01:41
clarkbthough maybe is:starred works well enough for that without changing labels01:42
clarkb-1 Workflow to deprioritize and apply star to prioritize01:43
timburkei wonder if there'd be a relatively easy way to search for "starred by someone who can merge code in this repo"...01:46
timburkeshort of a giant "(starredby:... OR starred by:... OR ...)"01:47
*** yamamoto has quit IRC01:47
clarkbtimburke: I don't think gerrit exposes that aggregation in a simpler manner01:48
clarkbso maybe the label with +1 that can only be applied by $group does have an advantage01:48
clarkbit simplifies knowing what to look for01:48
*** yamamoto has joined #openstack-infra01:48
*** dave-mccowan has joined #openstack-infra01:49
clarkb(as I emntioned in the ml thread, this is potentially something we could chagne upstream then backport though)01:49
clarkb(we don't want to carry non upstreamable patches locally anymore though)01:49
*** markmcd has quit IRC01:51
*** yamamoto has quit IRC01:53
*** markvoelker has joined #openstack-infra01:55
*** wolverineav has quit IRC01:58
*** wolverineav has joined #openstack-infra01:59
*** sshnaidm_ has quit IRC02:02
clarkbcmurphy: ok I've just approved the last change before the nameservers in the stack. I'm off for the night (zk01 was fine so zk* should be too) and will look for nameserver updates in the morning andp ush them myself if you haven't been able to02:02
*** wolverineav has quit IRC02:03
*** yamamoto has joined #openstack-infra02:10
*** harlowja has quit IRC02:12
*** hongbin has joined #openstack-infra02:15
*** sshnaidm_ has joined #openstack-infra02:19
openstackgerritwanghao proposed openstack-infra/project-config master: Add 'Review-Priority' for Zaqar repos  https://review.openstack.org/62832302:19
openstackgerritMerged openstack-infra/system-config master: Turn on future parser for all zookeeper instances  https://review.openstack.org/61599602:22
*** mandre has quit IRC02:24
*** mandre has joined #openstack-infra02:29
*** bobh_ has joined #openstack-infra02:32
*** bobh has quit IRC02:34
*** markvoelker has quit IRC02:46
*** yamamoto has quit IRC02:51
*** yamamoto has joined #openstack-infra02:52
*** anteaya has quit IRC03:03
*** psachin has joined #openstack-infra03:04
*** bobh has joined #openstack-infra03:05
*** psachin has quit IRC03:07
*** psachin has joined #openstack-infra03:08
*** bobh_ has quit IRC03:08
*** hwoarang has quit IRC03:09
*** lifeless has joined #openstack-infra03:10
*** hwoarang has joined #openstack-infra03:10
*** apetrich has quit IRC03:15
*** hwoarang has quit IRC03:18
*** hwoarang_ has joined #openstack-infra03:18
*** jamesmcarthur has quit IRC03:21
*** jamesmcarthur has joined #openstack-infra03:24
*** yamamoto has quit IRC03:24
*** hongbin has quit IRC03:26
*** bobh_ has joined #openstack-infra03:26
*** bhavikdbavishi has joined #openstack-infra03:29
*** bobh has quit IRC03:29
*** bhavikdbavishi1 has joined #openstack-infra03:29
*** jamesmcarthur has quit IRC03:31
*** bhavikdbavishi has quit IRC03:33
*** bhavikdbavishi1 is now known as bhavikdbavishi03:33
*** ykarel|away has joined #openstack-infra03:37
*** bobh_ has quit IRC03:50
*** jamesmcarthur has joined #openstack-infra04:00
*** hongbin has joined #openstack-infra04:01
*** ramishra has joined #openstack-infra04:18
*** dave-mccowan has quit IRC04:25
*** shardy has quit IRC04:33
*** shardy has joined #openstack-infra04:34
*** wolverineav has joined #openstack-infra04:36
*** jamesmcarthur has quit IRC04:47
*** jamesmcarthur has joined #openstack-infra04:47
*** udesale has joined #openstack-infra04:52
*** hongbin has quit IRC04:52
*** bobh has joined #openstack-infra04:55
*** diablo_rojo has quit IRC05:03
*** wolverineav has quit IRC05:10
*** wolverineav has joined #openstack-infra05:11
*** wolverineav has quit IRC05:15
*** bobh has quit IRC05:22
*** yamamoto has joined #openstack-infra05:22
*** whoami-rajat has joined #openstack-infra05:22
*** ykarel|away has quit IRC05:26
*** jamesmcarthur has quit IRC05:27
*** ykarel|away has joined #openstack-infra05:43
*** ykarel|away is now known as ykarel05:44
*** yamamoto has quit IRC06:04
*** bhavikdbavishi has quit IRC06:13
*** hwoarang_ has quit IRC06:25
*** armax has quit IRC06:26
*** hwoarang has joined #openstack-infra06:29
*** auristor has quit IRC06:32
*** bhavikdbavishi has joined #openstack-infra06:34
*** agopi has quit IRC06:34
*** wolverineav has joined #openstack-infra06:34
*** auristor has joined #openstack-infra06:34
*** bhavikdbavishi1 has joined #openstack-infra06:37
*** bhavikdbavishi has quit IRC06:38
*** bhavikdbavishi1 is now known as bhavikdbavishi06:38
*** yamamoto has joined #openstack-infra06:42
*** rcernin has quit IRC06:47
*** janki has joined #openstack-infra06:48
*** janki has quit IRC06:49
*** janki has joined #openstack-infra06:49
*** jamesmcarthur has joined #openstack-infra06:50
*** jamesmcarthur has quit IRC06:55
*** quiquell|off has quit IRC07:13
*** quiquell has joined #openstack-infra07:14
*** dpawlik has joined #openstack-infra07:24
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for one nameserver  https://review.openstack.org/61599807:28
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for all nameservers  https://review.openstack.org/61599907:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for master nameserver  https://review.openstack.org/61600007:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on future parser for one nodepool launcher  https://review.openstack.org/61628807:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on future parser for all nodepool launchers  https://review.openstack.org/61628907:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on future parser for one nodepool builder  https://review.openstack.org/61629007:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on future parser for all nodepool builders  https://review.openstack.org/61629107:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on future parser for one zuul executor  https://review.openstack.org/61629207:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on future parser for all zuul executors  https://review.openstack.org/61629307:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for one zuul merger  https://review.openstack.org/61629407:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for all zuul mergers  https://review.openstack.org/61629507:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for zuul.openstack.org  https://review.openstack.org/61629607:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Fix glob for lists.katacontainers.io  https://review.openstack.org/62821607:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Fix glob for lists.o.o  https://review.openstack.org/62821707:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for openstackid-dev  https://review.openstack.org/61600107:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for openstackid  https://review.openstack.org/61600207:29
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Turn on the future parser for refstack  https://review.openstack.org/62815307:29
*** pgaxatte has joined #openstack-infra07:30
openstackgerritwanghao proposed openstack-infra/project-config master: Add 'Review-Priority' for Zaqar repos  https://review.openstack.org/62832307:31
*** apetrich has joined #openstack-infra07:32
*** dpawlik has quit IRC07:34
*** dpawlik has joined #openstack-infra07:38
*** wolverineav has quit IRC07:40
*** wolverineav has joined #openstack-infra07:43
*** slaweq has joined #openstack-infra07:45
*** rpittau has joined #openstack-infra07:53
*** yamamoto has quit IRC07:58
*** ccamacho has joined #openstack-infra08:01
*** aojea has joined #openstack-infra08:01
*** ykarel is now known as ykarel|lunch08:02
*** shardy has quit IRC08:05
*** shardy has joined #openstack-infra08:05
*** rascasoft has joined #openstack-infra08:07
*** markvoelker has joined #openstack-infra08:08
*** markvoelker has quit IRC08:13
*** lpetrut has joined #openstack-infra08:15
*** armax has joined #openstack-infra08:27
*** armax has quit IRC08:32
*** wolverineav has quit IRC08:34
*** jaosorior has joined #openstack-infra08:39
*** jpich has joined #openstack-infra08:44
*** markmcd has joined #openstack-infra08:45
*** agopi has joined #openstack-infra08:47
*** tosky has joined #openstack-infra08:47
*** udesale has quit IRC08:52
*** bhavikdbavishi has quit IRC08:53
*** bhavikdbavishi1 has joined #openstack-infra08:53
*** udesale has joined #openstack-infra08:54
*** bhavikdbavishi1 is now known as bhavikdbavishi08:55
*** lewo has joined #openstack-infra09:09
*** xek_ has joined #openstack-infra09:10
*** ykarel|lunch is now known as ykarel09:12
*** gfidente has joined #openstack-infra09:13
*** zaneb has quit IRC09:17
*** agopi has quit IRC09:17
*** lpetrut has quit IRC09:18
*** agopi has joined #openstack-infra09:23
*** shardy has quit IRC09:27
*** shardy has joined #openstack-infra09:27
ssbarneai see lots of error that indicate a broken ruby bundler on build nodes: see http://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22bundled_gems%2Fbin%2Fbundle:%20No%20such%20file%20or%20directory%5C%2209:30
ykarelmmm bundler 2.0.1 also released09:33
ykarelhmm released today09:33
ykarelwe are pinning to 1.17.3 in https://review.openstack.org/#/c/628155/09:34
ykarelssbarnea, ^^09:34
ykarelu can check if that's take care of ur issue09:34
*** derekh has joined #openstack-infra09:35
ssbarneai am working on it...09:36
ssbarneaseems to be related to code in puppet-openstack-integration09:37
ykarelyes most of jobs are configured there only09:37
ssbarneaperfect, no need to fix it :D09:38
ssbarneaykarel: i personally do not like the pinning approach, it should be smarter: install a version supported by current ruby09:39
ssbarneabut is clearly better than now09:39
*** udesale has quit IRC09:39
ykarelssbarnea, so new ruby is required if we don't go with pinning09:39
ykarelthere may be some constraints to bump ruby . /me not aware about that09:40
ssbarneais the gem installer so stupid? pip at least will not install packages not supporting current interpreter.09:40
*** udesale has joined #openstack-infra09:40
*** yamamoto has joined #openstack-infra09:40
ykarelokk ruby coming from centos base and we don't controll that09:41
ykareldon't know about that gem installer, so can't say09:41
ssbarneai am having a look now, better to know for the future09:42
*** yamamoto has quit IRC09:43
ykarelack09:44
*** markmcd has left #openstack-infra09:47
*** agopi has quit IRC09:47
*** jpich has quit IRC09:48
*** jpich has joined #openstack-infra09:48
ssbarneai was able to replicate the issue, clearly new bundler broke centos-7, but fedora-28 works.09:50
*** agopi has joined #openstack-infra09:50
*** e0ne has joined #openstack-infra09:52
ssbarneaykarel: i ended up creating https://github.com/bundler/bundler/issues/687009:56
ssbarneais better to assure that we raise issues upstream before adding workarounds09:57
ykarelssbarnea, ack09:57
ykareli think EmilienM mwhahaha would have idea around it09:58
EmilienMhello09:58
ssbarneaykarel: imagine what would happen if pip will do that09:58
ykarelssbarnea, in pip also i think we need to set what we want with py2/py3, no?10:00
*** wolverineav has joined #openstack-infra10:04
*** wolverineav has quit IRC10:08
*** markmcd has joined #openstack-infra10:09
*** markmcd has left #openstack-infra10:09
*** markvoelker has joined #openstack-infra10:09
*** bhavikdbavishi has quit IRC10:10
*** armax has joined #openstack-infra10:29
openstackgerritMerged openstack-infra/system-config master: Update Zuul service documentation  https://review.openstack.org/60555610:29
*** armax has quit IRC10:33
openstackgerritMerged openstack-infra/puppet-infra-cookiecutter master: Dissuade .gitignore references to personal tools  https://review.openstack.org/60833410:34
*** jpich has quit IRC10:43
*** jpich has joined #openstack-infra10:49
*** sshnaidm__ has joined #openstack-infra11:01
*** pbourke has quit IRC11:02
*** pbourke has joined #openstack-infra11:04
*** sshnaidm_ has quit IRC11:05
*** sshnaidm__ has quit IRC11:06
*** sshnaidm has joined #openstack-infra11:08
*** sshnaidm is now known as sshnaidm|off11:08
openstackgerritQuique Llorente proposed openstack-infra/zuul-jobs master: WIP: Default private_ipv4 to use public_ipv4 address when null  https://review.openstack.org/62329411:13
*** panda is now known as panda|reloc11:16
*** rpittau is now known as rpittau|lunch11:28
*** janki has quit IRC11:30
*** dtantsur|afk is now known as dtantsur11:32
openstackgerritSean McGinnis proposed openstack-infra/project-config master: Consolidate Cinder repo ACL settings  https://review.openstack.org/62840711:37
openstackgerritSean McGinnis proposed openstack-infra/project-config master: Remove Cinder CI Verified label  https://review.openstack.org/62840811:37
*** rpittau|lunch has quit IRC11:43
*** rpittau|lunch has joined #openstack-infra11:44
*** dkehn has quit IRC11:57
*** rpittau|lunch has quit IRC11:58
*** rpittau|lunch has joined #openstack-infra12:01
*** dkehn has joined #openstack-infra12:02
*** bhavikdbavishi has joined #openstack-infra12:10
*** rpittau|lunch has quit IRC12:13
*** rpittau|lunch has joined #openstack-infra12:13
*** rpittau|lunch has quit IRC12:28
*** armax has joined #openstack-infra12:29
openstackgerritlijunjie proposed openstack-infra/zuul master: Fix the misspelling of "connection"  https://review.openstack.org/62842212:32
*** rpittau has joined #openstack-infra12:33
openstackgerritlijunjie proposed openstack-infra/puppet-translation_checksite master: Fix the misspelling of "openstack"  https://review.openstack.org/62842512:33
*** armax has quit IRC12:34
openstackgerritlijunjie proposed openstack-infra/openstackweb master: Fix the misspelling of "openstack"  https://review.openstack.org/62842612:34
*** lifeless has quit IRC12:42
fungiykarel: ssbarnea: cmurphy pinned to bundler<2 yesterday in our infra puppet integration tests and that seems to have worked for us anyway12:48
ssbarneafungi: i got a big more flexible approach: https://review.openstack.org/#/c/628395/3/playbooks/prepare-node-integration.yaml12:49
ssbarneait should be better, sooner or later we will hit something that needs newer bundler12:50
*** ssbarnea has quit IRC12:51
ykarelfungi, ack12:55
*** panda|reloc is now known as panda13:00
AJaegerfungi: https://review.openstack.org/628283 is also needed for pinning AFAIU13:02
*** jcoufal has joined #openstack-infra13:02
cmurphyyes that will fix https://review.openstack.org/60602113:03
*** rlandy has joined #openstack-infra13:06
*** rlandy is now known as rlandy|rover13:14
*** boden has joined #openstack-infra13:16
*** slaweq has quit IRC13:23
*** slaweq has joined #openstack-infra13:24
*** trown|outtypewww is now known as trown13:26
*** bhavikdbavishi has quit IRC13:28
*** markvoelker has quit IRC13:33
*** wolverineav has joined #openstack-infra13:40
*** wolverineav has quit IRC13:44
*** agopi has quit IRC13:49
*** agopi has joined #openstack-infra13:51
*** ykarel is now known as ykarel|away13:59
*** dave-mccowan has joined #openstack-infra14:01
openstackgerritMerged openstack-infra/irc-meetings master: Updated meeting time for D&I WG  https://review.openstack.org/62821414:03
*** lbragstad has joined #openstack-infra14:08
*** iurygregory has quit IRC14:15
*** iurygregory has joined #openstack-infra14:15
*** ssbarnea has joined #openstack-infra14:20
*** shardy has quit IRC14:23
*** ykarel|away has quit IRC14:24
*** mriedem has joined #openstack-infra14:30
*** armax has joined #openstack-infra14:30
*** EmilienM is now known as EvilienM14:34
*** armax has quit IRC14:35
*** quiquell is now known as quiquell|off14:45
*** sshnaidm_ has joined #openstack-infra14:46
*** sshnaidm|off has quit IRC14:47
*** sshnaidm_ is now known as sshnaidm|off14:50
*** kgiusti has joined #openstack-infra14:56
*** kgiusti has left #openstack-infra14:56
ssbarneafungi: clarkb: a very old one https://review.openstack.org/#/c/570546/ - so old that I do not remember exactly where I encountered the bug, but should be no brainer to review it.14:57
*** udesale has quit IRC14:59
*** Goneri has quit IRC15:05
*** rlandy|rover is now known as rlandy|rover|brb15:10
*** rlandy|rover|brb is now known as rlandy|rover15:22
*** ykarel|away has joined #openstack-infra15:24
*** dpawlik has quit IRC15:25
corvusmordred, fungi: it looks like the issue with gitea replication is that the ssh connection on gerrit's side is in close_wait:15:25
corvusjava       12318     gerrit2  308u  IPv6 1607582864      0t0  TCP 104.130.246.32:52934->38.108.68.66:ssh (CLOSE_WAIT)15:26
openstackgerritjacky06 proposed openstack/diskimage-builder master: Update hacking version to latest  https://review.openstack.org/62846615:26
corvusmordred, fungi: i don't see a running sshd subprocess on gitea's side to correspond with that15:27
corvusthat public ip terminates at an octavia lb -- can we log into that and look at netstat there?15:28
mordredcorvus: I don't know if we can or not ... mnaser ?15:29
mordredcorvus: in other news, the ceph cluster seems to be back to being happy15:29
corvusmordred: neat, any idea what happened?15:30
mordredcorvus: no clue at all15:30
mordredcorvus: I also don't know what made it unhappy in the first place15:30
corvusok... i guess "wait and see if it fixes itself" is a viable option to consider in response to "osds are unbalanced"15:31
mordredyeah. maybe osds just get unbalanced from time to time?15:31
corvusit looks like osd1 is still a little more used than the others15:31
johnsomcorvus: You can log into the amp via ssh if octavia is configured with an ssh keypair.  Are you trying to do an ssh through the lb vip?15:32
mordredjohnsom: yah - it's lb'ing ssh15:33
johnsomOk, did you crank up the listener timeouts?15:33
mordredwe did not - we didn't actually create the lb ourselves, k8s created it for us - is that a thing we can alter in a running lb?15:34
johnsomThe defaults are good for HTTP, but ssh will idle and run longer.15:34
johnsomYes, they can be updated after the fact15:34
* corvus logs into horizon15:34
mordredcool15:35
fungi(unless we set ssh keepalives on both the client and server side, which is probably a pain to do for whatever gerrit's using)15:35
corvusjohnsom: when you say 'octavia configured with an ssh key' is that an operator or end-user feature?15:35
*** lbragstad has quit IRC15:36
johnsomOperator only15:36
corvuswe're not the operator here, so i guess we can't log into it then15:36
corvusjohnsom: for the timeout, i assume that would be on the listener?15:36
johnsomYes15:37
*** EvilienM is now known as EmilienM15:37
corvusit doesn't look like that's an option in horizon: https://screenshots.firefox.com/kC6p0Rl9HeLs4WUP/dashboard.vexxhost.net15:38
corvusi guess i should dig out the cli :)15:38
johnsomYeah, that looks like an older dashboad15:38
mordredthat's weird- mnaser is normally running all the coolest/latest things15:39
*** lbragstad has joined #openstack-infra15:40
corvushere we are: http://paste.openstack.org/show/739934/15:41
mnaserHi15:41
mnaserYes our dashboard is queens :( things are changing soon though15:42
johnsomcorvus: those values are in ms, crank the client and server way up for your estimated ssh times.15:43
corvusso that's 5 seconds to connect, and 50 seconds without sending data?  that is probably too low for an interactive ssh session, but since this is non-interactive, i wonder if there's still another problem.15:47
clarkbI agree, if running git push over ssh that should be plenty15:48
johnsomDoesn’t hurt to bump them anyway unless you are expecting a very high number of connections coming/going.15:50
corvusyeah, i'll go ahead and do that -- it could be that the server is slow enough to trip the timeout, and that's the immediate cause of the close_wait.  perhaps increasing the timeout will improve that, but still leave us with the question of why the connection went idle.15:51
corvusloadbalancer listener set 3f56eb2d-c85d-44d9-a472-addd038fc6b2 --timeout-client-data=181000 --timeout-member-data=18100015:53
*** diablo_rojo has joined #openstack-infra16:00
*** smarcet has joined #openstack-infra16:03
clarkbis that an openstack client command?16:04
corvusya16:04
*** apetrich has quit IRC16:07
openstackgerritSorin Sbarnea proposed openstack-infra/git-review master: Switch from testrepository to stestr  https://review.openstack.org/62852516:07
*** jamesmcarthur has joined #openstack-infra16:07
*** adriancz has quit IRC16:08
*** jrist has joined #openstack-infra16:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for master nameserver  https://review.openstack.org/61600016:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on future parser for one nodepool launcher  https://review.openstack.org/61628816:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on future parser for all nodepool launchers  https://review.openstack.org/61628916:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on future parser for one nodepool builder  https://review.openstack.org/61629016:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on future parser for all nodepool builders  https://review.openstack.org/61629116:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on future parser for one zuul executor  https://review.openstack.org/61629216:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on future parser for all zuul executors  https://review.openstack.org/61629316:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for one zuul merger  https://review.openstack.org/61629416:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for all zuul mergers  https://review.openstack.org/61629516:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for zuul.openstack.org  https://review.openstack.org/61629616:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Fix glob for lists.katacontainers.io  https://review.openstack.org/62821616:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Fix glob for lists.o.o  https://review.openstack.org/62821716:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for openstackid-dev  https://review.openstack.org/61600116:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for openstackid  https://review.openstack.org/61600216:09
openstackgerritClark Boylan proposed openstack-infra/system-config master: Turn on the future parser for refstack  https://review.openstack.org/62815316:09
clarkbcmurphy: ^ adns was in same situation16:09
clarkbfungi: care to rereview changes starting at https://review.openstack.org/#/c/615998/7 ? They've changed slightly16:09
cmurphyclarkb: oops thansk16:10
*** armax has joined #openstack-infra16:10
*** dpawlik has joined #openstack-infra16:11
*** arxcruz is now known as arxcruz|brb16:11
*** dpawlik has quit IRC16:15
*** smarcet has quit IRC16:18
*** ykarel|away has quit IRC16:18
*** smarcet has joined #openstack-infra16:21
openstackgerritjacky06 proposed openstack/ansible-role-cloud-launcher master: Update hacking version to latest  https://review.openstack.org/62853716:21
*** bhavikdbavishi has joined #openstack-infra16:26
*** apetrich has joined #openstack-infra16:28
*** smarcet has quit IRC16:29
*** hamerins has joined #openstack-infra16:30
*** smarcet has joined #openstack-infra16:31
*** psachin has quit IRC16:32
*** Goneri has joined #openstack-infra16:34
*** apetrich has quit IRC16:35
*** anteaya has joined #openstack-infra16:35
*** e0ne has quit IRC16:36
*** sthussey has quit IRC16:39
*** smarcet has quit IRC16:42
openstackgerritweizj proposed openstack/os-performance-tools master: Update hacking version to latest  https://review.openstack.org/62859516:42
*** smarcet has joined #openstack-infra16:43
openstackgerritweizj proposed openstack/os-testr master: Update hacking version to latest  https://review.openstack.org/62860816:43
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Run k8s-on-openstack to manage k8s control plane  https://review.openstack.org/62696516:46
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add resources for deploying rook and xtradb to kuberenets  https://review.openstack.org/62605416:46
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Increase number of k8s nodes to 4  https://review.openstack.org/62803116:46
*** iurygregory has quit IRC16:48
*** sthussey has joined #openstack-infra16:50
zxiiroAnyone know if there's a way to list all object storage items using the openstack client? "openstack object list" seems to be about listing a specific container and not about listing all available containers.17:03
*** jbadiapa has quit IRC17:04
*** derekh has quit IRC17:04
zxiiroor do I have ot use the swift client for this?17:04
*** pgaxatte has quit IRC17:04
*** derekh has joined #openstack-infra17:04
ssbarneahttps://review.openstack.org/#/c/628525/ -- adopts use of stestr instead of testrepository for git-review17:04
clarkbzxiiro: You want to list all objects in all containers?17:05
clarkbzxiiro: my hunch is that you have to list all containers then do a foreach type loop since objects are parented to containers and list is a container activity. notmyname should know though17:06
*** ramishra has quit IRC17:06
zxiiroclarkb: just want to list containers17:07
clarkbzxiiro: `openstack container list` iirc17:07
zxiiroclarkb: AH! Just occurred to me...17:07
zxiiroduh17:08
clarkbzxiiro: in this context they are swift containers not linux containers (because swift came first in our world I don't know if it came first on wall clock)17:08
pabelangercorvus: I believe so, but cannot remember ATM. I'll be able to look this afternoon and refresh my memory17:08
zxiirocool thanks this works for me.17:09
timburkeclarkb: fwiw, there have been rumblings about wanting to rename them to be "buckets" to match S3's nomenclature, but the transition would be... painful...17:10
clarkbtimburke: fwiw when I think of the shipping container memes I think they apply to swift better than docker :)17:10
*** rpittau has quit IRC17:12
notmynameclarkb: :-)17:15
fungialso, yes, swift had "containers" before docker popularized the term to apply to system isolation17:16
fungithough maybe the term dates back to lxc, which predates swift by two years17:17
*** armax has quit IRC17:18
openstackgerritSorin Sbarnea proposed openstack-infra/git-review master: Switch from testrepository to stestr  https://review.openstack.org/62852517:19
*** armax has joined #openstack-infra17:20
clarkbfungi: https://review.openstack.org/#/c/615998/ fixes the futureparser stuff for name servers. I'll merge and monitor those again this morning17:20
fungithanks, i've been sidelined by a bunch of discussions this morning, so just getting to reviews now17:21
notmynamefungi: swift got the name "containers" from the internal project that it replaced at rackspace. IIRC that project at rackspace started in 2007. swift started in the middle of 2009 to replace that system17:22
fungioh, also, in container history, i found a reference to https://lkml.org/lkml/2006/10/20/251 introducing "generic process containers" (before they got renamed to "control groups" the next year)17:23
fungiso i guess the linux kernel has been using the term in more or less that way dating back to '0617:23
*** whoami-rajat has quit IRC17:27
fungibut yeah, given linux only briefly used the term to refer to control groups in 06, rackspace can't be blamed for not being aware of possible future confusion over use of the term when it designed an object storage system around the same time ;)17:31
openstackgerritLogan V proposed openstack-infra/project-config master: Add node host_ids to logstash fields  https://review.openstack.org/62864217:34
clarkblogan-: ^ thank you for that17:35
logan-clarkb: reading your backscroll yesterday I think ^ will help me a lot when im digging thru logstash for failures17:35
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Pin bundler in puppet-lint and -syntax jobs  https://review.openstack.org/62828317:35
clarkbyup and http://logs.openstack.org/15/625915/3/gate/tripleo-ci-centos-7-undercloud-containers/0a7a9b7/zuul-info/inventory.yaml shows it is available as we expect it to be on the zuul side17:36
clarkbpabelanger: ^ maybe you can review https://review.openstack.org/628642 too? its related to your host id work17:37
*** e0ne has joined #openstack-infra17:43
*** e0ne has quit IRC17:44
*** hamerins has quit IRC17:44
*** hamerins has joined #openstack-infra17:46
*** jpich has quit IRC17:49
*** dustinc has joined #openstack-infra17:51
openstackgerritMerged openstack-infra/system-config master: Turn on the future parser for one nameserver  https://review.openstack.org/61599817:54
*** josephrsandoval has joined #openstack-infra17:55
*** martinkennelly has joined #openstack-infra17:56
*** josephrsandoval has quit IRC17:57
dirkcould anyone help me understand where this fedora 27 image reference comes from?17:58
dirkhttp://logs.openstack.org/13/628413/1/check/heat-functional-convg-mysql-lbaasv2/3ab6aa4/logs/devstacklog.txt.gz#_2019-01-04_12_35_42_37717:58
dirkit seems running jobs against stable/pike doesn't work right now17:58
dirkit looks like the mirror only has fedora 28+17:59
clarkbdirk: we deleted fedora27 as it is eol now aiui17:59
*** hamerins has quit IRC18:00
martinkennellyHey all, I am developing a CI. I have a service account setup, but how do I get my CI posting results like this for each patch? https://imgur.com/zmVmk4L18:00
clarkbdirk: I think that url comes from the devstack config?18:00
fungidirk: clarkb: looks like that's a stable/pike job18:01
fungiso http://codesearch.openstack.org/?q=Fedora-Cloud-Base-27-1 isn't much help18:01
dirkah, it looks like we need https://review.openstack.org/#/c/622853/18:01
dirkwhich also fails18:01
*** hamerins has joined #openstack-infra18:01
dirksigh18:01
clarkbmartinkennelly: the way that table is generated is we load js into gerrit that scans the comments for comments in a specific format that it recognizes18:01
martinkennellyclarkb: Awesome thank you!!18:02
clarkbmartinkennelly: if your CI starts commenting with that format then it will get picked up by the js comment scanner18:02
clarkbmartinkennelly: you can expand the comments below on the change to see that format18:02
martinkennellyclarkb: exactly what I wished to understand :)18:02
clarkband I think we've got it written down somewhere but I'm not sure where ( maybe in our zuul config? or is that default zuul gerrit report format?)18:03
fungimartinkennelly: https://docs.openstack.org/infra/system-config/third_party.html#posting-result-to-gerrit18:03
fungithe last paragraph of that section18:03
fungi"For example, to have your test results formatted in the same manner as the upstream Jenkins results, use a template for each result matching..."18:03
*** derekh has quit IRC18:03
*** gyee has joined #openstack-infra18:04
dirkclarkb: but it looks like non-EOLed images are not working with pike18:04
martinkennellyfungi: Thank you so much. I am humbled by your help.18:04
clarkblogan-: http://logs.openstack.org/42/628642/1/check/openstack-zuul-jobs-linters/94c4e02/job-output.txt.gz#_2019-01-04_17_43_01_264400 pep8 is unhappy with that change if you want ot update it :)18:04
*** aojea has quit IRC18:04
fungimartinkennelly: i am honored to be able to help!18:04
clarkbdirk: this is why we have been wary to support old branches for long periods of time :/18:04
dirkwell18:05
clarkbdirk: eventually you end upfixing all the things if you didn't pick something with long term support to start with18:05
fungidirk: it's perfectly acceptable under the extended maintenance model to declare a branch untestable, or to declare certain jobs no longer viable on a branch and stop running them18:05
*** xek_ has quit IRC18:05
dirkthis isn't a nodepool OS18:05
dirkits a test image. so I would assume that newer images work like older images18:05
clarkbdirk: fedora 28 is very different than fedora 2718:06
dirkhmm18:06
dirkso we're sure its not 28 vs 29 but 27 vs 28?18:06
dirkthe review I linked above goes from 27 to 2918:06
clarkbdirk: i'm not sure where it will breka but in general fedora has massive changes between releases18:07
*** ccamacho has quit IRC18:07
clarkbmy suggestion would be to stop using fedora there if you want to support it over a very long period of time18:07
clarkband use something with a long support period instead18:07
*** tosky has quit IRC18:09
dirkclarkb: the fedora 29 image works with queens18:09
dirkso I guess something was fixed in queens that needs to be backported to pike now18:09
*** jamesmcarthur has quit IRC18:10
*** jamesmcarthur has joined #openstack-infra18:10
*** jamesmcarthur has quit IRC18:11
*** dtantsur is now known as dtantsur|afk18:11
*** jamesmcarthur has joined #openstack-infra18:11
fungihuh, so github is sending out uselessly vague notices to projects about a high-severity vulnerability in pyyaml: "Known high severity security vulnerability in pyyaml v< 4.2b1 ..." (3.13 is the latest non-prerelease version of pyyaml on pypi, so...)18:12
fungimy best guess is it's about https://github.com/yaml/pyyaml/issues/518:12
clarkbfungi: ya they were going to flip 4.x over to safe load by default18:12
fungiaha, breadcrumb leads to https://github.com/opsdroid/opsdroid/issues/58018:13
clarkband I think have removed at least one release that did so from pypi18:13
openstackgerritSorin Sbarnea proposed openstack-infra/git-review master: Adds JAVA_HOME to tox passenv  https://review.openstack.org/62852518:13
fungiyeah, just wondering what triggers github to send out notices recommending upgrading to a beta release of pyyaml after some 7 months18:13
clarkbfungi: probably a question for jlk18:14
openstackgerritLogan V proposed openstack-infra/project-config master: Add node host_ids to logstash fields  https://review.openstack.org/62864218:14
fungifurther tracking leads to https://github.com/yaml/pyyaml/issues/20718:14
fungito be honest i find the whole situation with https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-18342 ridiculous18:18
*** jcoufal has quit IRC18:18
clarkbfungi: considering that is a feature of yaml: yes18:19
fungii mean, it's a library for loading data, and includes support for loading all sorts of data types not limited to "safe" ones18:19
fungiand its documentation clearly states not to use load() on untrusted data18:19
fungithis is like saying applications using python are unsafe because someone might write an application which uses the pickle routines from stdlib on untrusted data18:20
fungiyeah, so?18:20
openstackgerritSorin Sbarnea proposed openstack-infra/git-review master: Switch from testrepository to stestr  https://review.openstack.org/62865518:20
*** smarcet has quit IRC18:21
fungianyway, basically every repository on github which indicates a dependency on a non-beta version of pyyaml just got spammed about this "known high-severity security vulnerability in pyyaml"18:23
clarkbwell now we know :)18:24
fungiand knowing's half teh battle, go joe18:24
*** smarcet has joined #openstack-infra18:24
* fungi disappears to get some grocery shopping out of the way, bbiab18:24
*** smarcet has quit IRC18:35
*** smarcet has joined #openstack-infra18:39
*** jamesmcarthur has quit IRC18:46
aspierssmarcet: please could you explain https://review.openstack.org/#/c/585516/3/www/templates/deprecated_badge.tmpl@6 to me?18:47
*** jamesmcarthur has joined #openstack-infra18:47
aspiersin particular "if this html its included externally, as it does on doctheme"18:47
*** smarcet has quit IRC18:50
*** jamesmcarthur has quit IRC18:51
*** smarcet has joined #openstack-infra18:52
clarkbfungi: looks like github is also complaining about django18:52
*** jamesmcarthur has joined #openstack-infra18:55
*** gfidente has quit IRC18:56
*** whoami-rajat has joined #openstack-infra18:57
openstackgerritMerged openstack-infra/system-config master: Turn on the future parser for all nameservers  https://review.openstack.org/61599918:58
anteayafungi: I agree with the mailing list kudos, for me I would substitute the 'actually' with 'consistently'19:00
*** jcoufal has joined #openstack-infra19:00
aspierssmarcet: ohhh, I guess it's referring to https://github.com/openstack/openstackdocstheme/blob/a0684ace5e49ef935bd314928dc1acde3da6e11a/openstackdocstheme/theme/openstackdocs/layout.html#L10719:00
*** bobh has joined #openstack-infra19:01
aspiersas used in places like https://docs.openstack.org/nova/queens/ ... OK, returning to #openstack-docs now :)19:03
openstackgerritMerged openstack-infra/devstack-gate master: Avoid failure to install zuul when venv exists  https://review.openstack.org/57054619:03
*** wolverineav has joined #openstack-infra19:04
*** wolverineav has quit IRC19:04
*** wolverineav has joined #openstack-infra19:04
openstackgerritMerged openstack-infra/system-config master: Turn on the future parser for master nameserver  https://review.openstack.org/61600019:11
*** jamesmcarthur has quit IRC19:11
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Add #opendev to accessbot  https://review.openstack.org/62866419:12
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add #opendev to meetbot  https://review.openstack.org/62866519:12
corvusinfra-root config-core: ^ group registration for the #opendev namespace is in progress; the channel is now registered to us19:14
*** jamesmcarthur has joined #openstack-infra19:17
*** bobh_ has joined #openstack-infra19:17
*** bobh has quit IRC19:19
*** smarcet has quit IRC19:20
mriedemclarkb: i'm seeing gate failures but not getting comments from e-r, is the worker ok?19:22
clarkbmriedem: I don't see anything obivously wrong in the logs19:29
clarkbbut these tend to be painful to dig through because tehy have the gerrit comments in them which have things like "traceback" in them too :)19:29
mriedemew yeah19:30
mriedemshould maybe audit what/how e-r logs stuff to make that more useful19:30
clarkbmriedem: it looked at https://review.openstack.org/#/c/628511/ recently which recently failed19:34
mriedembut didn't report on it19:35
fungithanks for the update, corvus!19:35
clarkbno, but that could be because there were no matches? I'm still trying to run through what it did there19:36
clarkbit found all console logs were ready for all those failed jobs19:36
mriedemok i don't think e-r comments on tox job failures...looking19:36
*** dkehn has quit IRC19:36
fungianteaya: thanks for the kudos! it's not why i'm here, but it's always nice to feel appreciated nonetheless19:37
mordredclarkb, fungi, corvus: https://review.openstack.org/#/c/511828/ <-- could I get a +A on that? I'm gonna see if I can't finish driving the logs/artifacts stack in19:37
clarkbmriedem: ya I don't think it does19:37
fungimordred: lookinb19:37
fungier, looking19:37
clarkbmriedem: do you hvae an example where you expected it to comment? I can try looking that direction instead19:37
clarkbcorvus: safe to approve the accessbot change? looking at chanserv it appears to be ready19:38
*** dkehn has joined #openstack-infra19:38
mriedemhttps://review.openstack.org/#/c/628302/19:39
*** wolverineav has quit IRC19:39
*** wolverineav has joined #openstack-infra19:39
clarkbmriedem: I wonder if our regex only matches the legacy jobs19:39
clarkbmriedem: jobs_re=(dsvm|tripleo)19:40
clarkbthat is at least part of the problem19:40
mriedemyeah JOBS_RE = 'dsvm'19:42
*** jamesmcarthur has quit IRC19:43
*** jamesmcarthur has joined #openstack-infra19:43
mriedemdo we have an elasticRecheck.conf in system-config or something?19:44
anteayafungi glad the appreciation is felt :)19:44
mordredfungi: thanks!19:46
mriedemhttp://git.openstack.org/cgit/openstack-infra/puppet-elastic_recheck/tree/templates/elastic-recheck.conf.erb19:46
clarkbmriedem: http://git.openstack.org/cgit/openstack-infra/puppet-elastic_recheck/tree/templates/elastic-recheck.conf.erb19:46
*** bhavikdbavishi has quit IRC19:46
clarkbya19:46
mriedemi win19:46
* anteaya hands mriedem a cookie19:46
mriedemyeah so i suppose that should include |devstack|tempest19:46
*** jamesmcarthur has quit IRC19:46
mriedemand |grenade19:47
*** jamesmcarthur has joined #openstack-infra19:47
mriedempatch incoming19:47
clarkbcorvus: puppet is broken on adns1.openstack.org because /etc/ansible/hosts/group_vars/adns.yaml sets dnssec_keys to include opendev. I think this means on the flip side adns1.opendev.org should already have keys set up for zuulci.org and zuul-ci.org19:49
clarkbcorvus: rather than try and support two nameservers like this should we just move ahead with moving zuul*.org to opendev.org name servers?19:50
clarkbcorvus: I think you need to do glue things on the registrar side (and we need to double check zone files are in place in addition to the keys), so I don't think I should just go ahead and do that19:50
mordredclarkb: I agree with you19:50
openstackgerritMerged openstack-infra/project-config master: Use ensure-output-dirs in base-test  https://review.openstack.org/51182819:50
*** eernst has joined #openstack-infra19:51
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: DNM Testing ensure-output-dirs  https://review.openstack.org/62866819:51
openstackgerritMatt Riedemann proposed openstack-infra/puppet-elastic_recheck master: Update elastic-recheck jobs_re config  https://review.openstack.org/62866919:51
fungiyeah, it did appear the zones were being served both by adns1 and ns1/219:51
fungiwhen i was figuring out how to reload nsd19:51
*** jamesmcarthur has quit IRC19:52
openstackgerritMatt Riedemann proposed openstack-infra/puppet-elastic_recheck master: Update elastic-recheck jobs_re config  https://review.openstack.org/62866919:52
clarkbanother option is to not use create_resources on the dnssec keys and explicitly list them in puppet as we explicitly list the zones in puppet19:52
*** eernst has quit IRC19:53
clarkbif we want to keep the old servers around longer19:53
fungihttp://paste.openstack.org/show/739941/19:53
fungithe serials differ though19:53
fungichecking that out now19:53
clarkboh interesting19:54
fungithe on-disk serial for /var/lib/bind/zones/zuul-ci.org/zone.db is 1522167762 for both of them19:55
fungii suspect those are the serials for the signed zones19:55
fungiyeah, all three are from 2018-03-2719:56
fungimere seconds apart19:56
pabelangerclarkb: logan-: +319:57
*** hamerins has quit IRC19:59
fungithe zonefiles themselves have identical checksums19:59
fungiso i think this is fine19:59
*** hamerins has joined #openstack-infra20:00
clarkbya so next steps should be updating registrar and our ns records then we cna clena up the old servers and the puppet manifests20:04
fungiand the zsk20:05
*** eernst has joined #openstack-infra20:06
fungibecause pretty sure the zsk on openstack is different from the one on opendev20:06
fungids records20:06
fungiwhatever20:06
clarkbah ya thats server specific right20:07
fungiwell, it's tied to the master nameserver generally20:07
fungiwe could have reused them, but pretty sure a fresh one was generated on adns1.opendev.org (which is preferable anyway)20:08
openstackgerritMerged openstack-infra/project-config master: Add node host_ids to logstash fields  https://review.openstack.org/62864220:09
*** lifeless has joined #openstack-infra20:12
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Use ensure-output-dirs in base  https://review.openstack.org/51182920:17
mordredpabelanger: ^^ would you remove your -2 from that - also, a +2 would be lovely20:17
mordredpabelanger, fungi, clarkb: ^^ it has been tested with base-test here: http://logs.openstack.org/68/628668/1/check/tox-py35/1eecc09/ara-report/result/2b78b874-968b-40fa-b571-4f255084c9a0/20:17
pabelanger+220:18
mordredpabelanger: (I'm picking up the baton on this one again :) )20:18
pabelangerYay20:18
mordredpabelanger: maybe between the two of us we'll get it done before we both retire20:19
pabelangerikr20:19
mordredpabelanger: when I"m done, I might go beat the drum on your base-minimal stack too20:19
mordredbut I wanna get this darned thing done :)20:19
pabelangermordred: sure, wfm. I should have some cycles to help also. Let me know20:20
fungimordred: is that also a candidate for the example base and base-test jobs in teh zuul-base-jobs repo?20:22
openstackgerritMerged openstack-infra/system-config master: Turn on future parser for one nodepool launcher  https://review.openstack.org/61628820:22
mordredfungi: yah. lemme do that real quick20:22
fungii mean, let's dogfood it in project-config first for sure20:23
fungisince as previously discussed ni #zuul we do apparently have users of the zuul-base-jobs repo so may need to announce any behavior changes there20:23
*** hamerins has quit IRC20:23
openstackgerritMonty Taylor proposed openstack-infra/zuul-base-jobs master: Add ensure-output-dirs to base jobs  https://review.openstack.org/62867420:25
mordredfungi: there you go20:25
fungithanks!20:25
mordredfungi: I believe these changes should be mostly transparent to people using zbj (and to use in project-config)20:25
openstackgerritMerged openstack-infra/project-config master: Use ensure-output-dirs in base  https://review.openstack.org/51182920:34
mordredwoot. we'v made progress20:35
martinkennellyI am trying to use the ci sandbox (https://git.openstack.org/cgit/openstack-dev/ci-sandbox/), but I am having trouble connecting to it. I am not sure what to put in as the frontend url for it. I only have the git link to it. I cant seem to find it in any documentation.20:37
mordrednow that I have your attention - pabelanger, fungi: https://review.openstack.org/#/c/511843 ... pabelanger wrote a different version - but if it's ok with y'all I'd like to just get in the single role for now (so we can reduce the number of base-test patches we need to cycle through) and maybe refactor later? or would you prefer to go with the split approach?20:37
clarkbmartinkennelly: it is a project hosted by our gerrit server, review.openstack.org. Depending on your CI tooling the tooling will talk to it over https or ssh20:38
clarkbmartinkennelly: are you using Zuul or something else?20:38
openstackgerritMerged openstack-infra/puppet-elastic_recheck master: Update elastic-recheck jobs_re config  https://review.openstack.org/62866920:39
martinkennellyclarkb: Jenkins. I see. I see. Think its clicking with me now.20:39
clarkbmartinkennelly: with jenkins you'll configure the gerrit plugin to talk to our gerrit and listen for events from the sandbox repo20:39
*** bdodd has quit IRC20:40
martinkennellyclarkb: Yes, thats what I am configuring. I am just unsure how to link it to the ci sandbox. I have no trouble doing it with normal gerrit on review.openstack.org20:40
clarkbmartinkennelly: it should be the same, our sandbox repo is just lik eevery othe rrepo except tha tanyone can vote on it to show people how that works20:41
martinkennellyclarkb: I understand now. You have just made that connection with me. Thank you so much for your time.20:41
clarkbno problem20:42
clarkbthe goal with that is its a plcae you can hook up to that looks identical to the real repos, make sure everything works then switch the config over20:42
martinkennellyclarkb: yes yes :) and I can post test patches to this sandbox to trigger it off.20:42
clarkbexactly20:43
martinkennellyclarkb: have a nice day :)20:43
corvusclarkb: yes accessbot/meetbot changes should be fine to aprv20:49
corvusclarkb, fungi: i'll start working on moving zuul-ci.org over; we don't need to update glue records, just ns records and possible the zsk as fungi mentioned (i'll double check that)20:50
fungioh, right, no glue because the nameservers aren't in the zuul-ci.org zone20:50
fungijust need to update ns records at the registrar (as well as in the zone we're serving, though those should be doable asynchronously) and also maybe the ds20:51
mordredcorvus, clarkb: you might also have opinions on https://review.openstack.org/#/c/511843 and on which way we might want to go with that20:51
*** jtomasek has quit IRC20:51
clarkboh right20:51
corvusmordred: ack; will look as soon as i check on the ns stuff20:52
fungidepending on the registrar, they may want to see the ns records in the zone changed first, but doubtful in this case20:52
clarkbmordred: I'm lacking the context to know what the options are there?20:53
openstackgerritJames E. Blair proposed openstack-infra/zone-zuul-ci.org master: Move ns records to opendev  https://review.openstack.org/62868020:53
clarkbmany roles or one?20:53
mordredclarkb: sorry - there's a comment on the change- the other option is https://review.openstack.org/#/c/583346/ - which is to make three different roles20:53
clarkbwhen debugging and trying to understand things I tend to find grouping things into larger roles to be easier20:54
clarkbI'd probably go with one role over 3 for this for that reason20:54
openstackgerritJames E. Blair proposed openstack-infra/zone-zuul-ci.org master: Move ns records to opendev  https://review.openstack.org/62868020:55
openstackgerritMerged openstack-infra/project-config master: Add #opendev to accessbot  https://review.openstack.org/62866420:56
mordredclarkb: cool. well, https://review.openstack.org/#/c/511843/ is ready for review/landing then :)20:57
openstackgerritMerged openstack-infra/zone-zuul-ci.org master: Move ns records to opendev  https://review.openstack.org/62868020:58
corvusfungi: i believe the same zsk is in use on both sets of servers and are both signed by the same ksk, so i don't think the ds record needs updating with the registrar.  did you see something different?21:03
fungicorvus: ahh, no i hadn't looked. i thought i remembered you creating a new one when bootstrapping adns1.opendev.org but double-checking now21:03
corvusfungi: (we did make a new ksk/zsk for the opendev.org zone itself, so it doesn't share a key with zuul-ci.org)21:03
fungioh, right. that's what i was thinking of. it's tied to the zone not the master nameserver21:04
corvusok, so i think we're on the same page then.  we shouldn't need to do anything other than change the ns records (in dns and with the registrar)21:05
fungiconfirmed, the contents of all files in /etc/bind/keys/zuul-ci.org/ on both servers have matching checksums21:06
corvus#status log updated zuul-ci.org domain registration to use ns[12].opendev.org as nameservers21:07
openstackstatuscorvus: finished logging21:07
corvusokay, so sometime in the next day or so all of the respective rube goldberg machines should have updated and it'll be moved :)21:08
clarkband then we can delete old servers next week21:08
clarkb\o/21:08
fungiand yes, the change to the zones is approved now. i'll give the osf lot a heads up that it can be changed on the zuulci.org domain as well if they can convince the registrar21:08
fungithough i'll probably wait until it's working for zuul-ci.org first21:09
clarkboh are those separate?21:09
clarkbhrm maybe not quite so quick to deleting old servers21:09
fungiyeah, the osf registered zuulci.org through csc, over concerns that users don't know how to type hyphens or tend to assume they're optional21:09
fungibut it's still delegated to our nameservers21:10
corvusfungi: probably a good idea to submit the request after zuul-ci.org is confirmed working;  clarkb: after that though, i vote we don't wait more than a day; i'm not going to lose sleep if zuulci.org doesn't work.21:10
mordredI don't know what people have-against hyphens21:10
anteayait is the horizontalness of it all21:10
fungiif only rfc 883 had included | instead of -21:12
corvushuzzah!  i am a rook contributor!  https://github.com/rook/rook/pull/2450  :)21:12
fungicongrats!21:12
fungirapid turn-around21:13
clarkbmordred: I've +2'd the output dir collection change but left a comment for a followup thing21:14
corvusmordred, clarkb: i was just looking at that and wanted to refresh my memory about that -- what does/will happen with the stuff in those 3 dirs?21:15
clarkbcorvus: they get copied into executor.log_root/docs /logs /artifacts21:15
clarkbcorvus: then our existing log publishing will copy from that location out21:15
corvusright -- what's the plan there?21:15
clarkb(multinode jobs have the dirs prefixed by the hostname)21:15
clarkbcorvus: I think we add it to our base job, notify people that it can be used in this way then let them update jobs as appropriate? maybe we push to devtsack/tempest an update as an example?21:16
mordredcorvus: https://review.openstack.org/#/c/511853/5 and https://review.openstack.org/#/c/511858/4 have the changes to do thigns with them21:16
corvusyeah those 2 changes are the question i was trying to ask :)21:17
mordredyah - then once the base job will publish the thigns in those changes, we can let people know and they can start deleting custom fetching/uploading code21:17
corvusin particular, the current plan is basically about structured uploading to the log server, and not necessarily (at this point) anything about publishing docs to afs or tarballs to tarballs21:18
corvuswhich i think is *great* because these 3 changes, plus the promote work i'm doing, will get us to a really good place there21:19
mordredyeah. the stuff that's written just gets us to log uploading21:19
corvusso i think the story about publishing docs/tarballs should, in the future, basically be "use the promote pipeline to fetch previously uploaded logs/docs/artifacts" from the logserver and publish them in the right place.21:19
mordredyup21:19
fungiso the artifacts in that scenario are uploaded to the logserver by jobs run in the gate pipeline?21:20
corvusmordred: 511843 +3 -- though i agree with clark's doc++ comment21:20
mordredcorvus: yah - I do too - followup coming21:21
corvusfungi: yep21:21
fungiand then the promote pipeline jobs grab the gate pipeline artifacts?21:21
fungimakes sense21:21
corvusfungi: yep21:21
corvusfungi: zuul now has a concept of structured data for artifacts, so the promote jobs will be able to query zuul to find out where to get their artifacts21:21
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Combine artifact URLs with log_url if empty  https://review.openstack.org/62806821:21
fungisince we'll only trigger promote if the gate pipeline results in a merge (so all voting jobs succeed)21:22
corvusfungi, clarkb, mordred: ^ though i need that change to move forward21:22
corvusfungi: yep21:22
corvuser, let me write more in the commit message21:22
fungii guess for promote we should avoid jobs which overwrite the same published artifacts (e.g., documentation) since they might race?21:23
corvusfungi: promote can be supercedent21:23
fungias long as we don't need it to run jobs for every merge21:24
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add longer description to fetch-output  https://review.openstack.org/62868321:24
*** eernst has quit IRC21:25
mordredclarkb, corvus: ^^21:25
fungii guess that's the trade-off. there will be jobs we want run independently for every change which merges, and other jobs we want run for the most recent merge but it's okay if several roll up together in a short period of time21:25
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Combine artifact URLs with log_url if empty  https://review.openstack.org/62806821:25
corvusnow with actual commit message ^21:26
*** rlandy|rover has quit IRC21:26
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Combine artifact URLs with log_url if relative  https://review.openstack.org/62806821:26
mordredcorvus: SO ...21:26
corvusnow with sensible commit message ^21:27
corvus4 patchets, all commit message changes.  it's friday.21:27
clarkbis that why everything is ewird today?21:27
fungioh, wait, friday is today?21:27
* fungi 's sense of time is entirely screwed up21:28
clarkbI keep reminding myself "puppet-4 changes"21:28
clarkband somehow I am getting things done, but I also haveto do administrivia paperwork things, but maybe its easier to do that later :)21:28
mordredcorvus: you can remove a chunk of that and just urljoin all the time...21:28
*** tomaw_ has joined #openstack-infra21:29
corvusmordred: oh ok.  i'll try that, and i'll also add a test21:29
mordredcorvus: http://paste.openstack.org/show/739943/21:29
*** tomaw has quit IRC21:30
mordredcorvus: I learned **ALL** about this dealing with catalog urls, version discovery urls, relative paths - and openstack services that serve broken base urls in their discovery docs21:30
mordredI wish I new much less21:30
corvusmordred: should i keep the urljoin in the exception handler?  or does it never fail?21:31
mordredI don't believe it ever fails21:31
mordredI mean, unless you passed it a non-string21:31
*** tomaw_ is now known as tomaw21:32
corvusthat *should* be validated21:32
corvusbut i'll leave it just in case21:32
mordredcorvus: main key - make sure the log url has a trailing slash - that way 'foo/bar', '/foo/bar' and 'https://somewhere.else.io' will all override or append as you'd expect them to21:33
clarkbgit review struggles with ^21:33
clarkbmordred: I thought urljoin always rooted things though21:34
clarkbah it depends on that trailing / as you say21:36
clarkbif oyu odn't have it on the url side it roots oetherwise its appens21:36
mordredyeah21:36
mordredhttp://paste.openstack.org/show/739944/ might be a clearer example21:37
*** dave-mccowan has quit IRC21:42
*** bobh_ has quit IRC21:47
*** bobh has joined #openstack-infra21:49
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Combine artifact URLs with log_url if relative  https://review.openstack.org/62806821:50
corvusokay, that's way more complete; thanks mordred :)21:50
*** smarcet has joined #openstack-infra21:51
clarkblogan-: I see build_hostids in elasticsearch now. thanks!21:51
clarkbmriedem: your config update for elastic-recheck watch bot seems to have applied about 20 minutes ago21:51
mordredcorvus: yay!21:53
mordredcorvus: oh no - now there's more lines to review. what was I thinking?21:54
*** bobh has quit IRC21:54
corvusbut i totally removed one21:54
mriedemclarkb: ok21:57
mriedemi'm out next week,21:58
mriedemso not sure i'll get to benefit until i'm back21:58
mordredclarkb: can you see what failed here: http://logs.openstack.org/43/511843/4/gate/openstack-zuul-jobs-linters/01ae832/job-output.txt.gz ?21:58
clarkbcorvus: left a note about your docs example (its path is not relative)21:59
*** xek has joined #openstack-infra21:59
mordredclarkb: or did ansible-lint just now make a new release?21:59
clarkbmordred: that looks like ansible lint >= 4.0.022:00
clarkbmordred: not just now but semi recently yes22:00
mordredyup. jeez22:00
mordredshall I submit a cap patch?22:00
clarkbI think that is what was done elsewhere22:00
*** boden has quit IRC22:01
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Combine artifact URLs with log_url if relative  https://review.openstack.org/62806822:01
clarkbmordred: it is also affecting system-config too if you want to fix that repo while you are at it22:01
corvusclarkb: fixed; mordred re+2 pls?22:01
clarkbI'm guessing some transitive dep updated their ansible lint dep?22:02
clarkbactually I have a pin proposed already fo rsystem-cofnig?22:02
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184322:02
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Pin ansible-lint to < 4  https://review.openstack.org/62869222:02
clarkbmordred: https://review.openstack.org/#/c/625999/ if you want to +3 that one22:02
corvusmordred: i think you need to rebase 69222:02
mordredclarkb: +A22:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Pin ansible-lint to < 4  https://review.openstack.org/62869222:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184322:03
mordredcorvus, clarkb: ^^ rebased and stuff - I also squashed the readme fix since it's a respin22:04
*** trown is now known as trown|outtypewww22:04
*** jcoufal has quit IRC22:10
corvusfungi, mordred: https://review.openstack.org/628665 is ready for +3 (accessbot has run)22:14
openstackgerritMerged openstack-infra/system-config master: Cap ansible-lint at < 4.0.0  https://review.openstack.org/62599922:20
corvusinfra-root: i'd like to restart gerrit now and see if gitea replication survives now that we've increased the lb timeout22:24
mordredcorvus: I would also like for you to do that22:25
logan-clarkb: awesome, looks great. thanks22:25
corvus#status log restarted gerrit to restart gitea replication thread22:26
openstackstatuscorvus: finished logging22:26
*** whoami-rajat has quit IRC22:37
openstackgerritMerged openstack-infra/system-config master: Add #opendev to meetbot  https://review.openstack.org/62866522:38
*** jaosorior has quit IRC22:42
*** smarcet has quit IRC22:49
*** jbadiapa has joined #openstack-infra22:50
*** bobh has joined #openstack-infra22:52
openstackgerritMerged openstack-infra/zuul-jobs master: Pin ansible-lint to < 4  https://review.openstack.org/62869222:55
*** hwoarang has quit IRC23:03
*** hwoarang has joined #openstack-infra23:04
corvusmnaser: i'm not sure how, but i seem to have caused a server to go into error state by asking for a hard reboot23:07
corvusmnaser: | fault                       | {u'message': u"Domain not found: no domain with matching uuid '6913a24e-8da5-4805-b075-4d539ccda138' (instance-0007c6c6)", u'code': 500, u'created': u'2019-01-04T23:06:12Z'} |23:08
*** xek has quit IRC23:09
clarkbcorvus: I think that means nova couldn't find the VM in libvirt to restart it23:09
clarkb(which is odd)23:09
clarkbespecially if the instance is running23:09
corvusyeah, it was running.  it just hung at the end of the soft reboot process23:10
mordredcorvus: I'm impressed23:11
*** rascasoft has quit IRC23:12
clarkbI've reapproved the fetch output role23:12
corvusthis is at the end of a long series of disappointments: k8s container wouldn't terminate; docker container process is zombied; reboot the system; soft reboot the instance; hard reboot the system23:12
openstackgerritMerged openstack-infra/system-config master: Turn on future parser for all nodepool launchers  https://review.openstack.org/61628923:14
openstackgerritMerged openstack-infra/system-config master: Turn on future parser for one nodepool builder  https://review.openstack.org/61629023:14
clarkbwoo got ^ moving again after the system-config ansible-lint pin23:15
mordredcorvus: perhaps that series of disappointments is all caused by something going south on the hypervisor host - so it's really just all one disappointment with many faces23:15
corvusmordred: i hope i haven't ruined mnaser's friday23:15
mnasercorvus: mordred no idea how that’s even possible. I will dig in 20 minutes23:17
mordredcorvus: look, he's excited about the possibility of joyous debugging you've provided him23:17
*** bdodd has joined #openstack-infra23:19
*** bobh has quit IRC23:21
*** Adri2000 has quit IRC23:22
*** Adri2000 has joined #openstack-infra23:25
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add fetch-output to base-test  https://review.openstack.org/51185023:30
*** bobh has joined #openstack-infra23:36
*** owalsh_ has joined #openstack-infra23:38
*** owalsh has quit IRC23:41
*** openstack has joined #openstack-infra23:44
*** ChanServ sets mode: +o openstack23:44
clarkbhogepodge: who is a good person to point out http://logs.openstack.org/11/627411/7/gate/openstack-tox-py35/db0d0d5/job-output.txt#_2019-01-04_22_25_48_666928 that test bug to in airship?23:45
clarkbI suppose I could just file a bug23:45
clarkbhttps://storyboard.openstack.org/#!/story/2004703 if anyone is curious23:54
*** bobh has quit IRC23:58
hogepodgeclarkb: I'll ping the team. The #airshipit channel is a good place for this23:59

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