Tuesday, 2019-03-05

corvusclarkb: lgtm00:00
openstackgerritClark Boylan proposed openstack-infra/system-config master: Replicate to the standalone gitea severs  https://review.openstack.org/64043100:01
clarkbcorvus: mordred ^ there is the update and I put more info in the commit message about it00:02
clarkburl = ssh://git@github.com:22/${name}.git was the full config line on review-dev ftr. Our puppetry appends the {name}.git suffix00:03
dmsimardianw: find anything ?00:04
ianwdmsimard: umm, sorry had to duck out, i dunno where the xvdb came from00:05
*** hwoarang has quit IRC00:06
ianwit's booted into 4.15.0-45-generic00:07
ianwi notice that bridge is running 4.15.0-22, but has 44 & 45 installed, presumably through updates00:07
*** hwoarang has joined #openstack-infra00:08
*** sdake has joined #openstack-infra00:08
clarkbianw: yes likely just pending a reboot there00:09
ianwyeah, i wonder though if the snapshot is in a weird state00:10
ianwdmsimard: i may have screwed it up in emergency mode, i'm just going to try rebuilding it and examining it again, knowing that i'm looking for swapped partitions00:10
corvusclarkb: i tested your replication url syntax locally with docker-compose and gerrit and gitea, and it seems to work00:11
corvusclarkb: i'm pretty confident that should address frickler's comment, and mordred previously approved, so i'm going to +300:12
*** sdake has quit IRC00:12
corvus(we still have the manual gerrit restart step to do to actually kick that off.00:12
*** sdake has joined #openstack-infra00:12
clarkbcorvus: yup and review-dev is running the exact same version of gerrit and the plugin so that should be a good other test00:13
clarkband ya we'll need to restart gerrit. Remember when we did that ~weekly?00:13
openstackgerritMerged openstack-infra/storyboard-webclient master: Rebuild node-sass bindings when running the webclient dev server  https://review.openstack.org/63461500:14
openstackgerritMerged openstack-infra/storyboard-webclient master: Make tags section more consistent  https://review.openstack.org/63361300:14
*** wolverineav has joined #openstack-infra00:16
corvusclarkb, fungi: i've +3d https://review.openstack.org/636775 since the git playbook is now < 30m00:16
pabelangerodd, I cannot seem to connect to https://zuul.o.o on this public network, but https://zuul.opendev.org works00:17
corvusdmsimard: http://zuul.openstack.org/config-errors00:19
clarkbpabelanger: they are the same server and you should get the same A or AAAA record00:19
pabelangerclarkb: yah, dig looks to be the same00:20
pabelangerhttp://paste.openstack.org/show/747263/00:20
clarkbI can confirm both work from here so it isn't a universal problem00:21
pabelangerI'm not going to spend too much time on it, suspect something on firewall is filtering something00:21
*** jcoufal has joined #openstack-infra00:22
*** Adri2000 has quit IRC00:22
*** Adri2000 has joined #openstack-infra00:25
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use a buildset registry in the container jobs  https://review.openstack.org/64090000:28
openstackgerritJames E. Blair proposed openstack-infra/zuul-preview master: DNM: break dockerfile  https://review.openstack.org/64090100:30
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: test broken zuul-preview dockerfile  https://review.openstack.org/64090300:31
openstackgerritJames E. Blair proposed openstack-infra/zuul-preview master: DNM: fix dockerfile  https://review.openstack.org/64090400:31
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: test fixed zuul-preview dockerfile  https://review.openstack.org/64090500:32
openstackgerritJosef Wells proposed openstack-infra/zuul master: quickstart: mount playbooks volume for web image  https://review.openstack.org/64090600:33
clarkbcorvus: the current run of ansible should docker compose up with up to date images00:33
corvusclarkb: cool, though i'm not sure if we have any outstanding changes at this point00:33
clarkbcorvus: the gitea sign in00:34
corvusclarkb: ah nice00:34
*** ekultails has quit IRC00:35
*** sreejithp has quit IRC00:37
*** uberjay has quit IRC00:38
*** uberjay has joined #openstack-infra00:38
*** jcoufal has quit IRC00:39
*** sdake has quit IRC00:41
*** wolverineav has quit IRC00:47
*** markvoelker has joined #openstack-infra00:48
*** ericyoung has quit IRC00:50
*** ericyoung has joined #openstack-infra00:51
clarkbfwiw seems that afs02.dfw.openstack.org is currently not speaking ssh00:53
clarkb(it is holding up the ansible run, I was expecting the limestone mirror)00:53
corvusclarkb: looks like it stopped responding at 00:30 http://cacti.openstack.org/cacti/graph.php?action=zoom&local_graph_id=6403&rra_id=5&view_type=&graph_start=1551744735&graph_end=1551747020&graph_height=120&graph_width=500&title_font_size=1200:55
openstackgerritJeremy Stanley proposed openstack-infra/storyboard master: Use ColumnElements instead of strings in migration  https://review.openstack.org/64089200:57
*** wolverineav has joined #openstack-infra00:58
fungiclarkb: corvus: i don't suppose it's related to the cinder block device outage saturday?00:58
clarkbfungi: well it was working, but maybe it was unhappy and that finally tipped it over?00:58
clarkbI've actually got to pop out now as I said I'd take kids outside before sunset00:58
fungirackspace opened a ticket saying they rebooted the backend providing afs02.dfw.openstack.org/main0200:59
clarkbbut can take a look after if still necessary00:59
*** hwoarang has quit IRC00:59
openstackgerritMerged openstack-infra/system-config master: Reduce timeouts in run_all.sh  https://review.openstack.org/63677500:59
openstackgerritMerged openstack-infra/system-config master: Replicate to the standalone gitea severs  https://review.openstack.org/64043100:59
*** hwoarang has joined #openstack-infra01:00
corvus(openstack) volume list01:00
corvuspublicURL endpoint for volumev2 service in DFW region not found01:00
corvuswhy doesn't openstack work?01:00
openstackgerritMerged openstack-infra/devstack-gate master: Add the setup support for stable/stein  https://review.openstack.org/64064101:01
fungiit works until you have to reboot something :/01:01
corvus(openstack) console log show afs02.dfw.openstack.org01:02
corvusThere is no such action: os-getConsoleOutput (HTTP 400) (Request-ID: req-a3b62e80-2627-455e-bcb8-dd9c523c50fc)01:02
corvusand now it's time for me to go too.01:02
fungii think the console depends on the kernel not being totally crashed01:03
*** ricolin has joined #openstack-infra01:03
fungithe serial console anyway01:03
fungihrm, though no the hypervisor should be buffering it01:04
*** sdake has joined #openstack-infra01:07
ianwyeah, i wonder if something's up, that server i'd "quickly" rebuild has been going for over an hour01:17
*** markvoelker has quit IRC01:22
*** sthussey has quit IRC01:22
ianwright, so it's the xvde that's causing all this snapshot foo foo01:28
dmsimardcorvus: thanks, I must have screwed up somewhere in retiring 3 repos01:28
dmsimardI'm not finished cleaning up, I got tied up in some rabbit holes today :D01:30
ianwbridge.o.o doesn't appear to have any volumes attached though01:30
dmsimardianw: yeah it really took more than an hour to spawn01:30
dmsimardeverything is just sluggish01:31
ianwi can see it now, it fails mounting opt, that's what gets you into the prompt01:31
dmsimardoh, which sort of makes sense01:31
dmsimardbecause it's in the fstab and is probably not in the new VM01:31
ianwi'm either blind or where is the bridge.o.o volume?01:31
ianwyeah, and doesn't have appropriate don't mount flags01:31
dmsimardmaybe we could try mounting a volume and give it a hint for xvde01:31
dmsimardianw: iscsi ?01:31
clarkbthe e is ephemeral. part of the flavor01:32
dmsimardoh01:32
ianwahhhh, that's right01:32
fungii was able to pull up the graphical console for afs0201:35
ianw"data disk" it's called01:35
fungibuffer io error on dm-0 xvdc01:36
fungiand a bunch of hung kernel task warnings01:36
fungiand no working getty01:36
*** dpawlik has joined #openstack-infra01:37
ianwdependency failed for /opt & local file systems01:40
ianwso i wonder how to make this snapshotable01:40
*** dpawlik has quit IRC01:42
fungiprobably can't snapshot servers using an ephemeral disk01:46
*** rascasoft has joined #openstack-infra01:46
fungii sort of wonder if resize will have similar issues. may want to backup whatever we've got on that fs01:46
ianwfungi: i think maybe nofail would at least let it *boot*, and it would be a small swizzle to get it going01:46
ianwfungi: aiui, there's no resize as such, it's new image from snapshot01:47
ianwlet me get into emergency console and add nofail, and see if it at least boots to a state we could then set the right uuid's and have a working server.  if so, we can do that on bridge.o.o01:47
fungiianw: the snapshot experiment was to make sure we've got a good copy of the server before we try to resize01:47
*** dave-mccowan has joined #openstack-infra01:48
fungidmsimard confirmed he was able to use the api to do a resize of a server booted from the same image and flavor as bridge.o.o01:48
*** gouthamr has quit IRC01:48
fungi(even though the option isn't provided by the dashboard any longer)01:48
dmsimardyes, the expectation is that if you don't test your backups, they're broken :)01:48
ianwoh, ok, might have missed that bit.  i'm guessing it's going to be just backend hiding making a new server anyway :)01:49
fungidmsimard: i can attest to this ;)01:49
dmsimardianw: so how would we restore from a snapshot should it be necessary ?01:49
fungiianw: yes, i believe it is, though may reuse port and ip address01:49
fungiand uuid01:49
ianwdmsimard: the system would boot, modify the fstab for the newly attached ephemeral disk, reboot, and it should be back01:50
ianwi don't think anything on /opt required for boot01:50
*** dmellado has quit IRC01:50
*** stevebaker has quit IRC01:50
*** rascasoft has quit IRC01:50
fungiand hopefully we separately back up what's in /opt or just don't care if it gets blown away01:50
dmsimardianw: well, I mean, there's a lot we can do here -- we can allow ourselves to take a second (fixed) snapshot :p01:50
ianwfungi: i'm assuming that when it ansible's itself, it writes /opt01:52
ianwand typing that, i just realised that the playbooks are probably on /opt/system-config01:52
ianwhrm, it seems like figuring this out now, while it's not a fire-drill, is probably not a bad idea01:52
fungiagreed01:52
dmsimardwe have 128MB on /opt01:52
dmsimardthis can live on / where we have 23GB available :D01:53
ianwit presumably wouldn't need more than system-config01:53
dmsimardwe could probably even take out the swap since the plan is to upgrade to a flavor with more ram01:53
fungiso anyway, it seems that afs02.dfw is now not even able to get a graphical console connection any longer. going to try a reboot and see if it's any better01:53
dmsimardit all depends on the amount of effort for deploying a new bridge.o.o01:53
*** jamesmcarthur has joined #openstack-infra01:55
*** hwoarang has quit IRC01:55
*** hwoarang has joined #openstack-infra01:57
*** wolverineav has quit IRC01:57
ianwi think it's minimal01:59
ianwbut a resize and fiddle two lines in fstab is even more minimal :)01:59
fungiassuming the resize doesn't revert and boot the old instance again if something goes sideways with the new instance it's trying to migrate to02:00
fungithe reboot of afs02.dfw seems to have made it reachable via ssh again02:01
*** whoami-rajat has joined #openstack-infra02:02
fungi#status log afs02.dfw rebooted after hanging, likely related to outage for main02 block device on 2019-03-0202:02
openstackstatusfungi: finished logging02:02
ianwwell nofail fixed up the boot02:04
dmsimardianw: taking the snapshot and booting the VM takes a while though02:04
*** jamesmcarthur has quit IRC02:04
ianwguess what it doesn't have ... networking02:04
dmsimardI mean, we need to keep this in mind in the future in case of actual disaster recovery02:04
dmsimardianw: I think I know that part02:04
*** sdake has quit IRC02:05
ianwyeah, we almost need to run virt-sysprep on it02:05
dmsimardianw: I'm going to bet that the network config is persisted in /etc/netplan02:05
*** rascasoft has joined #openstack-infra02:05
fungilooking at afs02.dfw, the filesystem/logvol seems to be intact following reboot02:06
*** jamesmcarthur has joined #openstack-infra02:06
fungiand with that, i'm knocking off for the evening as well02:06
*** jamesmcarthur has quit IRC02:10
*** anteaya has quit IRC02:11
*** rascasoft has quit IRC02:12
*** hongbin has joined #openstack-infra02:13
*** markvoelker has joined #openstack-infra02:19
ianwdmsimard: ok, i got 104.239.240.196 up ... let's do a check list in https://etherpad.openstack.org/p/bridge-snapshot-bringup02:26
dmsimardI'll catch up later, need to take care of something02:27
*** sdake has joined #openstack-infra02:28
openstackgerritMerged openstack-infra/storyboard master: Use ColumnElements instead of strings in migration  https://review.openstack.org/64089202:29
*** hwoarang has quit IRC02:33
*** wolverineav has joined #openstack-infra02:33
*** hwoarang has joined #openstack-infra02:35
*** jamesmcarthur has joined #openstack-infra02:37
*** psachin has joined #openstack-infra02:39
ianwdmsimard: ok, so https://etherpad.openstack.org/p/bridge-snapshot-bringup is a plan, if nothing else.  worked with 104.239.240.196 , which i've now shut down to avoid confusion02:50
*** gouthamr has joined #openstack-infra02:50
*** markvoelker has quit IRC02:51
*** jamesmcarthur has quit IRC02:51
*** jamesmcarthur has joined #openstack-infra02:54
*** dmellado has joined #openstack-infra02:55
*** wolverineav has quit IRC03:00
*** eventingmonkey has quit IRC03:01
*** wolverineav has joined #openstack-infra03:01
*** wolverineav has quit IRC03:06
*** bhavikdbavishi has joined #openstack-infra03:10
*** jamesmcarthur has quit IRC03:10
*** jamesmcarthur has joined #openstack-infra03:10
*** apetrich has quit IRC03:15
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Add activity indicator to new worklist modal and detail  https://review.openstack.org/64084503:16
*** roman_g has quit IRC03:17
*** yamamoto has joined #openstack-infra03:17
*** stevebaker has joined #openstack-infra03:25
*** wolverineav has joined #openstack-infra03:31
*** rkukura has quit IRC03:34
*** yamamoto has quit IRC03:35
*** yamamoto has joined #openstack-infra03:35
*** dpawlik has joined #openstack-infra03:38
*** dave-mccowan has quit IRC03:42
*** dpawlik has quit IRC03:43
*** markvoelker has joined #openstack-infra03:48
*** sdake has quit IRC03:51
*** gyee has quit IRC03:54
*** jamesmcarthur has quit IRC03:57
*** rascasoft has joined #openstack-infra04:00
*** rascasoft has quit IRC04:04
*** wolverineav has quit IRC04:07
*** ramishra has joined #openstack-infra04:09
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add triggers information to pipeline list  https://review.openstack.org/63767004:15
*** udesale has joined #openstack-infra04:18
*** markvoelker has quit IRC04:22
*** ykarel|away has joined #openstack-infra04:31
*** ykarel|away is now known as ykarel04:31
*** raukadah is now known as chandankumar04:34
*** jamesmcarthur has joined #openstack-infra04:35
*** jamesmcarthur has quit IRC04:44
*** armax has quit IRC04:48
*** jamesmcarthur has joined #openstack-infra04:48
*** owalsh_ has joined #openstack-infra04:50
*** owalsh has quit IRC04:51
*** jamesmcarthur has quit IRC04:52
*** wolverineav has joined #openstack-infra05:00
*** jamesmcarthur has joined #openstack-infra05:00
*** wolverineav has quit IRC05:06
*** janki has joined #openstack-infra05:08
*** hongbin has quit IRC05:09
*** markvoelker has joined #openstack-infra05:19
*** udesale has quit IRC05:22
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Add DIB_APT_MINIMAL_CREATE_INTERFACES toggle  https://review.openstack.org/63986505:28
*** jamesmcarthur has quit IRC05:35
*** rkukura has joined #openstack-infra05:36
*** dpawlik has joined #openstack-infra05:39
*** dpawlik has quit IRC05:44
*** jhesketh has quit IRC05:47
*** jhesketh has joined #openstack-infra05:48
*** sdake has joined #openstack-infra05:48
AJaegerfungi, clarkb, docs-old can be removed that is old content05:50
*** sdake has quit IRC05:50
*** markvoelker has quit IRC05:51
*** pcaruana has joined #openstack-infra05:52
*** sdake has joined #openstack-infra05:56
*** jamesmcarthur has joined #openstack-infra06:03
*** pcaruana has quit IRC06:07
*** jamesmcarthur has quit IRC06:09
*** rh-jelabarre has quit IRC06:13
*** jtomasek has joined #openstack-infra06:16
*** dpawlik has joined #openstack-infra06:19
*** ccamacho has quit IRC06:21
*** dpawlik has quit IRC06:24
*** dims has quit IRC06:24
*** e0ne has joined #openstack-infra06:24
*** dims has joined #openstack-infra06:26
*** e0ne has quit IRC06:33
*** dims has quit IRC06:36
*** dims has joined #openstack-infra06:37
*** jamesmcarthur has joined #openstack-infra06:40
*** jamesmcarthur has quit IRC06:44
*** e0ne has joined #openstack-infra06:46
*** markvoelker has joined #openstack-infra06:48
*** wolverineav has joined #openstack-infra06:48
*** wolverineav has quit IRC06:53
*** jamesmcarthur has joined #openstack-infra06:56
*** jamesmcarthur has quit IRC07:01
*** dayou has quit IRC07:01
*** dayou has joined #openstack-infra07:02
*** AJaeger has quit IRC07:05
*** e0ne has quit IRC07:07
*** quiquell|off is now known as quiquell07:08
*** AJaeger has joined #openstack-infra07:09
*** slaweq has joined #openstack-infra07:11
*** wolverineav has joined #openstack-infra07:14
*** wolverineav has quit IRC07:14
*** pgaxatte has joined #openstack-infra07:16
*** markvoelker has quit IRC07:22
*** aojea has joined #openstack-infra07:23
openstackgerritMerged openstack/diskimage-builder master: Add DIB_APT_MINIMAL_CREATE_INTERFACES toggle  https://review.openstack.org/63986507:24
*** dpawlik has joined #openstack-infra07:25
*** stakeda has joined #openstack-infra07:30
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884007:30
*** flaper87 has quit IRC07:32
*** flaper87 has joined #openstack-infra07:33
*** yamamoto has quit IRC07:34
*** kopecmartin|off is now known as kopecmartin07:34
*** quiquell is now known as quiquell|brb07:35
*** yamamoto has joined #openstack-infra07:36
*** dayou has quit IRC07:37
mordredinfra-root: I'm up with an upset stomach - so I'm up, not actually here - and may not be as here in the morning as I might like07:38
mordredinfra-root: that said - the gitea dockers are having issues - upon first inspection it's something in the mariadb container - seems like a bonged restart of some sort?07:38
*** apetrich has joined #openstack-infra07:43
mordredinfra-root: on gitea01 I moved /var/gitea/db/tc.log out of the way and it seems to have been able to recover07:50
mordredI moved it out of the way on all of the hosts and things seem to be better07:50
mordredthe old tc.log files are all in /var/gitea07:51
mordredin case anyone wants to post-mortem07:51
*** udesale has joined #openstack-infra07:51
mordredlooking at https://github.com/docker-library/mariadb/issues/201 we may want to tie our mariadb image to a specific version rather than latest07:53
mordredand with that - I shall go back to communing with my stomach07:54
*** snapiri has joined #openstack-infra07:57
*** dayou has joined #openstack-infra07:58
*** udesale has quit IRC08:01
AJaegermordred: get well soon!08:05
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884008:06
*** quiquell|brb is now known as quiquell08:07
*** kjackal has joined #openstack-infra08:09
*** rascasoft has joined #openstack-infra08:09
*** rpittau|afk is now known as rpittau08:16
*** rh-jelabarre has joined #openstack-infra08:17
*** e0ne has joined #openstack-infra08:17
*** markvoelker has joined #openstack-infra08:18
*** sajauddin has joined #openstack-infra08:18
*** pcaruana has joined #openstack-infra08:18
*** jtomasek has quit IRC08:20
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Pin mariadb to 10.4  https://review.openstack.org/64094908:20
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add a stop timeout to gitea docker-compose up  https://review.openstack.org/64095008:20
mordredAJaeger: thanks! I think the worst of it right now is simply the fact that I'm awake08:20
*** tkajinam has quit IRC08:22
*** owalsh_ is now known as owalsh08:24
*** pcaruana has quit IRC08:25
*** ginopc has joined #openstack-infra08:25
*** yolanda has joined #openstack-infra08:25
*** helenfm has joined #openstack-infra08:29
*** ccamacho has joined #openstack-infra08:29
*** sdake has quit IRC08:36
*** pcaruana has joined #openstack-infra08:37
*** ykarel is now known as ykarel|lunch08:41
*** sajauddin has left #openstack-infra08:42
*** priteau has joined #openstack-infra08:42
*** sajauddin has joined #openstack-infra08:43
*** roman_g has joined #openstack-infra08:43
*** pcaruana has quit IRC08:44
*** sajauddin_ has joined #openstack-infra08:47
*** sajauddin_ has left #openstack-infra08:48
*** markvoelker has quit IRC08:51
*** jpena|off is now known as jpena08:52
*** jpich has joined #openstack-infra08:56
*** e0ne has quit IRC08:58
*** iurygregory has quit IRC08:58
*** e0ne has joined #openstack-infra09:00
*** pcaruana has joined #openstack-infra09:01
*** iurygregory has joined #openstack-infra09:01
openstackgerritRico Lin proposed openstack-infra/project-config master: Add an openstack/auto-scaling-sig repository  https://review.openstack.org/63712509:08
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088409:10
*** pgaxatte has quit IRC09:10
*** pgaxatte has joined #openstack-infra09:12
*** panda|ruck|off is now known as panda|ruck|flu09:13
*** roman_g has quit IRC09:20
*** geguileo has joined #openstack-infra09:20
AJaegermordred: yeah, that is always bad ;(09:20
*** dtantsur|afk is now known as dtantsur09:22
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884009:22
*** bhavikdbavishi has quit IRC09:27
*** bhavikdbavishi has joined #openstack-infra09:27
ricolinfungi, test job in https://review.openstack.org/637125 is green now09:30
*** IvensZambrano has joined #openstack-infra09:31
*** stakeda has quit IRC09:31
*** derekh has joined #openstack-infra09:36
*** ykarel|lunch is now known as ykarel09:39
*** roman_g has joined #openstack-infra09:41
*** markvoelker has joined #openstack-infra09:48
openstackgerritCamila Moura proposed openstack-infra/storyboard master: Creates the tag with project name and priority  https://review.openstack.org/64079309:50
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884009:52
openstackgerritCamila Moura proposed openstack-infra/storyboard master: Creates the tag with project name and priority  https://review.openstack.org/64079309:53
*** sdake has joined #openstack-infra09:55
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088409:55
*** bhavikdbavishi has quit IRC10:00
*** sajauddin has quit IRC10:02
*** cmoura has joined #openstack-infra10:03
*** yamamoto has quit IRC10:07
*** kjackal has quit IRC10:08
ttxfungi, clarkb: regarding zuul.opendev.org doing 404s on direct links -- tristanC points to missing mod_rewrite rules. The rules in puppet-zuul (https://git.openstack.org/cgit/openstack-infra/puppet-zuul/tree/templates/zuulv3.vhost.erb#n53) are indeed slightly different from what Zuul docs mandate (https://zuul-ci.org/docs/zuul/admin/installation.html#static-offload)10:09
ttxHowever I feel like there is some history I'm missing there, so I'll hold before proposing a fix10:10
*** kjackal has joined #openstack-infra10:10
*** gfidente has joined #openstack-infra10:12
*** luizbag has joined #openstack-infra10:12
*** markvoelker has quit IRC10:22
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884010:27
*** yamamoto has joined #openstack-infra10:42
*** sdake has quit IRC10:51
*** sdake has joined #openstack-infra10:55
*** bhavikdbavishi has joined #openstack-infra10:56
*** electrofelix has joined #openstack-infra10:57
*** markvoelker has joined #openstack-infra11:19
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088411:21
*** janki has quit IRC11:22
openstackgerritJens Harbott (frickler) proposed openstack-infra/elastic-recheck master: Make the functional queries test work with python3  https://review.openstack.org/64098411:26
fricklerinfra-root: mriedem: ^^ this seems to have been broken for a while, not sure how it was passing earlier, though, since the move to py3 was well before we started seeing all failures http://zuul.openstack.org/builds?job_name=elastic-recheck-tox-queries11:28
*** e0ne has quit IRC11:32
*** e0ne has joined #openstack-infra11:42
*** rascasoft has quit IRC11:43
*** markvoelker has quit IRC11:51
*** bhavikdbavishi has quit IRC11:54
*** udesale has joined #openstack-infra12:00
*** jtomasek has joined #openstack-infra12:05
*** ociuhandu has quit IRC12:14
*** ccamacho has quit IRC12:17
*** dave-mccowan has joined #openstack-infra12:20
*** xek has joined #openstack-infra12:27
*** apetrich has quit IRC12:30
*** jpena is now known as jpena|lunch12:31
*** apetrich has joined #openstack-infra12:31
*** apetrich has quit IRC12:38
*** sdake has quit IRC12:39
*** mriedem has joined #openstack-infra12:41
*** yamamoto has quit IRC12:44
*** yamamoto has joined #openstack-infra12:46
*** tosky has joined #openstack-infra12:48
*** markvoelker has joined #openstack-infra12:49
mriedemdoes something need to be restarted to kick http://status.openstack.org/elastic-recheck/data/integrated_gate.html ?12:50
mriedemwe fixed the backing query that populates that page yesterday but the results are still from feb 812:51
*** yamamoto has quit IRC12:52
*** apetrich has joined #openstack-infra12:56
*** yamamoto has joined #openstack-infra12:58
*** panda|ruck|flu is now known as panda|ruck|lunch13:03
*** gfidente has quit IRC13:06
*** weshay is now known as weshay|rover13:09
fricklermriedem: hmm, it does say "Generated at: 2019-03-05T12:50" for me. maybe you need to force-reload?13:10
mriedemah yes that always gets me13:11
fricklermriedem: did you see my comment about https://review.openstack.org/640984 before? I stumbled about that when looking at your fix earlier13:12
mriedemnope13:14
*** yamamoto has quit IRC13:14
mriedemah i see http://logs.openstack.org/69/640869/1/check/elastic-recheck-tox-queries/7b6eae5/testr_results.html.gz13:14
mriedemhmm, i dont know why elastic-recheck-tox-queries is non-voting13:14
fricklerfrom my understanding it should have been broken since the basepython was changed to py3, but it was passing until end of january. sadly the logs from then have expired http://zuul.openstack.org/builds?job_name=elastic-recheck-tox-queries13:16
*** quiquell is now known as quiquell|lunch13:16
*** janki has joined #openstack-infra13:17
*** jamesmcarthur has joined #openstack-infra13:18
*** sdake has joined #openstack-infra13:21
*** markvoelker has quit IRC13:21
*** jesusaur has quit IRC13:22
openstackgerritMerged openstack-infra/elastic-recheck master: Make the functional queries test work with python3  https://review.openstack.org/64098413:23
*** jesusaur has joined #openstack-infra13:25
fricklercorvus: clarkb: according to this doc we should be able to reload the replication config instead of doing a full gerrit restart, do we want to try that? https://gerrit.googlesource.com/plugins/replication/+doc/master/src/main/resources/Documentation/config.md#enabling-replication13:30
fricklerdidn't check whether that would need a newer gerrit version, though13:30
*** dtantsur is now known as dtantsur|brb13:31
*** jamesmcarthur has quit IRC13:31
*** rlandy has joined #openstack-infra13:36
*** oanson has joined #openstack-infra13:36
*** quiquell|lunch is now known as quiquell13:39
*** jpena|lunch is now known as jpena13:40
*** priteau has quit IRC13:41
*** dpawlik has quit IRC13:43
*** jamesmcarthur has joined #openstack-infra13:46
*** rascasoft has joined #openstack-infra13:50
*** panda|ruck|lunch is now known as panda|ruck|flu13:50
*** priteau has joined #openstack-infra13:52
*** yamamoto has joined #openstack-infra13:53
*** jcoufal has joined #openstack-infra13:53
Shrewsinfra-root: codesearch is giving a "too many open files" error. is the solution to restart houndd?14:00
*** yamamoto_ has joined #openstack-infra14:01
* Shrews tries14:02
*** gfidente has joined #openstack-infra14:03
*** yamamoto has quit IRC14:03
*** sthussey has joined #openstack-infra14:05
*** jamesmcarthur has quit IRC14:06
*** kgiusti has joined #openstack-infra14:06
Shrewsseems to be working again14:06
Shrews#status log houndd restarted on codesearch to correct "too many open files" error14:06
openstackstatusShrews: finished logging14:06
iurygregoryHi everyone, in zuulv2 we had the BASE env variable set but in Zuulv3 this changed?14:11
*** sdake has quit IRC14:11
*** janki has quit IRC14:12
*** janki has joined #openstack-infra14:12
*** dklyle has joined #openstack-infra14:13
*** dpawlik has joined #openstack-infra14:13
*** ykarel is now known as ykarel|away14:14
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088414:17
*** ykarel|away has quit IRC14:18
*** markvoelker has joined #openstack-infra14:18
*** ykarel|away has joined #openstack-infra14:19
*** sdake has joined #openstack-infra14:20
fricklerShrews: I've seen that occasionally, but always worked again after a reload/new search14:22
*** sdake has quit IRC14:23
Shrewsfrickler: it seemed to be affecting git operations by hound, according to the log. at any rate, seems ok for now14:25
*** asettle has joined #openstack-infra14:32
jangutterhas anyone got a magic spell for disk-image-builder that will assemble an EFI bootable whole-disk-image for centos7 ?14:32
*** sdake has joined #openstack-infra14:33
pabelangerjangutter: I think yolanda or ianw might know more14:34
jangutterpabelanger: thanks, for reference, I tried - centos7 block-device-gpt growroot dhcp-all-interfaces openssh-server cloud-init grub2 stable-interface-names bootloader14:35
jangutterpabelanger: a small blue imp appeared in a sulfurous haze, but nothing much else.14:36
pabelangerjangutter: http://git.openstack.org/cgit/openstack-infra/project-config/tree/nodepool/nb03.openstack.org.yaml#n63 is the nodepool config we are using for arm64 images, but only debuntu right now14:37
pabelangernot sure why we didn't do centos14:37
pabelangeror fedora14:37
jangutterpabelanger: python3 probably :-p14:37
jangutterpabelanger: are those booting baremetal nodes, or virtual?14:38
pabelangervms14:38
pabelangerbut require EFI, our only cloud that does right now14:39
jangutterpabelanger: ah, added wrinkle, I'm trying baremetal.14:39
jangutterpabelanger: ...again....14:40
pabelangerjangutter: maybe yolanda knows more in this case14:40
iurygregoryDoes anyone have any idea what var I can use instead of BASE when switching to Zuulv3? (http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/playbooks/devstack_gate_vars.yaml)14:40
*** dtantsur|brb is now known as dtantsur14:40
iurygregoryhttps://review.openstack.org/#/c/633010/18/ironicclient/tests/functional/hooks/post_test_hook.sh it seems to be empty when running with zuulv314:41
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088414:46
pabelangeriurygregory: you need to use environment: '{{ zuul | zuul_legacy_vars }}' to access legacy variables14:48
pabelangerI'm not sure what related BASE however14:48
*** sdake has quit IRC14:48
yolandahi so yes, i have composed bootable EFI images14:48
yolandain the context of TripleO but it will be useful as a sample anyway, let me check links14:49
iurygregorypabelanger, even if i'm moving a job to zuulv3 I can use zuul_legacy_vars?14:49
pabelangeriurygregory: you could, but likely shouldn't.  Depending where base is defined, you might need to template your hook via jinja214:50
yolandajangutter, pabelanger , see https://github.com/openstack/tripleo-common/blob/master/image-yaml/overcloud-hardened-images-uefi.yaml and https://github.com/openstack/tripleo-common/blob/master/image-yaml/overcloud-hardened-images-uefi-centos7.yaml14:50
pabelangereg: {{ base }}14:50
yolandathe trick is to add this element http://git.openstack.org/cgit/openstack/tripleo-image-elements/tree/elements/overcloud-secure-uefi14:50
iurygregorypabelanger, its possible to create an env var in the job definition?14:51
*** sdake has joined #openstack-infra14:51
pabelangeriurygregory: yes: https://zuul-ci.org/docs/zuul/user/jobs.html#variables14:51
*** markvoelker has quit IRC14:52
jangutteryolanda: is there a minimum DIB version required for that element?14:52
iurygregorypabelanger, going to try to set BASE in the job definition to see if it works14:53
jangutteryolanda: (I'm running 2.18.0)14:53
pabelangeriurygregory: you'll still need to pass the variable into the environment field for the shell task.  I want to say there is a better way to do this, but I'm currently not sure. I'd also ask in #openstack-qa, they've done a good job rewriting jobs14:54
jangutteryolanda: looking at my spell, the delta is block-device-default.yaml I think.14:54
iurygregorypabelanger, tks for the help i will try to ask them too =)14:54
*** pcaruana has quit IRC14:57
yolandajangutter, it's been there for some time. I don't remember the exact tag when it came, but anything from the last 6 months will be ok14:58
yolandain order to make uefi work, you need to define an special partition layout14:58
*** munimeha1 has joined #openstack-infra14:59
*** ekultails has joined #openstack-infra15:00
*** e0ne has quit IRC15:02
*** ccamacho has joined #openstack-infra15:03
*** sdake has quit IRC15:04
jangutteryolanda: ahhh, so block-device-efi is close, but not close enough.15:05
logan-jangutter: I see above you were trying block-device-gpt, but wouldn't you rather use block-device-uefi instead?15:06
*** sdake has joined #openstack-infra15:06
jangutterlogan: huh... drat, why is it gpt and not efi?15:06
logan-block-device-efi I mean15:06
jangutterlogan-: thanks, that's my n-th PEBCAK moment of the day.15:07
logan-block-device-gpt creates a GPT labelled root disk with a BSP instead of an ESP15:07
logan-np :) hope it works for you15:07
jangutterlogan:15:08
jangutterlogan-: That makes sense, you need Extra Sensory Perception rather than just Baffling Stupid Problems.15:08
*** eharney has joined #openstack-infra15:09
openstackgerritMerged openstack-infra/project-config master: Add an openstack/auto-scaling-sig repository  https://review.openstack.org/63712515:09
fungiiurygregory: you might have more luck asking in #openstack-qa if you're trying to use the zuul v3 job definition from the devstack repo, but looks like it uses devstack_base_dir as the ansible variable name for that: http://git.openstack.org/cgit/openstack-dev/devstack/tree/roles/run-devstack/defaults/main.yaml15:09
fungi(i strongly recommend against altering "legacy" devstack-gate jobs, they're not really maintainable)15:10
iurygregoryfungi, yeah i saw that, trying to figure out if its possible to call that variable in the shell15:10
*** anteaya has joined #openstack-infra15:10
fungiiurygregory: you can use jinja2 template variable expansion in shell or command tasks in your ansible15:11
fungior i think you can pass shell and/or command tasks an environment array15:12
toskyfungi: I suggested the usage of tox_environment15:12
iurygregoryfungi, ack 1st I'm going to try with tox_environment to see if goes15:12
fungiyeah, if you're trying to call it from within the scope of something tox is running, then that ought to be the place to plumb it in15:12
*** bhavikdbavishi has joined #openstack-infra15:12
*** anteaya has left #openstack-infra15:13
*** anteaya has joined #openstack-infra15:13
*** dpawlik has quit IRC15:13
*** e0ne has joined #openstack-infra15:13
*** anteaya has quit IRC15:17
*** openstackgerrit has quit IRC15:28
jangutterpabelanger, yolanda, logal-: switching block-device-gpt with block-device-efi made it go further (it's now failing to mount /sysroot). I'm pretty sure I can debug it further, thanks!15:32
*** jamesmcarthur has joined #openstack-infra15:37
yolandajangutter, good!15:41
yolandalet me know if you need some help15:41
jangutteryolanda: thanks very much!15:42
*** pcaruana has joined #openstack-infra15:42
priteaujangutter: I've built CentOS7 EFI whole disk images for Ironic before. I use these options:15:43
priteau-p biosdevname,gdisk block-device-efi disable-selinux enable-serial-console grub2 vm15:43
*** pgaxatte has quit IRC15:43
jangutteryolanda: looking at blkid on the node, it looks like there are two partitions with LABEL="cloudimg-rootfs".... I'm not exactly sure where that gets rendered.15:44
jangutterpriteau: yeah, block-device-gpt is _not_ block-device-efi15:45
priteauMake sure you use the "vm" element as well, to create a whole disk image15:45
*** armax has joined #openstack-infra15:46
*** chandankumar is now known as raukadah15:47
*** markvoelker has joined #openstack-infra15:48
*** sdake has quit IRC15:49
fungijangutter: also, while i find this discussion actually riveting and sort of on-topic for here, we do also have a dedicated #openstack-dib channel which might reach even more of the people you want for such future dib-related troubleshooting15:50
*** udesale has quit IRC15:50
jangutterfungi, PEBCAK n+1 moment of the day.15:50
jangutterfungi, thanks, I did not even know that existed,15:51
fungino worries, not complaining or anything, just thought you might not know it's there15:51
jangutterfungi: in fact next time I shall try harder to find the IRC channel for a particular tool.15:52
*** luizbag has quit IRC15:53
clarkbfrickler: we can test that reload replication command against review-dev and try it on production if it works on dev15:55
fungijangutter: we have https://wiki.openstack.org/wiki/IRC as an index, though no guarantee it's complete15:55
clarkbttx: I think we've decided to have zuul serve the static assets for simplicity of deployment, then if we need to we can have apache cache them on the way through15:55
clarkbttx: what are the 404s on direct links?15:56
fungithough also the rewrite rules in there look basically the same to me as well15:56
ttxclarkb: https://storyboard.openstack.org/#!/story/200498715:58
*** openstackgerrit has joined #openstack-infra15:59
*** xek has quit IRC15:59
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088415:59
ttxsee example reproduction there15:59
fungiour t/(.*) rule is first instead of last and slightly more greedy15:59
*** xek has joined #openstack-infra15:59
openstackgerritRico Lin proposed openstack-infra/irc-meetings master: Change Heat meeting time  https://review.openstack.org/64106215:59
fungittx: note that when looking at that erb template, it's the <% else -%> block which is relevant to the zuul.opendev.org vhost16:00
*** sdake has joined #openstack-infra16:00
ttxyep, I understood that much16:00
ttxwas just wondering (1) why there were small differences in the Rewrite set and (2) if those small differences could actually trigger that issue16:01
clarkblet me grab the actual file content as that might be easier to read16:02
fungialready doing it16:02
mordredfungi, clarkb, ttx: that direct link works for me16:02
fungittx: clarkb: this is how it gets expanded on the server: http://paste.openstack.org/show/747292/16:03
clarkbfungi beat me to it16:03
fungimordred: it has something to do with having to browse to a different url first and then it works16:03
clarkbmordred: ya it works for me too so I guess I've hit the different url first?16:04
mordredbut - I can confirm that refreshing the status page for the openstack tenant fails16:04
fungiright, that's the easier way to reproduce16:04
*** ricolin has quit IRC16:04
*** raukadah is now known as chandankumar16:04
*** bhavikdbavishi has quit IRC16:05
mordredif we're deferring to the underlying zuul-web process, do we need any of those other than the ws:// one?16:06
clarkbREQUEST_URI == REQUEST_FILENAME says apache docs16:07
clarkbmordred: no, I don't think so, but we seem to not be deferring currently or atetmpting to not defer16:07
mordredclarkb: nod. maybe it's worth shifting our approach to deferring?16:08
mordredbut also - don't listen to me - my brain is not working great this morning16:08
ttxbasically that bug prevents from linking to one specific tenant status page16:09
ttxwhich kind of sucks link-wise16:09
clarkbwe lack RewriteRule t/(.*)/(.*).html(.*) /t/$1/$2$3 [R=301,L,NE] ?16:10
mordredttx: yeah. totally. it's definitely an important bug to fix16:10
clarkbwhich is the backward compatible rewrite, maybe its less about just backward compat and necessary?16:10
*** bhavikdbavishi has joined #openstack-infra16:10
clarkbthough I guess there is no .html to match here16:11
jangutterpabelanger: you'll be glad to know it's not necessary to hack nodepool to do the "triple-image" thing for baremetal UEFI boot.16:14
openstackgerritDoug Wiegley proposed openstack-infra/elastic-recheck master: Add query for 1818696, weird qos port delete issue  https://review.openstack.org/64106716:14
jangutterpabelanger: lets hope that part of Cloud history never troubles us again.16:14
fungii think that was mostly an aws-ism which got cargo-culted into unnecessary places16:15
clarkbmordred: fungi /api/tenant/openstack/status HTTP/1.1" 200 171639 "https://zuul.opendev.org/t/openstack/status" is a working request path16:15
clarkb/t/openstack/status HTTP/1.1" 404 726 "https://storyboard.openstack.org/" is the failing16:15
openstackgerritJames E. Blair proposed opendev/base-jobs master: Revert "Revert "Save registry logs""  https://review.openstack.org/64056116:15
mordredclarkb: /t/openstack/status  should be being rewritten to serve index.html16:16
mordredbut that's apparently happening16:16
clarkbRewriteRule ^/t/(.*) /$1 [L] is our first rule16:17
clarkbso that becomes /openstack/status16:17
clarkband the [L] means stop processing?16:17
clarkbalso that rule isn't in our docs16:17
mordredI think that rule might be a thing that should go away16:18
clarkbya that may be our fix16:18
clarkbhttps://git.openstack.org/cgit/openstack-infra/puppet-zuul/tree/templates/zuulv3.vhost.erb#n48 is where that comes from16:18
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Fix project group view checkboxes  https://review.openstack.org/64059016:21
*** markvoelker has quit IRC16:22
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Show in-progress indicator when opening a board  https://review.openstack.org/64059316:23
openstackgerritDoug Wiegley proposed openstack-infra/elastic-recheck master: Add query for 1818697, fullstack qos bug  https://review.openstack.org/64107116:26
mordredinfra-root: re last night's mariadb restart/tc.log thing, Shrews found https://jira.mariadb.org/browse/MDEV-16442 which I have commented on. best we can tell, tc.log is useful for us, so updating $something to just always delete it during restarts is probably the safest action we can take, amusingly enough16:27
Shrewss/useful/useless/16:27
corvusShrews: thank you i was struggling with that :)16:27
Shrews:)16:28
clarkbmordred: is or isn't useful?16:28
mordredisn't16:28
mordredlisten to Shrews16:28
clarkbah ya sorry early still16:28
corvusShrews, mordred: maybe we should change the docker command?16:29
*** anteaya has joined #openstack-infra16:29
*** pcrews has joined #openstack-infra16:29
clarkbmordred: going back to zuul tenant urls. is t/openstack/status a valid zuul url if we pass it through to index.html (and remove the t/* rewrite?16:29
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Delete tc.log file when we start mariadb  https://review.openstack.org/64107316:31
mordredcorvus: ^^ do you think something liek that would work?16:31
mordredcorvus: or do you think instead we should change the command on the mariadb entry to be "rm -f /var/lib/mysql/tc.log && whatever-the-command-was-before"16:32
Shrewsmordred: any chance mariadb upgraded itself recently?16:32
Shrewsi ask b/c of https://github.com/docker-library/mariadb/issues/20116:32
clarkbShrews: our docker compose was updated to do a pull before up16:33
corvusShrews, clarkb: likely so, this will prevent further upgrades: https://review.openstack.org/64094916:33
corvusmordred: i'm looking into your change16:34
mordredcorvus: I'm not sure if it would get re-run if only the mariadb image updated16:34
mordredcorvus: oh - maybe it should use the mariadb image instead of ubuntu16:34
openstackgerritMerged openstack-infra/system-config master: Pin mariadb to 10.4  https://review.openstack.org/64094916:34
mordredso that it would be clear to docker-compose that it needs to run after pull?16:35
corvusmordred: interesting supposition.  that *seems* like it should work.  probably worth a test.16:35
mordredlemme push an update16:36
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Delete tc.log file when we start mariadb  https://review.openstack.org/64107316:36
corvusmordred: the other thing i was looking into is whether depends_on is effective.  it waits until the service is "started", which, given that it is a single command, should mean that the rm command is run before mariadb is started.16:36
*** quiquell is now known as quiquell|off16:37
corvus(depends_on is largely ineffective for long running services because docker has no idea when they are ready, only when they are started.  but "started" is probably good enough for a single command.)16:37
corvusthough, since all docker commands are run in the foreground, i suppose it's still technically possible to be a little racy16:38
fungiis this related to the zuul-preview.o.o outate?16:38
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088416:38
mordredcorvus: yeah - I was actually copy-pasting that from the zuul quick-start16:38
corvusmordred: all in all, it may be better to just change the startup command?16:38
corvusat least there are fewer moving parts that way16:38
corvusfungi: the what?16:39
fungithe container deployment errors/mariadb issue mordred mentioned overnight16:39
* fungi rereads16:39
mordredcorvus: indeed. the mariadb image has ENTRYPOINT docker-entrypoint.sh and mysqld as CMD16:39
openstackgerritClark Boylan proposed openstack-infra/puppet-zuul master: Don't rewrite t/ prefix on global zuul vhost  https://review.openstack.org/64107716:39
corvusfungi: mariadb is used for gitea.  what's a zuul-preview outate?  is it an update or outage?16:39
clarkbmordred: fungi ttx ^ I think that may do it16:39
fungioutage, yes sorry16:40
corvusfungi: i didn't know there was one.  where can i learn more?16:40
fungiyeah, rereading scrollback i guess he only said gitea docker issues16:40
fungiand suddenly the zuul-preview url i've been trying to load for minutes spontaneously popped up in my browser16:40
*** janki has quit IRC16:41
fungiso maybe it was a networking issue on my end16:41
*** ramishra has quit IRC16:41
corvusfungi: well, there is an issue with zuul-preview error handling16:41
*** kopecmartin is now known as kopecmartin|off16:41
*** janki has joined #openstack-infra16:41
*** auristor has quit IRC16:42
fungiahh, okay. for some reason trying to request http://site.7fe187f62d4a4f169c094fb1ebe4e41c.openstack.zuul-preview.opendev.org/ takes minutes to get a response16:43
openstackgerritJames E. Blair proposed openstack-infra/zuul-preview master: Return NULL on all errors  https://review.openstack.org/64107816:44
corvusfungi, mordred: ^ i think that will help16:44
corvusit's currently stuck in rewrite loops for /Error/Error/Error etc16:45
*** quiquell|off is now known as quiquell16:45
*** helenfm has quit IRC16:45
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Override mariadb command to delete tc.log  https://review.openstack.org/64107916:45
*** jpena is now known as jpena|brb16:46
mordredcorvus: how's that look?16:46
corvusmordred: sounds plausible :)16:47
mordredclarkb: lgtm16:47
mordredcorvus: yay!16:47
mordredcorvus: I'm pretty much only looking for plausible today16:47
fungioof, good catch!16:47
clarkbfrickler: we have to set https://review.openstack.org/Documentation/config-gerrit.html#plugins.allowRemoteAdmin to true to allow that reload command to work16:47
clarkbfrickler: it appears to be set to false so we'll need a restart to either get that in or to update the replication config :/16:48
*** roman_g has quit IRC16:51
openstackgerritJames E. Blair proposed opendev/base-jobs master: Correct provides/requires on buildset-registry job  https://review.openstack.org/64108316:53
*** roman_g has joined #openstack-infra16:53
fungiclarkb: frickler: though it's worth discussing whether we want to turn that on in the needed restart16:54
*** psachin has quit IRC16:54
clarkbfungi: ya the default being false implies that maybe there are reasons to not flip it to true :)16:55
*** ccamacho has quit IRC16:55
clarkbfungi: looks like you can install and uninstall as well as enable/disable plugins remotely in addition to simply reloading an existing plugin16:55
fungiyeah, reading the description there it seems maybe more dangerous, but then again our gerrit admins can already do those things just less seamlessly16:56
*** auristor has joined #openstack-infra16:56
fungiso maybe having it disabled is more a guard against a compromised gerrit admin account or some buggy codepath giving users admin privs16:56
fungi(in our case)16:56
*** jamesmcarthur has quit IRC16:57
corvusclarkb, fungi, mordred: can you review some base-jobs changes required for me to continue testing buildset registries: https://review.openstack.org/640561 and https://review.openstack.org/64108316:58
clarkbfungi: I'd be happy to leave it disabled for now and stick with the restart we know16:58
clarkbcorvus: yes I can take a look16:58
corvusmordred, clarkb, fungi, frickler: i think we're ready to restart gerrit, shall i do that now?16:59
fungicorvus: so the fix for the revert-revert was to remove the promote playbook?16:59
*** quiquell is now known as quiquell|off17:00
corvusfungi: no, sorry, there's a rebase in between those 2 patchsets too17:00
*** wolverineav has joined #openstack-infra17:00
fungicorvus: gerrit restart sounds fine to me though we should give #openstack-release a heads up about replication delays to git.o.o (maybe longer ones than in past restarts even)17:00
corvusfungi: it's splitting the post.yaml playbook into two plays -- it was only running on localhost, but needs to run on the registry host17:00
clarkbcorvus: yes, the config has been applied (I just checked) and I'm satisfied there isn't a short cut in the form of a replication config reload as above. Restart would be the next step and I'm happy for someone else to do it while I review those changes and catch up on email/irc17:01
clarkbhosts: all being the important bit in that playbook I guess17:02
fungiright, see it17:03
dtantsurhi folks! is it possible to delete some files from https://tarballs.openstack.org/ironic-python-agent/tinyipa/files/ manually? the files without a branch are not correct17:03
fungidtantsur: how did they get there? was it due to a problem which has since been corrected?17:04
corvusfungi: -release notified and acked17:04
dtantsurfungi: yep, we had an issue in our post job. it has just been fixed17:04
clarkbcorvus: does https://review.openstack.org/#/c/641083/1/zuul.yaml create a loop that can't be satisfied?17:04
*** wolverineav has quit IRC17:04
dtantsurfungi: fix https://review.openstack.org/64100717:05
corvusclarkb: remember that provides/requires is for sharing data between buildsets.  come to think of it "uses" might have been better than "requires".17:06
corvusclarkb: so if there are no dependent changes which provide docker-images, that's okay.17:06
clarkboh right. I keep thinking within a buildset17:06
*** jamesmcarthur has joined #openstack-infra17:06
clarkb(in my defense the dep trees have always been within a buildset when it comes to the job level :) )17:06
corvusclarkb: it just says "if there are dependent changes which provider docker-images, then please make sure i have them because i require [can use] them"17:06
clarkbcorvus: yup17:07
corvusclarkb: yep, it's not "automatically generate a job graph based on these expressions".  we may want to figure out how to convey that, because that's certainly a reasonable assumption to make.17:08
corvusstatus notice Gerrit is being restarted for a configuration change, it will be briefly offline.17:08
corvushow's that ^?17:08
openstackgerritMerged openstack-infra/zuul-preview master: Return NULL on all errors  https://review.openstack.org/64107817:08
clarkbcorvus: lgtm17:08
corvus#status notice Gerrit is being restarted for a configuration change, it will be briefly offline.17:09
openstackstatuscorvus: sending notice17:09
*** xek_ has joined #openstack-infra17:10
-openstackstatus- NOTICE: Gerrit is being restarted for a configuration change, it will be briefly offline.17:10
corvusrestarting now17:11
*** bhavikdbavishi has quit IRC17:12
*** jpena|brb is now known as jpena17:12
*** xek has quit IRC17:12
openstackstatuscorvus: finished sending notice17:12
*** bhavikdbavishi has joined #openstack-infra17:12
fungidtantsur: does that look better?17:12
*** chandankumar is now known as raukadah17:12
*** armax has quit IRC17:13
corvus[2019-03-05 17:13:30,742] [main] INFO  org.eclipse.jetty.util.log : Logging initialized @102594ms17:13
corvusthat's a lot of milliseconds :)17:13
*** jamesmcarthur has quit IRC17:14
corvusok, gerrit seems to be back up17:14
corvus  37794 tasks17:14
openstackgerritMerged opendev/base-jobs master: Revert "Revert "Save registry logs""  https://review.openstack.org/64056117:14
fungithanks for the restart corvus!17:14
*** jistr|sick is now known as jistr17:14
dtantsurfungi: yes, thank you!17:15
fungi#static log manually removed branchless artifacts from /srv/static/tarballs/ironic-python-agent/tinyipa/files/ on static.o.o17:15
fungidtantsur: let us know if you need any other help17:15
corvusnaturally, we forgot to accept the hostkeys17:15
clarkbcorvus: ah of course17:15
corvusfixing that now17:15
corvuser...17:17
corvusdoes that require another restart17:17
corvusi think it might -- i think i noticed that in my testing yesterday17:17
corvusfungi, clarkb: shall i restart again?17:18
clarkbif that is necessary we may as well take the hit now17:18
fungiyeah, better now than later i guess17:18
clarkb(its cheaper after the first one than waiting for tomorrow)17:18
fungisorry i didn't think to check that either :/17:18
*** e0ne has quit IRC17:18
corvusi *did* this in my local testing yesterday and forgot about it today. :(17:19
*** markvoelker has joined #openstack-infra17:19
corvusback up again17:19
fungiwe've got a couple commits queued in release-post, not sure if they'll care about a gerrit blip though17:20
fungioh, and they just vanished anyway17:20
corvuswhy is it still rejecting the host key?17:20
fungiis the error in the gerrit error.log?17:21
corvusreplication17:21
corvusorg.eclipse.jgit.errors.TransportException: ssh://git@gitea08.opendev.org:222/openstack-attic/image-api.git: reject HostKey: gitea08.opendev.org17:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-preview master: Add perf testing framework  https://review.openstack.org/63679217:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-preview master: Reimplement in Rust  https://review.openstack.org/63679317:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-preview master: Remove C++ version  https://review.openstack.org/63679417:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-preview master: Use rust:slim base image  https://review.openstack.org/63679517:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-preview master: Use slice matching for hostname unpacking  https://review.openstack.org/63679617:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-preview master: Port in changes from the c++ version  https://review.openstack.org/64108917:21
*** ykarel|away has quit IRC17:21
corvuswrong key type?17:22
clarkbcorvus: or port?17:22
corvuslooks like it did ecdsa17:22
clarkbdid you accept for port 22 not 222?17:22
corvusno i ssh'd to 22217:22
clarkbk17:22
fungihttp://paste.openstack.org/show/747298/17:22
*** yamamoto_ has quit IRC17:22
fungii dunno what the "key_load_public: invalid format" is about17:23
clarkbfungi: it means ssh doesn't understand your public key. We may just have the private key data there17:23
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Upgrade all dev servers to puppet 4  https://review.openstack.org/63039117:24
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Upgrade some servers to puppet 4  https://review.openstack.org/63472617:24
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Upgrade git01.openstack.org to puppet 4  https://review.openstack.org/63472717:24
*** yamamoto has joined #openstack-infra17:24
fungiwe have a ~gerrit2/.ssh/id_rsa.pub17:24
*** yamamoto has quit IRC17:24
clarkbthere goes my idea then17:25
fungibut it doesn't look like an ssh public key to me17:25
*** yamamoto has joined #openstack-infra17:25
* clarkb pops out real quick for breakfast17:25
fungilooks like it's missing the ssh-rsa prefix17:25
openstackgerritMerged openstack-infra/openstackid-resources master: Fixed cloud assets config  https://review.openstack.org/63979217:26
corvusso does anyone have handy the command to tell ssh to use the rsa host key?17:26
clarkbcorvus: ssh-keyscan -t rsa iirc17:26
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Revert "Revert "Update API code to work with Presentation Moderators collection (+N)""  https://review.openstack.org/64109117:26
clarkbfungi: puppet authorized keys you set the key type in the resource not the actual data. I wonder if we reuse that data value without being part of that authorized keys resources and thus lose the prefix17:27
fungii think we need to do something like -o HostKeyAlgorithms=ssh-rsa-cert-v01@openssh.com17:27
*** jamesmcarthur has joined #openstack-infra17:27
*** rkukura has quit IRC17:27
*** panda|ruck|flu is now known as panda|ruck|off17:27
fungissh -p 222 -o HostKeyAlgorithms=rsa-sha2-512,rsa-sha2-256,ssh-rsa git@gitea08.opendev.org17:28
fungicorvus: ^17:28
corvusfungi, clarkb: i think i got it with the keyscan method17:28
fungik17:28
corvusssh-keyscan -t rsa -p 222 gitea08.opendev.org  >> known_hosts17:28
corvusand it looks like a restart is not required17:28
fungigood to know17:28
corvusreplication to gitea is happening now17:28
*** yamamoto has quit IRC17:29
corvusso now we could either restart again to restart replication from the top, or leave this running and kick off another full replication to the gitea servers when this is done17:29
fungiso we accepted the ecdsa hostkey instead i guess?17:29
fungiand gerrit has broken ecdsa support17:29
corvusyep17:29
corvusi think i lean toward the second option, even though i think it will invalidate my bet with mordred on the replication time17:30
mordredcorvus: I vote for the choice that maximizes my winning chances17:31
corvuslooks like content is showing up in https://opendev.org/explore/repos17:31
clarkbgerrit retries indefinitely to replicate17:31
clarkbor at least it did when github rejected nova specs17:31
corvusclarkb: i do see retries at the end of the queue, so we may be covered.17:32
corvusftr, we're currently at   35380 tasks17:32
*** bhavikdbavishi1 has joined #openstack-infra17:32
*** jpich has quit IRC17:33
*** bhavikdbavishi has quit IRC17:33
*** bhavikdbavishi1 is now known as bhavikdbavishi17:33
mordredhttps://opendev.org/openstack-infra/puppet-zuul exists17:33
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Revert "Revert "Update API code to work with Presentation Moderators collection (+N)""  https://review.openstack.org/64109117:33
*** aojea has quit IRC17:34
*** sdake has quit IRC17:35
pabelangermordred: woah!17:37
openstackgerritMerged openstack-infra/puppet-zuul master: Don't rewrite t/ prefix on global zuul vhost  https://review.openstack.org/64107717:39
*** xek_ has quit IRC17:42
*** xek_ has joined #openstack-infra17:42
corvusfungi: can you +3 https://review.openstack.org/641083 ?17:43
corvuswhen that lands, i can get back to testing with untrusted projects :)17:44
fungiahh, yep was just catching up on what clarkb asked about provides/requires17:44
corvusclarkb, mordred: i was just looking at this failure, which happened twice: http://logs.openstack.org/00/640900/1/check/system-config-run-gitea/d38ca02/hosts/bridge.openstack.org/ara-report/result/a306f3b9-1bf6-4255-b62f-2f8ee76972e3/17:45
mordredcorvus: aroo?17:47
corvusand i had a terrible thought -- what if docker uses the mirrors in parallel?  so that it tried to pull mariadb from the first mirrer and gitea-web from the second (which failed so it rolled over to upstream dockerhub)17:47
corvus(the earlier change to get the registry logs should help confirm that, but we haven't run with that yet)17:48
clarkbthat would be sad17:48
corvusthough, in this case, the first mirror should have nothing... so it'd have to be "mariadb: use the first, fail, use the second" followed by "gitea: use the first, fail, skip the second because it's un use, use upstream"17:49
*** agopi has joined #openstack-infra17:50
*** rkukura has joined #openstack-infra17:51
*** markvoelker has quit IRC17:51
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Show in-progress indicator when opening a board  https://review.openstack.org/64059317:52
fungia really hacky way to prevent that might be to override hostname lookups17:52
*** kjackal has quit IRC17:52
*** rpittau is now known as rpittau|afk17:52
corvusfungi: i don't think we can do that17:52
corvusthe only behavior we can control is the registry itself, we can't impose requirements on the system under test17:53
fungioh, good point17:53
fungiat least not without also controlling the recursive resolver it uses17:53
*** gyee has joined #openstack-infra17:54
corvusthe docker source code is impossible to follow, so i have no idea where this is actually used17:55
clarkbI asked gitea for cinder commit listing via UI. It isn't responding very quickly, but does eventually erspond. Somethnig we may want to look at (possible we just need to use the service so that caches are built up)17:56
clarkbalso cinder seems to default ot stable/ocata branch and not master17:56
fungior may just be busy handling the replication flood still17:56
smcginnisI'm really liking this interface over the cgit one.17:57
openstackgerritMerged opendev/base-jobs master: Correct provides/requires on buildset-registry job  https://review.openstack.org/64108317:57
clarkbfungi: ya17:57
smcginnisclarkb: There's nothing under openstack/cinder master, so maybe it defaulted to ocata since that's the latest it has?17:57
clarkbsmcginnis: oh maybe? possible that this is lag in replication and it doing its best17:58
fungiyeah, i wouldn't read much into any observed behaviors until replication catches up17:59
corvusespecially since it's replicating the "c" repos now :)17:59
clarkbinfra-root I'm going to add those RO afs volumes to afs01.dfw and afs02.dfw now17:59
clarkbI'll start with project.* then before I do the opensuse addsite I'll disable the mirroring cron for that site via holding the lock18:02
*** ginopc has quit IRC18:02
clarkbI don't want that one to try and replicate via the normal cron process18:02
clarkbThe project.* volumes shouldn't be replicated by anything and if they are they are small18:02
*** agopi has quit IRC18:04
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Update API code to work with Presentation Moderators collection (+N)  https://review.openstack.org/64109118:06
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul Web: add /api/user/actions endpoint  https://review.openstack.org/64109918:07
dtantsurfolks, is there something we can urgently do when a third party CI goes a bit crazy like in https://review.openstack.org/#/c/639050/ ?18:11
dtantsurI doubt the contributors are online at this hour..18:12
openstackgerritClark Boylan proposed openstack-infra/system-config master: vos release project website volumes  https://review.openstack.org/64110118:12
clarkbinfra-root ^ that will ensure that the RO volumes for zuul-ci and docs.starlingx.io are releases regularly. Note that we appear to already use the RO path in our apache docroot config so I don't think I need to update anything there18:13
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul Web: add /api/user/actions endpoint  https://review.openstack.org/64109918:13
clarkbdtantsur: we can disable their account18:13
dtantsurclarkb: lemme try re-upload the patch before we resort to such measures.18:14
clarkbdtantsur: ok let us know if ironic would like us to do that and we'll get it done18:14
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Fix project group view checkboxes  https://review.openstack.org/64059018:14
dtantsurclarkb: OUCH, it became even worse: https://review.openstack.org/#/c/639050/. we have to ask you for help..18:16
clarkbdtantsur: ok it will be a few minutes to figure out what that user is in the db then we can disable it18:17
dtantsurthanks!18:17
clarkbunless I can do it by full name18:18
clarkbfungi: ^ do you know18:18
fungii'll look18:18
*** e0ne has joined #openstack-infra18:19
fungi2980018:19
fungii've just set it inactive18:20
clarkb2980018:20
clarkbcool you found what I found18:20
fungiyeah, you can't set-account using the full (display) name18:21
fungitried on a whim18:21
dtantsurlooks quiet now, thanks a lot!18:21
fungiyou can use e-mail address unless it's erroneously non-unique18:22
fungithat account also had an open connection to the ssh api (3ade947f) which i've closed18:23
clarkbfungi: do you want to send email to the accoutn email addr or should I?18:25
fungii can do it in a sec18:26
fungiusually we just reach out to the third-part-announce ml18:26
fungier, third-party-announce18:27
*** jamesdenton has quit IRC18:27
clarkbfungi: ya, though I wonder if at this point anyone pays attention to that18:28
fungilooks like they don't mention any irc contacts on https://wiki.openstack.org/wiki/ThirdPartySystems/Huawei_Ironic_CI18:28
clarkbI've taken the suse lock on mirror-update. Adding site to that volume then releasing it18:28
*** jamesdenton has joined #openstack-infra18:29
*** jpena is now known as jpena|off18:33
clarkbfwiw https://opendev.org/openstack/designate/branches/ shows default branch as stable/ocata18:34
*** xek_ has quit IRC18:34
clarkbso that might be a cleanup we have to do?18:34
*** xek has joined #openstack-infra18:35
*** mriedem has quit IRC18:37
fungii wonder how it decides that18:38
*** wolverineav has joined #openstack-infra18:38
*** mriedem has joined #openstack-infra18:39
clarkbgit push doesn't replicate HEAD18:40
clarkbpossible it infers it through first pushed branch and that is the order gerrit pushes in?18:40
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul Web: add /api/user/actions endpoint  https://review.openstack.org/64109918:44
clarkbhttps://github.com/go-gitea/gitea/pull/3715 I think that may be what is happening18:45
*** IvensZambrano has quit IRC18:46
clarkbhttps://github.com/go-gitea/gitea/issues/3913 an as of yet unfixed bug saying the behavior added in ^ is problematic for our git push --mirror use case18:46
*** markvoelker has joined #openstack-infra18:48
clarkbheh the swagger api docs don't metion repo settings url. I guess this i the thing that was mentioned before about how they don't have all the endpoints exposed there18:50
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Show in-progress indicator when opening a board  https://review.openstack.org/64059318:50
*** gfidente is now known as gfidente|afk18:51
corvuswe could do a one-time fix for all the repos easily enough18:52
corvusbut if we can't update the settings of all the projects after creation, then every new project (that has more than one branch) is likely to end up with the wrong branch18:52
*** jamesmcarthur has quit IRC18:52
clarkbcorvus: I wonder if we can git push the null commit to the project as part of project creation to make gitea think that master is already there18:55
*** yamamoto has joined #openstack-infra18:55
clarkbin theory you should always be able to fast forward from the null commit right?18:55
clarkbactually it doesn't matter bceause gerrit will force push and overwrite18:55
clarkbso that might be the hack null commit or not18:55
clarkbassuming that setting it explicitly as a setting does get overwritten on first push18:57
clarkbif not then setting it as a setting is likely better18:57
clarkbReminder: Our team meeting starts in ~3 minutes over in #openstack-meeting18:57
corvusmordred: can you look into that ^ ?18:58
corvusmordred: setting the default branch explicitly on creation, to see if that fixes the default branch problem18:58
mordredcorvus: I can look in to that19:01
mordredcorvus: is there a "good" way for me to log in to one of our giteas - just grab the root password from one of the configs?19:02
mordredcorvus: (I tried using try.gitea the other day to poke at default branch, but what I found there did not work on our gitea - so I'd like to poke at ours)19:02
*** electrofelix has quit IRC19:02
corvusmordred: i think that's it19:02
mordredkk19:02
*** yamamoto has quit IRC19:07
*** e0ne has quit IRC19:17
*** jamesmcarthur has joined #openstack-infra19:21
*** markvoelker has quit IRC19:21
dtantsurclarkb, fungi, have you reached out to huawei folks re that problem or should I?19:22
clarkbdtantsur: fungi said he would email our third-party-announce list (I've not checked if that was done yet as I'm running the infra meeting now)19:22
SotKmordred, diablo_rojo:  I'm around to chat about formsubmit swift stuff for a few hours after the meeting btw19:22
dtantsurI guess it won't hurt if I send them another one19:24
*** jamesmcarthur_ has joined #openstack-infra19:24
*** jamesmcarthur has quit IRC19:25
*** armax has joined #openstack-infra19:26
fungidtantsur: clarkb: i haven't yet, no19:26
*** dtantsur is now known as dtantsur|afk19:31
*** sdake has joined #openstack-infra19:33
*** e0ne has joined #openstack-infra19:37
*** kjackal has joined #openstack-infra19:37
*** wolverineav has quit IRC19:41
*** wolverineav has joined #openstack-infra19:42
*** wolverineav has quit IRC19:43
*** wolverineav has joined #openstack-infra19:43
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Revert "Revert "Set default branch for repo""  https://review.openstack.org/64110819:44
*** jento has joined #openstack-infra19:45
*** wolverineav has quit IRC19:45
*** wolverineav has joined #openstack-infra19:46
*** wolverineav has quit IRC19:46
*** wolverineav has joined #openstack-infra19:46
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Use auto_init to create a repo with a master branch  https://review.openstack.org/64111019:46
mordredclarkb, corvus: ^^ there's two options19:46
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Remove releases link from gitea  https://review.openstack.org/64111119:51
*** dave-mccowan has quit IRC19:55
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Revert "Revert "Set default branch for repo""  https://review.openstack.org/64110819:55
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Use auto_init to create a repo with a master branch  https://review.openstack.org/64111019:55
clarkbmordred: thanks I'll review that now then I need to find lunch19:59
*** dave-mccowan has joined #openstack-infra20:01
clarkbmordred: with auto_init do we still need to set the branch to master or is master implied in that case?20:02
fricklerclarkb: mordred: maybe we're not there yet, but how about setting up a gitea-dev instance where we could test such things? also for the issues/releases tabs20:02
corvusfrickler: we could also write a test job that exercises project creation20:03
*** xek has quit IRC20:03
*** xek has joined #openstack-infra20:03
clarkba setup to run a gitea docker compose locally might be better than long running dev instance?20:03
corvusi say "also", but i think i may actually mean "instead"20:03
clarkbIn theory the use of docker images makes that easier than if we had to bootstrap from scratch20:03
clarkbcorvus: ++20:04
corvusyeah.  i think 'run gitea locally in docker' to develop things, and 'run zuul jobs to test changes' is a good recipe20:04
corvuswe just don't have test coverage of project creation yet, only bootstrapping the service.20:05
mordredcorvus: yeah - seems like running the project creation role would give us some good coverage there20:05
fricklerhmm, yeah, testing locally should work20:05
mordredbecause the original 'set branch to master' threw a 404 and ansible was sad - so gate test would have totally shown an error20:06
*** dpawlik has joined #openstack-infra20:09
clarkbinfra-root can I get a quick review on https://review.openstack.org/#/c/641101/ as part of my afs prep work?20:11
*** eharney has quit IRC20:12
*** dpawlik has quit IRC20:16
*** sdake has quit IRC20:17
*** sdake has joined #openstack-infra20:19
*** markvoelker has joined #openstack-infra20:19
clarkbhttps://review.openstack.org/#/c/641077/1 merged to fix the zuul ui redirect thing but hasn't applied on zuul01 yet. Looking at puppet modules on zuul01 the module hasn't updated to that version but pupet has run several times since merging20:20
clarkbI'll have to track that down after lucnh if we don't already know why that is happening. Did we stop running the update repos playbook?20:20
*** whoami-rajat has quit IRC20:22
*** bhavikdbavishi has quit IRC20:23
*** wolverineav has quit IRC20:31
*** wolverineav has joined #openstack-infra20:32
*** wolverineav has quit IRC20:37
*** markvoelker has quit IRC20:52
corvusclarkb, mordred: oh, whew -- i *think* the docker registry mirror error from earlier may be due to leftover v1 manifests -- i'm digging into that theory now20:54
mordredcorvus: kk20:56
corvusclarkb, mordred: yes, the last gitea image changes merged ~50 minutes before the v1/v2 manifest fix merged, so they're still v1 images.  the next gitea image updates should fix that. so i'll go review the change to remove the relases tab :)20:57
mordredcorvus: woot!20:57
*** jamesmcarthur has joined #openstack-infra20:57
*** jamesmcarthur_ has quit IRC21:00
*** wolverineav has joined #openstack-infra21:02
*** eharney has joined #openstack-infra21:03
priteauThe Blazar gate was failing around 7 PM UTC because it is pulling keystoneauth1 3.13.0 which has a bug, even though 3.13.1 was released earlier today. Could this be caused by an out of date PyPI mirror?21:05
priteauNevermind, it may be due to upper-constraints.21:06
ttxclarkb: https://review.openstack.org/#/c/641077/ did fix https://storyboard.openstack.org/#!/story/200498721:07
clarkbttx: cool I was just about to look at why that wasn't applying but it appears that it just applied21:08
priteau^^^ keystoneauth1 bump here: https://review.openstack.org/#/c/641036/121:08
clarkbpriteau: we usually defer to requirements core on those but if smcginnis dhellmann prometheanfire or tonyb don't get to it soon I suppose we can approve it21:09
prometheanfirehi?21:09
smcginnisNeed to wait for the zuul jobs to complete yet.21:10
priteauI've just resubmitted a blazar patch with a Depends-On on it to check that it fixes the problem21:10
prometheanfireclarkb: priteau asking because of the release timeline?21:10
smcginnispriteau: I don't believe depends-on will work in this situation.21:10
clarkbsmcginnis: prometheanfire: these things are why I will defer to the requirements team generally :)21:10
*** pcaruana has quit IRC21:10
clarkb(I don't know what all the criteria are)21:10
* prometheanfire thinks non-client libs are frozen21:11
priteauprometheanfire: Asking because 3.13.0 has broken our gate jobs21:11
*** wolverineav has quit IRC21:11
* prometheanfire has been distracted21:11
prometheanfirepriteau: it's still early enough that it's probably fine21:11
tonybpriteau: Looks pretty straight forward just waiting on zuul21:11
clarkbprometheanfire: understood, no worries. I'm sure smcginnis or dhellmann will be able to handle it21:11
clarkboh and not tony :)21:11
clarkb*now21:11
fungidepends-on does usually handle proposed constraints changes for openstack's tox jobs21:11
tonybfungi: It certainly should :)21:12
smcginnisWasn't sure if the constraints line in tox.ini would interfere with that.21:12
fungithe role we use grabs the constraints file from zuul's checkout of the requirements repo, but yeah if the project does something funky in tox.ini to break that then ymmv21:12
clarkbinfra-root mirror.opensuse replication to afs02.dfw completed and I've removed my hold of that lock on mirror-update.o.o21:13
*** e0ne has quit IRC21:13
smcginnisHmm, I don't see 641036 in zuul.o.o21:13
tonybsmcginnis: what fungi said ;P21:13
clarkbmordred: is the and in your release template postfix?21:14
priteauThanks everyone, hopefully zuul won't take too long21:14
clarkbsmcginnis: possile that it tried to report back when we restarted gerrit today21:15
clarkbin that case may need a recheck21:15
gouthamrhello o/ I have a patch stuck in "Ready to Submit", https://review.openstack.org/#/c/638075/21:19
gouthamrhaven't seen that issue before, something wrong with the vote from zuul? ^21:20
clarkbgouthamr: the issue is the parent commit for that change is not the latest patchset on the parent change. Gerrit only allows the latest patchset to merge so a change depending on not the latest patchset cannot merge. The fix is to rebase21:22
clarkbgouthamr: Gerrit tries to indicate this to you via the orange dot on the Parent(s) entry on the change page21:22
dmsimardclarkb, gouthamr: the parent change actually hasn't merged yet21:22
dmsimardoh, yeah, it has21:23
dmsimardthe outdated patchset hasn't merged, the updated one has :p21:23
clarkbdmsimard: ya21:23
gouthamrclarkb: ah! since zuul always tests by rebasing, assumed this merge would work21:24
gouthamrclarkb dmsimard: thanks for the clarification, rebased now21:24
clarkbgouthamr: note this is entirely a gerrit thing not a zuul thing21:25
clarkbwe could potentially work around it if zuul pushed commits into the repos nstead of gerrit21:25
*** eernst has joined #openstack-infra21:25
clarkbfungi: can you sanity check https://review.openstack.org/#/c/630391/7/inventory/groups.yaml for wiki-dev? want to make sure that won't step on any toes21:25
*** jcoufal has quit IRC21:25
clarkbfungi: if that doesn't conflict with your work there I can approve21:26
*** e0ne has joined #openstack-infra21:26
*** kjackal has quit IRC21:27
fungiclarkb: it'll be fine21:28
clarkbfungi: tahnks for checking.21:28
fungii've booted wiki-dev02 but still need to propose it to the inventory21:28
fungiso it hasn't puppeted yet21:28
clarkbfwiw lists-dev and groups-dev won't match there because they don't have digited servers yet21:29
*** wolverineav has joined #openstack-infra21:29
*** janki has quit IRC21:29
clarkbI'll scribble a note to follow up on that21:29
*** e0ne has quit IRC21:30
clarkbcmurphy: ^ fyi on that21:31
*** kgiusti has quit IRC21:36
clarkbI'm actually curious how https://review.openstack.org/#/c/630391/7 manages to merge given that we fixed groups and groups-dev entries in futureparser to drop the digits21:38
clarkbI'll have to check the resulting git log and push a fixup21:38
*** eernst has quit IRC21:40
*** eernst_ has joined #openstack-infra21:40
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage user ssh keys from urls  https://review.openstack.org/60493221:44
clarkbfungi: ^ opendev.org urls. I'm researching the CAA RRs now21:45
*** eernst_ has quit IRC21:46
*** xek has quit IRC21:46
*** wolverineav has quit IRC21:47
*** wolverineav has joined #openstack-infra21:48
clarkbfungi: I'm not finding anything in requests/urllib3/pyOpenSSL docs about checking those records21:49
*** markvoelker has joined #openstack-infra21:49
clarkbthis may be something we have to test21:49
fungiclarkb: thanks, and yeah it may be they don't support caa checking yet21:49
*** wolverineav has quit IRC21:53
ianwclarkb: who would be responsible for transferring the signing domain acme-opendev.org to openstack/opendev?21:54
ianwi'm thinking right now getting graphite.opendev.org as the initial letsencrypt target21:54
ianwbut only with staging keys, until ready for full production21:54
openstackgerritMerged openstack-infra/system-config master: vos release project website volumes  https://review.openstack.org/64110121:54
openstackgerritMerged openstack-infra/system-config master: Remove releases link from gitea  https://review.openstack.org/64111121:54
ianw(i just mean getting the keys on the host ... wiring them up to apache will be a little separate puppet exercise)21:55
clarkbianw: the foundation uses a registrar manager service thing. jamesmcarthur is our interface with them typically.21:55
clarkbianw: does one of us already own that domain or do we also need to purchase it?21:56
clarkb(I can work with jamesmcarthur to get that going)21:56
clarkblet me ping him to look here21:56
jamesmcarthurhello21:56
clarkbohai21:56
*** eernst has joined #openstack-infra21:56
clarkbjamesmcarthur: so one of the things we'd like to simplify letsencrypt cert issuance authorization is a separate dns domain we can delegate to for that21:57
clarkbsomething like acme-opendev.org21:57
jamesmcarthursure21:57
clarkbianw has been driving a lot ofthis work so can probably articulate why this is useful better than I can but basically allows us to have a zone that doesn't get changes made to it reviewed and instead we automate that for cert issuance21:58
clarkbI'm guessing we want to manage that domain in the same way that we do the main one21:58
jamesmcarthurDoes someone already own acme-opendev.org?21:58
*** eernst has quit IRC21:58
fungiianw: any reason we can't just delegate from openstack.org to a subdomain-rooted zone?21:58
clarkbjamesmcarthur: fungi those are both questions I'll have to defer to ianw on21:58
openstackgerritMerged openstack-infra/system-config master: Upgrade all dev servers to puppet 4  https://review.openstack.org/63039121:58
fungier, i mean from opendev.org21:58
jamesmcarthurI was about to say :)21:58
ianwfungi: yeah i did mention it in https://specs.openstack.org/openstack-infra/infra-specs/specs/letsencrypt.html#opendev21:59
jamesmcarthuracme.opendev.org would already be under your control21:59
fungicouldn't we just put acme.opendev.org as the root of a separate zone not managed through code review and cname into there?21:59
ianwwe *can*, personally i think it's better to use a separate domain21:59
fungii'll reread the spec21:59
fungii think i missed the explanation for why a separate domain improves it, rather than just needing to be a separate zonefile21:59
ianwfor example, if we get dns-as-a-service, we can point the signing domain at it, and use the acme client to automatically update without any interference to opendev.org22:00
fungiwe can still do exactly that with a subdomain, and even more easily22:00
fungisince we don't have to interface with a registrar to update the delegation22:00
*** priteau has quit IRC22:01
fungithere's nothing special about how an acme.opendev.org zone would be served than an acme-opendev.org zone22:01
clarkbright we can set glue records in our gerrit reviewed zone then delegate that to whereever?22:02
fungino need for glue records, we just set ns records in opendev.org for the acme.opendev.org zone to the different nameservers22:02
ianwfungi: i'm not sure how that would work, isn't a dns-as-a-service going to want to own the opendev.org domain?22:03
fungiwhy would it?22:03
fungitell the dns-as-a-service it owns acme.opendev.org22:03
fungiwe delegate to it with ns records for acme in the opendev.org zone22:04
clarkbfungi: oh right they don't have to be in the same domain22:04
clarkbfungi: we'd also have to set up dnssec?22:05
clarkbbut that should all be doable22:05
jamesmcarthurYeah, I would stick with a sub-domain here if possible.22:05
dmsimardEh "Introducing Kraken, an Open Source Peer-to-Peer Docker Registry": https://eng.uber.com/introducing-kraken/22:07
ianwhrm, personally i still like the separation of powers of two domains, let me think some more22:08
fungiianw: it makes sense to have two domains if you want two completely different sets of people/software controlling them. if you trust and want the folks who control the opendev.org domain to also have control over acme.opendev.org then that works fine. systems you give access to update acme.opendev.org should have no way to break anything else in opendev.org outside that subdomain22:10
fungiif the concern was that folks controlling opendev.org should have no control over acme-opendev.org that's when a separate domain would also provide a separation of powers22:11
*** wolverineav has joined #openstack-infra22:12
*** eharney has quit IRC22:14
ianwfungi: ok, let me see if i can come up with something that seems like it will roll out a valid configuration using a subdomain22:14
fungiit ought to look identical other than containing an additional . right?22:22
*** markvoelker has quit IRC22:22
*** sdake has quit IRC22:28
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck master: Update query for bug 1818560  https://review.openstack.org/64114722:28
openstackbug 1818560 in OpenStack Compute (nova) "Nova test_report_client uses nova conf when starting placement intercept, causing missing config opts" [Critical,Fix released] https://launchpad.net/bugs/1818560 - Assigned to Eric Fried (efried)22:28
*** sdake has joined #openstack-infra22:32
*** sdake has quit IRC22:32
clarkbI'm keeping an eye for puppet 4 install/runs on review-dev22:35
clarkbI think the next run in a few minutes will be with puppet422:36
*** sdake has joined #openstack-infra22:37
clarkbhttp://paste.openstack.org/show/747305/ puppet4 on review dev22:39
clarkbthe unresolved dependencies for those puppet modules are something we should look into as is why did rc.d change for gerrit22:39
clarkbI'm going to check the other servers now since ^ shouldn't represent any emergencies22:39
*** jamesmcarthur has quit IRC22:41
clarkbhttp://paste.openstack.org/show/747306/ translate-dev had a thing chagne that I wouldn't have expected. Storyboard-dev is cleaner, http://paste.openstack.org/show/747307/. Both don't know what ::project_config_ref is22:43
clarkbsince ::project_config_ref is common to all three I'll start there22:43
clarkbcmurphy: ^ fyi there are things we should investigate and possibly clean up22:43
*** gfidente|afk has quit IRC22:44
*** sdake has quit IRC22:45
*** wolverineav has quit IRC22:47
openstackgerritClark Boylan proposed openstack-infra/puppet-project_config master: Check if project_config_ref is defined before using it  https://review.openstack.org/64115222:51
clarkbfix 1:22:51
*** sdake has joined #openstack-infra22:51
*** munimeha1 has quit IRC22:52
*** rlandy is now known as rlandy|bbl22:54
*** tkajinam has joined #openstack-infra22:54
openstackgerritIan Wienand proposed openstack-infra/zone-opendev.org master: Add acme.opendev.org  https://review.openstack.org/64115422:55
*** jamesmcarthur has joined #openstack-infra22:57
*** jamesmcarthur has quit IRC22:58
openstackgerritIan Wienand proposed openstack-infra/system-config master: Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115522:58
*** jamesmcarthur has joined #openstack-infra22:58
*** sdake has quit IRC23:02
openstackgerritClark Boylan proposed openstack-infra/system-config master: Install latest puppet-vcsrepo  https://review.openstack.org/64115623:02
clarkbfix 2:23:02
fungi"This message is to inform you that the host your cloud server 'graphite.openstack.org' resides on alerted our monitoring systems at 23:02 UTC."23:03
fungiianw: ^ now there's irony for you23:03
openstackgerritMerged openstack-infra/elastic-recheck master: Update query for bug 1818560  https://review.openstack.org/64114723:04
openstackbug 1818560 in OpenStack Compute (nova) "Nova test_report_client uses nova conf when starting placement intercept, causing missing config opts" [Critical,Fix released] https://launchpad.net/bugs/1818560 - Assigned to Eric Fried (efried)23:04
*** yamamoto has joined #openstack-infra23:05
ianwfungi: hrm, if it's messed it up in any way we can rm it23:08
openstackgerritClark Boylan proposed openstack-infra/system-config master: Bump puppet/staging to 1.0.1 to satisfy mysql module dep  https://review.openstack.org/64115723:08
clarkbfix 323:08
ianwall it's currently doing is rejecting random stats packets sent at it from externals :)23:08
jamesmcarthursorry ianw and fungi: I dropped off for a few. If y'all need assistance in the domain reg department, just hit me up.23:09
*** yamamoto has quit IRC23:10
ianwjamesmcarthur: ok, thanks, will proceed with subdomain for now, will ping if anything changes23:11
jamesmcarthurroger!23:12
fungithanks jamesmcarthur!23:13
clarkbpuppet-gerrit actually does say to create those rc.* files23:14
clarkbso its fine that it did, but I'm not sure why they weren't already there23:14
*** hwoarang has quit IRC23:14
clarkbprobably because S04gerrit is there instaed?23:14
clarkbon prod gerrit we have S04gerrit and S90gerrit with puppet323:16
clarkbwith review-dev we have only S04gerrit with puppet4. I wonder if puppet is cleaning the S90 values up23:16
clarkbbecause S04 supercedes23:16
fungiotherwise it could double-start i suppose23:17
clarkbI think the fix here is going to be dropping the explicit symlinks in puppet-gerrit and letting the service ersources manage that23:17
clarkbLooking at the wildfly change now23:17
*** hwoarang has joined #openstack-infra23:18
*** markvoelker has joined #openstack-infra23:19
*** wolverineav has joined #openstack-infra23:24
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115823:25
corvusmordred: ^23:25
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115823:26
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115823:27
mordredcorvus: \o/23:28
ianwfungi: hrm, testing Add acme.opendev.org  https://review.openstack.org/641154 is going to be a pain.  the roles clone from git repos, but don't know to use the zuul checkout23:29
*** wolverineav has quit IRC23:30
ianws/the roles/the dns roles/23:30
clarkbok the translate change is interesting. We are writing a config with username and password for wildfly. This change seems to have either added a user or modified its password? I'm looking at the file bucket now to see if I can tell23:30
corvusianw: i would think that you just don't want an acme zone file in that repo23:31
corvusianw: i think you'd just want to have le-ansible write that file, and, once that exists on the actual production server, add the NS records to the zone23:32
*** ekultails has quit IRC23:32
corvusianw: (otherwise, the system-config dns roles are going to fight with le-ansible over that file)23:32
ianwcorvus: my thinking is that /etc/named.conf needs to have a reference to at least a empty template to be valid23:33
*** sdake has joined #openstack-infra23:33
clarkbfile_line doesn't seem to use the clientbucket?23:33
ianwcorvus: in terms of fighting, the next change marks it as "unmanaged"23:33
ianw^ Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115523:33
corvusianw: well that changes things :)23:34
clarkbwell translate-dev01 is stable now and that config setting should be fine to update so I think I'll just ignore that one.23:34
clarkbNow to fix puppet-gerrit23:34
clarkboh hrm we don't actually have a gerrit service resource? then what is deleting S90gerrit23:35
*** derekh has quit IRC23:36
*** jamesmcarthur has quit IRC23:37
corvusianw: that seems like a reasonable implementation.  i'm not sure it's going to be worth it to build full integration testing of proposed data changes.  however, i think it's safe to land the zone addition in 641154 but without the NS records.  perhaps that's enough to satisfy the tests?23:37
clarkbok best guess is it is actually the puppet install that removes S90gerrit since I think we install puppet4 each time through23:37
clarkband that might trigger something in systemd that does a cleanup of redundant init things. Now my question is where does S04gerrit come from?23:38
*** wolverineav has joined #openstack-infra23:38
*** jamesmcarthur has joined #openstack-infra23:38
clarkbmy guess for that is systemd init.d support23:38
clarkbif you put an init script in /etc/init.d/ does systemd daemon-reload automatically enable that?23:38
clarkbfungi: ianw ^ you may know?23:38
fungii don't really know the first thing about systemd23:39
ianwhrm, my instinct says no but ... yeah ...23:39
*** jamesmcarthur has quit IRC23:40
ianwcorvus: yeah, i think having that file there would at least allow testinfra to check bind started (synatx ok) and that files are deployed23:41
ianwi might be able to do a run using the review git URLs somehow, i'll try that first23:42
corvusclarkb, fungi, mordred: good news everyone!  also some bad news23:43
fungiif futurama is to be believed, "good news everyone" is almost always followed by bad news23:44
corvusthe good news is that the buildset registry is working (now that the v2 manifests are in place on the gitea images, the apparent registry problem from earlier is gone, so that hypothesis was correct)23:44
corvusthe other good news is that the latest runs of a series of changes designed to show that the system is working with dependent changes also seem to be working, in that the sha of the image deployed in change #2 is the same sha previously built in change #123:45
corvusthe bad news is that in order to *really* show it was working, i also tried to break the image and deploy it, but apparently our testinfra tests of zuul-preview weren't able to detect that breakage, so they passed23:46
openstackgerritClark Boylan proposed openstack-infra/openstack-zuul-jobs master: Add puppet-vcsrepo as required project on puppet jobs  https://review.openstack.org/64116123:46
openstackgerritClark Boylan proposed openstack-infra/system-config master: Install latest puppet-vcsrepo  https://review.openstack.org/64115623:46
clarkbcorvus: progress at least23:47
mordredcorvus: oh. well - that's some lovely good news23:47
corvusit looks like if you tell docker to bind a port, it actually listens on that port, even if the container isn't23:48
clarkbinfra-root cmurphy https://review.openstack.org/#/q/owner:cboylan+topic:puppet-4+status:open is a set of changes to clean up the puppet-4 stuff I noticed23:48
clarkbcmurphy: I think we want to cleanup all of the puppet module dependencies before deploying more puppet-423:48
clarkbcmurphy: once those initial dep fixes go in I'll generate the tree and we can go through it and fix the rest23:49
clarkbas for Gerrit I think we should be cautious for now and just let it create the symlinks that will be cleaned up by something23:49
clarkband when we understand that better fix it properly23:49
clarkbalso I'm beginning to think we should puppet-4 things in batches like that so we can monitor and address unexpected behaviors23:50
*** markvoelker has quit IRC23:52
clarkbI'm going to do a systemctl daemon-reload once puppet puts S90gerrit back and see if that cleans up S90gerrit in favor os S04gerrit23:54
corvusclarkb, mordred: is it perhaps possible that EXPOSE is meaningless in host networking?23:55
clarkbcorvus: oh that could be it23:56
corvusokay, so i probably didn't break zuul-preview hard enough :)23:56
mordredcorvus: entirely likely23:56
fungitry harder!23:56
clarkbcorvus: the process running in the host network namespace is gonna bind to what it binds to regardless of docker network magic23:56
openstackgerritJames E. Blair proposed openstack-infra/zuul-preview master: DNM: break dockerfile  https://review.openstack.org/64090123:57
corvusclarkb, mordred, fungi: ^ think that'll break enough?23:57
clarkbcorvus: that should break when the container is started but not at build time I would expect23:57
clarkbso good enough if you want runtime breakage23:58
corvusyep, that's what i'm looking for23:58
corvusworking build, failing deploy23:58
clarkbthen ya that should break as expected (at least based on experiences trying to exec bash in an image with only sh)23:58
*** wolverineav has quit IRC23:59
openstackgerritJames E. Blair proposed openstack-infra/zuul-preview master: DNM: fix dockerfile  https://review.openstack.org/64090423:59
*** wolverineav has joined #openstack-infra23:59

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