Wednesday, 2019-03-06

corvusclarkb, mordred, fungi: https://review.openstack.org/640900 is ready for review00:00
*** tosky has quit IRC00:00
corvuswait no its not00:01
mordredpoo00:01
corvussorry, it's missing a correction i thought i made earlier :/00:01
*** armax has quit IRC00:01
*** armax has joined #openstack-infra00:02
*** dmellado has quit IRC00:03
*** wolverineav has quit IRC00:04
*** dmellado has joined #openstack-infra00:04
*** hwoarang has quit IRC00:05
*** armax has quit IRC00:08
*** hwoarang has joined #openstack-infra00:09
openstackgerritIan Wienand proposed openstack-infra/zuul-sphinx master: Add type to role variables  https://review.openstack.org/64116800:14
*** rkukura has quit IRC00:17
clarkbsystemctl daemon-reload does not remove S90gerrit00:21
*** wolverineav has joined #openstack-infra00:22
clarkbthe LSB info in that sysv init script probably does explain the S04gerrit and that would be loaded by systemd's compat layer00:24
*** sshnaidm is now known as sshnaidm|afk00:28
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115500:29
*** jesusaur has quit IRC00:30
*** rkukura has joined #openstack-infra00:32
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use a buildset registry in the container jobs  https://review.openstack.org/64090000:34
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: test broken zuul-preview dockerfile  https://review.openstack.org/64090300:34
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: test fixed zuul-preview dockerfile  https://review.openstack.org/64090500:34
*** jesusaur has joined #openstack-infra00:34
corvusmordred, clarkb: even with that, there's still a hole in our testing, in that we don't have a great way of saying that if the gitea deployment job needs the gitea image build job, it should depend on it, otherwise it should not.  this only comes about because we build and deploy the gitea image in the system-config repo.  we should be good for images that are built outside of system-config.00:36
corvuswe may need a new zuul feature to handle that.  i'll need to think about it when my mind is more clear00:36
ianwdid something change with log copying?  in http://logs.openstack.org/55/641155/1/check/system-config-run-dns/6d095d6/ ... everything used to be under "hosts/<hostname>" ... now we have both the hostnames at top level and a hosts/ directory with the ara report etc00:37
clarkbianw: that may be a side effect of mordred and pabelangers artifact collection changes00:37
ianwhrm, yeah, all the top level dirs have "docker" and "syslog"00:38
corvusyes, i added the stage-output role00:38
corvuswhich has a difference of opinion with fetch-output on where output should be staged00:39
clarkbinfra-root cmurphy based on watching when the S90gerrit file goes away and mapping to syslog it is the package install of puppet that removes S90gerrit00:39
corvusthis is something that should be corrected.  i haven't had time yet.00:39
clarkbso that mystery at least is mostly solved and we should stop deleting those files when we stop installing puppet more than we need to00:39
clarkbcmurphy: ^ maybe we can improve the install puppet script?00:39
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115800:42
clarkbcorvus: are the opendev-build-docker-image and friends jobs able to be deleted?00:48
*** wolverineav has quit IRC00:48
*** markvoelker has joined #openstack-infra00:49
*** wolverineav has joined #openstack-infra00:51
clarkbinfra-root https://review.openstack.org/#/q/owner:cboylan+topic:puppet-4+status:open is all green with zuul now00:52
clarkbI'm going to call it a day00:52
clarkbcmurphy: ^ tomorrow morning (for me) I'll do my best to generate that module deps list and we can work through it00:52
fungii will be doing the same once tc office hour is over00:52
fungithankfully election transition from tc to ptl a few minutes ago had lots of hands on deck00:53
clarkbthe vcsrepo should actually clean up a bunch of the deps issues with puppet400:53
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115500:55
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: switch jobs list to a tree view  https://review.openstack.org/63343700:58
*** mriedem has quit IRC01:00
*** wolverineav has quit IRC01:03
*** sthussey has quit IRC01:04
*** wolverineav has joined #openstack-infra01:04
*** whoami-rajat has joined #openstack-infra01:09
*** wolverineav has quit IRC01:09
*** armax has joined #openstack-infra01:13
*** oanson has quit IRC01:13
corvusclarkb: the opendev-build-docker-image job is brand new01:19
corvusi don't think we should delete it01:19
clarkbcorvus: ok between ps1 and 2 it went from using that job as a pare t to not01:20
corvusclarkb: it's used by zuul-preview01:20
clarkbwasnt sure if that implied the zuul-jobs job was the location we are moving to01:20
corvusclarkb: it's the version of the job that *includes* a buildset registry, whereas the jobs in system-config build on their own and use a buildset registry provided by a separate job (opendev-buildset-registry)01:21
*** gyee has quit IRC01:21
*** markvoelker has quit IRC01:22
corvusi will write this all up once the dust settles :)01:22
corvuslmit 7601:25
corvusgah01:25
*** sdake has quit IRC01:29
*** ricolin has joined #openstack-infra01:34
*** wolverineav has joined #openstack-infra01:36
*** sdake has joined #openstack-infra01:46
openstackgerritMerged openstack-infra/system-config master: Use a buildset registry in the container jobs  https://review.openstack.org/64090001:53
*** jhesketh has quit IRC01:57
*** jhesketh has joined #openstack-infra01:58
*** wolverineav has quit IRC02:00
*** wolverineav has joined #openstack-infra02:00
*** bhavikdbavishi has joined #openstack-infra02:02
*** sdake has quit IRC02:05
openstackgerritMerged openstack-infra/storyboard-webclient master: Clear task title field on saving a new task  https://review.openstack.org/63455702:14
*** hwoarang has quit IRC02:14
*** hwoarang has joined #openstack-infra02:16
*** markvoelker has joined #openstack-infra02:19
*** wolverineav has quit IRC02:22
*** hongbin has joined #openstack-infra02:26
*** wolverineav has joined #openstack-infra02:27
*** sdake has joined #openstack-infra02:30
*** rkukura has quit IRC02:35
*** bhavikdbavishi has quit IRC02:39
*** wolverineav has quit IRC02:40
*** sdake has quit IRC02:42
*** psachin has joined #openstack-infra02:42
*** markvoelker has quit IRC02:52
*** ykarel has joined #openstack-infra02:54
*** yamamoto has joined #openstack-infra03:00
*** rlandy|bbl is now known as rlandy03:06
*** hwoarang has quit IRC03:12
*** anteaya has quit IRC03:15
*** hwoarang has joined #openstack-infra03:16
*** sdake has joined #openstack-infra03:20
*** sdake has quit IRC03:23
*** sdake_ has joined #openstack-infra03:24
*** bhavikdbavishi has joined #openstack-infra03:24
*** bhavikdbavishi has quit IRC03:28
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115503:29
*** bhavikdbavishi has joined #openstack-infra03:42
*** wolverineav has joined #openstack-infra03:43
*** wolverineav has quit IRC03:44
*** janki has joined #openstack-infra03:49
*** markvoelker has joined #openstack-infra03:49
*** janki has quit IRC03:50
*** janki has joined #openstack-infra03:50
*** roman_g has quit IRC03:57
*** sdake_ has quit IRC03:58
*** wolverineav has joined #openstack-infra03:58
*** hwoarang has quit IRC04:12
*** rlandy has quit IRC04:12
*** gregoryo has joined #openstack-infra04:14
*** ramishra has joined #openstack-infra04:14
*** hwoarang has joined #openstack-infra04:18
*** markvoelker has quit IRC04:22
*** armax has quit IRC04:32
*** sdake has joined #openstack-infra04:32
*** sdake has quit IRC04:32
*** hongbin has quit IRC04:42
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115504:46
openstackgerritDoug Wiegley proposed openstack/gertty master: Enable review keys in diffs, and close change on review  https://review.openstack.org/64119704:55
*** raukadah is now known as chandankumar04:56
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115505:02
*** dave-mccowan has quit IRC05:03
*** sdake has joined #openstack-infra05:04
*** sdake has quit IRC05:04
*** sdake has joined #openstack-infra05:05
*** sdake has quit IRC05:05
*** ianychoi_ has joined #openstack-infra05:24
*** ianychoi has quit IRC05:28
*** yboaron has joined #openstack-infra05:29
*** sdake has joined #openstack-infra05:30
*** sdake has quit IRC05:30
*** sdake has joined #openstack-infra05:37
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115505:39
*** wolverineav has quit IRC05:40
*** kjackal has joined #openstack-infra05:52
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115506:00
*** udesale has joined #openstack-infra06:07
*** wolverineav has joined #openstack-infra06:14
*** wolverineav has quit IRC06:18
*** wolverineav has joined #openstack-infra06:18
*** rkukura has joined #openstack-infra06:21
*** hwoarang has quit IRC06:26
*** hwoarang has joined #openstack-infra06:28
*** e0ne has joined #openstack-infra06:31
*** wolverineav has quit IRC06:32
*** markvoelker has joined #openstack-infra06:49
*** quiquell|off is now known as quiquell06:53
*** pgaxatte has joined #openstack-infra07:21
*** markvoelker has quit IRC07:21
*** rascasoft has quit IRC07:36
*** rascasoft has joined #openstack-infra07:37
*** ccamacho has joined #openstack-infra07:41
*** e0ne has quit IRC07:47
*** quiquell is now known as quiquell|brb07:50
*** e0ne has joined #openstack-infra07:54
*** gregoryo has quit IRC07:57
*** cmoura has quit IRC07:59
*** cmoura has joined #openstack-infra08:00
*** rpittau|afk is now known as rpittau08:02
*** dtantsur|afk is now known as dtantsur08:03
*** dtantsur is now known as dtantsur|mtg08:03
cmurphyclarkb: i think we had that warning about missing module deps pre puppet 4, i was never concerned about it since puppet-apply and beaker were always passing08:03
*** wolverineav has joined #openstack-infra08:03
*** ginopc has joined #openstack-infra08:04
*** kopecmartin|off is now known as kopecmartin08:05
*** wolverineav has quit IRC08:08
*** tkajinam has quit IRC08:09
*** wolverineav has joined #openstack-infra08:10
*** e0ne has quit IRC08:16
*** wolverineav has quit IRC08:17
*** wolverineav has joined #openstack-infra08:18
*** markvoelker has joined #openstack-infra08:19
*** helenafm has joined #openstack-infra08:27
*** quiquell|brb is now known as quiquell08:28
*** eumel8 has joined #openstack-infra08:29
*** tosky has joined #openstack-infra08:29
*** pcaruana has joined #openstack-infra08:29
*** yboaron_ has joined #openstack-infra08:32
*** yboaron has quit IRC08:34
*** aojea has joined #openstack-infra08:49
*** xek has joined #openstack-infra08:49
*** markvoelker has quit IRC08:52
*** jpena|off is now known as jpena08:55
*** dpawlik has joined #openstack-infra08:57
*** jpich has joined #openstack-infra08:58
*** gfidente has joined #openstack-infra09:00
*** yboaron_ has quit IRC09:06
*** yboaron_ has joined #openstack-infra09:07
*** roman_g has joined #openstack-infra09:07
*** dpawlik has quit IRC09:17
frickler#status log manually removed /var/log/exim4/paniclog on new g*.opendev.org servers after crosschecking the contents to reduce spam09:18
openstackstatusfrickler: finished logging09:18
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115509:18
fricklerinfra-root: what is the status of zp01.opendev.org? it's trying to send mails, which bounce due to lacking A record09:19
openstackgerritIan Wienand proposed openstack-infra/zone-opendev.org master: Add acme.opendev.org  https://review.openstack.org/64115409:19
*** wolverineav has quit IRC09:19
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool master: Implement a Runc driver  https://review.openstack.org/53555609:22
*** sshnaidm|afk has quit IRC09:26
openstackgerritIan Wienand proposed openstack-infra/system-config master: [WIP] Add unmanaged domains, add acme.opendev.org  https://review.openstack.org/64115509:36
*** ykarel is now known as ykarel|lunch09:36
*** derekh has joined #openstack-infra09:37
*** aojeagarcia has joined #openstack-infra09:42
*** aojea has quit IRC09:46
*** markvoelker has joined #openstack-infra09:49
*** luizbag has joined #openstack-infra10:06
openstackgerritMerged openstack-infra/irc-meetings master: Change Heat meeting time  https://review.openstack.org/64106210:10
openstackgerritMerged openstack-infra/irc-meetings master: Update Airship meeting time, change chair  https://review.openstack.org/64035910:10
*** markvoelker has quit IRC10:22
*** ykarel|lunch has quit IRC10:27
openstackgerritIan Wienand proposed openstack-infra/system-config master: master-nameserver: Add unmanaged domains; add acme.opendev.org  https://review.openstack.org/64115510:28
*** ykarel|lunch has joined #openstack-infra10:28
*** yamamoto has quit IRC10:34
*** e0ne has joined #openstack-infra10:35
*** ramishra has quit IRC10:40
ianwfungi / corvus: appreciate reviews on 641154, 641155 and 641168 for the initial letsencrypt work to get acme.opendev.org available10:44
*** ykarel|lunch is now known as ykarel10:45
*** electrofelix has joined #openstack-infra10:49
*** wolverineav has joined #openstack-infra10:53
*** sshnaidm|afk has joined #openstack-infra10:54
*** iurygregory is now known as iurygregory|lunc10:54
*** udesale has quit IRC10:57
*** sshnaidm|afk has quit IRC10:57
*** wolverineav has quit IRC10:57
*** xek has quit IRC10:58
*** xek_ has joined #openstack-infra10:58
*** ramishra has joined #openstack-infra10:59
*** FlorianFa has quit IRC11:01
*** FlorianFa has joined #openstack-infra11:02
*** helenafm_ has joined #openstack-infra11:04
*** pcaruana has quit IRC11:04
*** raissa has joined #openstack-infra11:05
*** helenafm has quit IRC11:06
*** panda|ruck|off is now known as panda|ruck|flu11:10
*** yamamoto has joined #openstack-infra11:10
*** bhavikdbavishi has quit IRC11:16
*** yamamoto has quit IRC11:18
*** markvoelker has joined #openstack-infra11:19
*** ianychoi_ is now known as ianychoi11:20
*** ginopc has quit IRC11:25
*** ginopc has joined #openstack-infra11:26
*** yamamoto has joined #openstack-infra11:27
fricklerinfra-root: forget to mention in the meeting yesterday: system-config-run-base-ansible-devel is failing with some centos7+sudo/become issue, would be great if someone had time to look into that in more detail http://logs.openstack.org/55/641155/11/check/system-config-run-base-ansible-devel/78f9bb4/job-output.txt.gz#_2019-03-06_10_46_24_12608011:28
*** pcaruana has joined #openstack-infra11:32
*** ldnunes has joined #openstack-infra11:36
*** ginopc has quit IRC11:40
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: Proposed spec: tenant-scoped admin web API  https://review.openstack.org/56232111:45
*** dave-mccowan has joined #openstack-infra11:46
*** ginopc has joined #openstack-infra11:49
*** markvoelker has quit IRC11:52
*** xek_ has quit IRC11:54
*** sshnaidm|afk has joined #openstack-infra12:01
*** iurygregory|lunc is now known as iurygregory12:02
*** ginux has joined #openstack-infra12:03
*** ginopc has quit IRC12:05
*** ginux is now known as ginopc12:05
*** bhavikdbavishi has joined #openstack-infra12:19
*** jpena is now known as jpena|lunch12:31
*** kgiusti has joined #openstack-infra12:35
*** udesale has joined #openstack-infra12:38
*** kgiusti has quit IRC12:38
*** markvoelker has joined #openstack-infra12:49
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Updates for CFP  https://review.openstack.org/64134712:57
openstackgerritMerged openstack-infra/openstackid-resources master: Updates for CFP  https://review.openstack.org/64134712:59
*** TheJulia_sick is now known as TheJulia13:05
*** pgaxatte has quit IRC13:05
*** pgaxatte has joined #openstack-infra13:07
*** rlandy has joined #openstack-infra13:21
*** markvoelker has quit IRC13:22
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Update API code to work with Presentation Moderators collection (+N)  https://review.openstack.org/64109113:25
*** sshnaidm|afk is now known as sshnaidm13:32
*** jpena|lunch is now known as jpena13:34
*** mriedem has joined #openstack-infra13:34
*** bhavikdbavishi has quit IRC13:36
*** quiquell is now known as quiquell|lunch13:42
*** gfidente has quit IRC13:44
*** jamesmcarthur has joined #openstack-infra13:45
*** yamamoto has quit IRC13:47
*** yamamoto has joined #openstack-infra13:48
*** pcaruana has quit IRC13:51
*** eharney has joined #openstack-infra13:51
*** sdake has quit IRC13:54
*** quiquell|lunch is now known as quiquell13:55
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] web: add tenant and project scoped, JWT-protected actions  https://review.openstack.org/57690713:55
openstackgerritjacky06 proposed openstack/os-performance-tools master: Update hacking version  https://review.openstack.org/64136313:58
*** sdake has joined #openstack-infra14:00
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Allow operator to generate auth tokens through the CLI  https://review.openstack.org/63619714:01
*** pcaruana has joined #openstack-infra14:01
*** rfolco|pto is now known as rfolco|ruck14:02
*** wolverineav has joined #openstack-infra14:08
*** FlorianFa has quit IRC14:09
*** wolverineav has quit IRC14:12
*** sdake has quit IRC14:17
*** jamesmcarthur has quit IRC14:18
*** markvoelker has joined #openstack-infra14:19
*** rascasoft has quit IRC14:20
openstackgerritjacky06 proposed openstack/os-testr master: Update hacking version  https://review.openstack.org/64138414:22
*** sdake has joined #openstack-infra14:22
quiquellHi all14:24
quiquellDo nested virtualization works at zuul jobs ?14:24
*** sdake has quit IRC14:24
*** jcoufal has joined #openstack-infra14:27
mordredquiquell: not really in the openstack zuul cloud providers - as in, it's not something you can/should count on being available right now. on some of our providers nested virt triggers kernel panics, so we have it configured off14:28
mordredquiquell: there are folks, like mnaser, who are interested in being able to provide working and reliable nested virt14:29
mnaseryes, and it was working, until a few days ago :(14:29
mordred(it works well at vexxhost already - but we need it working more consistently across multiple providers before it's a thing we can enable in teh gate)14:29
mordredoh - yeah - until it stops working :)14:30
mnaserunfortunately i haven't had the most of time to dig into it14:30
mnaseri bet kata hasn't been hit because they don't update their kernels all the time14:30
quiquellso it works when it works :-)14:30
quiquellok, thanks14:30
quiquellsshnaidm: ^14:30
*** jamesmcarthur has joined #openstack-infra14:35
*** jamesmcarthur has quit IRC14:36
*** jamesmcarthur_ has joined #openstack-infra14:36
quiquell mordred so vexxhost is consistent with nested ?14:37
*** sdake has joined #openstack-infra14:39
*** kgiusti has joined #openstack-infra14:39
mordredquiquell: well - it was, until a couple of days ago14:40
quiquellack14:41
mordredquiquell: something broke - we think a kernel update14:41
quiquellsshnaidm: ^ maybe vexxhost has issues too :-/14:41
*** rascasoft has joined #openstack-infra14:43
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Allow operator to generate auth tokens through the CLI  https://review.openstack.org/63619714:43
*** anteaya has joined #openstack-infra14:46
*** gfidente has joined #openstack-infra14:47
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] web: add tenant and project scoped, JWT-protected actions  https://review.openstack.org/57690714:47
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Allow operator to generate auth tokens through the CLI  https://review.openstack.org/63619714:47
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul CLI: allow access via REST  https://review.openstack.org/63631514:49
*** rascasoft has quit IRC14:50
*** rascasoft has joined #openstack-infra14:51
*** markvoelker has quit IRC14:52
*** sdake has quit IRC15:00
*** sthussey has joined #openstack-infra15:01
*** pgaxatte has quit IRC15:02
*** chandankumar is now known as chkumar|pto15:03
*** pgaxatte has joined #openstack-infra15:04
*** sdake has joined #openstack-infra15:06
*** agopi has joined #openstack-infra15:14
*** ekultails has joined #openstack-infra15:16
*** agopi has quit IRC15:18
*** quiquell is now known as quiquell|off15:21
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] web: add tenant and project scoped, JWT-protected actions  https://review.openstack.org/57690715:26
*** xek_ has joined #openstack-infra15:30
*** xek__ has joined #openstack-infra15:32
*** xek_ has quit IRC15:35
*** udesale has quit IRC15:37
*** Sundar has joined #openstack-infra15:38
SundarHi, I have 2 email addresses registered -- my Intel email and gmail. My submission for PTL candidacy hit an issue because it could not locate the Intel email address, though it is registered and is the preferred email. Please LMK what is needed.15:40
Sundar^ Logs here: http://logs.openstack.org/07/641407/1/check/openstack-tox-docs/116a531/job-output.txt.gz#_2019-03-06_15_05_02_81840515:40
fungiSundar: you have it added to your openstack.org foundation member profile?15:41
Sundarfungi: Yes15:41
SundarI can see it in Profile, Preferences, etc.15:41
SundarWhen i attempted o register it now, it says that is already registered.15:41
fungiSundar: i think you're talking about gerrit and not your osf membership15:42
fungiSundar: see https://docs.openstack.org/contributors/common/accounts.html#setup-your-openstack-foundation-account15:42
SundarOh wait, openstack.org foundation? Is that a separate step from registering in review.openstack.org?15:42
fungiyes, completely separate15:42
fungiall ptls must be individual members of the openstack foundation in good standing15:43
Sundarfungi: Thanks :) Will follow up.15:43
*** lpetrut has joined #openstack-infra15:50
*** markvoelker has joined #openstack-infra15:50
openstackgerritJan Gutter proposed openstack/diskimage-builder master: Clean the BSP partition for the EFI layout  https://review.openstack.org/64142615:50
*** dtantsur|mtg is now known as dtantsur|afk15:50
*** pcaruana has quit IRC15:53
clarkbcmurphy: gotcha, the bit that scared me was puppet saying those modules won't be available. But I'm guessing that is because it thinks the module isn't available at all when it actually is15:53
clarkbin any case cleaning that up would be nice and my changes should put a big dent in that (particularly the vcsrepo stuff)15:53
cmurphyagreed15:53
cmurphy+1'd/+2'd15:53
*** janki has quit IRC15:55
clarkbinfra-root can I get reviews on https://review.openstack.org/#/q/owner:cboylan+topic:puppet-4+status:open to help clean up some puppet-4 related things?15:55
*** armax has joined #openstack-infra15:56
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Allow operator to generate auth tokens through the CLI  https://review.openstack.org/63619715:57
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul CLI: allow access via REST  https://review.openstack.org/63631515:57
*** raissa has quit IRC15:59
mordredinfra-root: the tc.log issue with mariadb that requires deleting the tc.log on restart sometimes has been fixed in the 10.4 branch  of mariadb upstream so should be included in the next docker image16:00
*** luizbag has quit IRC16:00
*** luizbag has joined #openstack-infra16:01
clarkbmordred: cool. Looks like we still need to update the default branches? I'm guessing we need to run the fixup playbook for that/16:01
clarkbmordred: do we want to test the ansible POST on a single repo to start?16:01
*** yboaron_ has quit IRC16:04
Shrewsmordred: neat16:06
*** pcaruana has joined #openstack-infra16:06
mordredclarkb: yes - probably so. I've got a phone call in 25 minutes so I don't want to dive in to working on that just yet - but can work on it once my call is over16:06
mordredalthoughj I will not be upset if someone else decides they want to work on it in the meantime16:07
clarkbmordred: I too am in a meeting right now so I'm unlikley to get to it soon16:07
*** sdake has quit IRC16:09
*** ricolin has quit IRC16:11
mordredkk16:13
*** sdake has joined #openstack-infra16:13
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Add Authorization Rules configuration  https://review.openstack.org/63985516:14
*** roman_g has quit IRC16:14
Sundarfungi: I am an OpenStack Foundation member with my gmail address. If I just add my corporate address to the same profile, is that enough? Or should I separately register the corporate address?16:14
*** roman_g has joined #openstack-infra16:14
*** markvoelker has quit IRC16:22
*** ccamacho has quit IRC16:27
fungiSundar: whatever address you want to use in connection with your potential leadership role as a ptl can be added to an available field in your foundation member profile16:34
fungiit doesn't have to be the primary address in your foundation member profile, it can be in any of the three available e-mail address fields16:35
fungibut at least one of the addresses in your member profile also needs to be registered on your gerrit contact settings too (that's how we confirm you are a code contributor to the project)16:36
fungiand so the address you use for your ptl platform submission needs to be registered in your gerrit contact settings16:36
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Add Authorization Rules configuration  https://review.openstack.org/63985516:39
openstackgerritJan Gutter proposed openstack/diskimage-builder master: [DNM] Clean the BSP partition for the EFI layout  https://review.openstack.org/64142616:43
*** sdake has quit IRC16:48
*** sdake has joined #openstack-infra16:49
openstackgerritJames E. Blair proposed openstack-infra/zuul master: WIP allow soft job dependencies  https://review.openstack.org/64143916:49
Sundarfungi: Thanks. Just one more question. Foundation web page says there is one more account with same first/last names as me, with email s*************r@gmail.com. But that is not my email, and it could be somebody else's. When I click on 'This is not my account', it says it will delete that -- but somebody else may get affected then.16:50
*** pgaxatte has quit IRC16:51
clarkbSundar: that is probably something we want to bring up with jamesmcarthur_16:51
clarkbSundar: as you say it is entirely possible this isn't an error16:51
Sundarclarkb: Thanks, will do.16:52
Sundarjamesmcarthur_: Please take a look at my query above.16:53
jamesmcarthur_Sundar: It should just allow you to dismiss the message. You won't be able to delete the other account w/o a login.16:53
Sundarjamesmcarthur_: OK, I'll click on 'This is not my account' and see what happens.16:54
*** pcaruana has quit IRC16:55
*** sdake has quit IRC16:56
*** panda|ruck|flu is now known as panda|ruck|off16:57
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115816:57
*** ccamacho has joined #openstack-infra17:02
*** eharney has quit IRC17:06
*** gyee has joined #openstack-infra17:06
*** eharney has joined #openstack-infra17:13
*** ramishra has quit IRC17:14
*** lpetrut has quit IRC17:14
*** ccamacho has quit IRC17:15
*** yamamoto has quit IRC17:15
clarkbianw: I think I'll take you up on your offer of afs upgrade help if you want to walk through an upgrade of afs01.ord today via shared screen session?17:16
clarkbhrm one thing I realize is I don't know how do-release-upgrade is going to handle puppet installs. But we should be able to fix that post install since puppet/ansible won't be running anyway17:16
*** lpetrut has joined #openstack-infra17:17
clarkbianw: if today is bad I can do tomorrow as well17:18
*** ccamacho has joined #openstack-infra17:19
*** markvoelker has joined #openstack-infra17:19
*** ykarel is now known as ykarel|away17:20
fungidoes ubuntu provide release notes explaining what do-release-upgrade is going to do aside from an apt full-upgrade command?17:21
*** rlandy is now known as rlandy|brb17:21
fungii'm too used to performing in-place upgrades with debian, where you just read the release notes and then adjust your sources.list and use the package manager, it doesn't provide any separate tool17:22
clarkbfungi: not that I am aware of17:23
fungii've gotten the impression in the past that do-release-upgrade is basically just automating any arbitrary manual things which would normally be spelled out in the release notes in debian's terms17:24
clarkbif we want ot be really cautious I can snapshot afs01.ord and do-release-upgrade on the snapshot first17:24
clarkbnot sure how I need to disconnect it from the afs cluster for that though17:24
clarkbI'm guessing I'll mostly be fine since all that is managed by the db servers and the new server won't be in the db17:24
clarkbcorvus: ^ any concern with booting a snapshot of afs01.ord and causing problems for afs?17:28
*** yamamoto has joined #openstack-infra17:28
corvusclarkb: yes -- i'd have to refresh my memory of how fileservers talk to dbservers.  it may well have the requisite credentials already installed.17:29
clarkbcorvus: unrelated https://review.openstack.org/#/c/641158/5 may have found an ansible-lint bug?17:32
clarkb(I left details on the cahnge)17:32
*** lpetrut has quit IRC17:32
*** kopecmartin is now known as kopecmartin|off17:33
*** yamamoto has quit IRC17:34
*** e0ne has quit IRC17:34
corvusclarkb: probably if you shut down afs and removed /etc/openafs/server before making the snapshot it'd be ok?17:34
*** helenafm_ has quit IRC17:34
clarkbcorvus: ok17:34
pabelangerclarkb: left comment also17:35
corvusclarkb: (but put that directory back before bringing it back online)17:35
clarkbcorvus: ya17:36
corvusyou know what?  i'd really rather not spend my day guessing which of 10 perfectly valid ways of running a command ansible-lint wants me to type.17:37
*** jpich has quit IRC17:37
*** rkukura has quit IRC17:38
*** rlandy|brb is now known as rlandy17:38
mordredclarkb: I'm going to test the branch setting now17:39
corvusclarkb, pabelanger: can the two of you please give me a recommendation on how to proceed?17:39
clarkbmordred: I think corvus is doing it via https://review.openstack.org/#/c/641158/5 and it seems to work there17:39
corvusclarkb: nope17:40
corvusclarkb: that's the baseline -- if that change works, you can reparent mordred's change to mine and it will be tested17:40
clarkbcorvus: gotcha17:40
clarkbwell corvus' chagne does work other than a sad linter17:40
clarkbcorvus: re the linter I don't know what the best course is there. I find myself annoyed by ansible-lint often myself17:41
mordredoh - duh. /me will wait for that change17:41
clarkbcorvus: we could no lint the task, disable that rule from being checked, or stop linting entirely? The problem with not linting at all is it does catch real issues it just also catches these non issues with obtuse errors17:42
corvusmy next patchset is either going to disable that rule or the linter entirely because ansible-lint has never prevented an error, only annoyed me.  so someone who cares about ansible-lint is welcome to propose an alternative.17:42
corvusclarkb: what real issue has ansible-lint caught?17:42
clarkbcorvus: yaml format errors iirc17:42
clarkb(there are other ways of catchign those fwiw)17:42
corvusso do you disable this with -x ANSIBLE0304 ?17:43
clarkbcorvus: yes that appears to be the case17:44
*** wolverineav has joined #openstack-infra17:44
clarkbI checked that 304 is the environment error number to make sure I am reading that error line properly and it is17:44
clarkbhttps://docs.ansible.com/ansible-lint/rules/default_rules.html is the table of errors17:45
corvusmordred: no need to wait.  please proceed with fixing default branch as quickly as possible.  my next task is waiting on that.17:45
mordredcorvus: ok! I will proceed17:45
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115817:45
corvusthere's the test disabled.  we're up to 9 now.17:46
clarkbhttps://github.com/ansible/ansible-lint/commit/80ed68746432cf9484a5cb8bb3d912d10aa374b3#diff-2d074af6d234f70358049a66b660c721 they have fixed that bug at least17:46
pabelangerI want to say it might be -x 304, because I have no idea why17:46
pabelangernew rules don't seem to use ANSIBLE prefix17:47
corvusmordred: if https://review.openstack.org/641158 fails again, can you take it over?  i only wrote it to test your change anyway.17:48
mordredcorvus: yup17:48
pabelangerbut ya, ansible-lint has started be become very opinionated in playbook design. I can see us disabling it moving forward17:48
*** wolverineav has quit IRC17:49
clarkbpabelanger: for me the major fail there is lack of info on how to fix the error. That said this appears to have been a bug they fixed of course we can't get to ansible-lint 4.x without a ton of changes anyway so bugfix not super valuable right now17:49
pabelangerclarkb: agree17:50
*** jamesmcarthur_ has quit IRC17:50
*** jamesmcarthur has joined #openstack-infra17:50
*** jamesmcarthur has quit IRC17:50
*** jamesmcarthur has joined #openstack-infra17:51
pabelangerclarkb: the roadmap for ansible-lint is very much tied to galaxy, so if we don't plan on uploading to galaxy (and want a good score), we likely can consider other yaml lint tools17:51
*** markvoelker has quit IRC17:52
mordredclarkb: the 'set default branch' task seems to work as long as there is content in the repo - I did just get a 404 for https://opendev.org/openstack-attic/compute-api - but there is no content in that repo in gitea right now, so it matches17:54
*** wolverineav has joined #openstack-infra17:54
mordredclarkb: I have also seen the task succeed on repos that do exist- although the one I happened to catch as it scrolled by only had a master branch17:54
mordredclarkb: I think that means that the set-default-branch option isn't going to work and we're going to need to do the create-empty-project option17:55
clarkbmordred: ok and in that case pushing an ephemeral master branch should be sufficient to make the default branch master (however we shouldn't do that in the fixup case if the project already has content? so maybe we need a combo of both?)17:56
mordredclarkb: ok - it did fix https://opendev.org/openstack-attic/akanda-appliance-builder17:56
mordredclarkb: yeah - to do the fixup I think the easiest thing to do is actually likely just do a one-off sql query - or else make sure to replicate to compute-api and any other repo that doesn't have content yet17:57
*** mattw4 has joined #openstack-infra17:57
corvusi wonder why it doesn't have content?  replication finished yesterday17:58
clarkband we aren't excluding the attic projects from our replication config17:59
corvus(it really didn't take very long)17:59
mordredhttp://git.openstack.org/cgit/openstack-attic/compute-api17:59
clarkbthey are excluding from replicating to github but not gitea17:59
mordredis empty17:59
corvusthat project doesn't exist in gerrit?18:00
mordredactually - openstack-attic isn't even on git.o.o18:00
*** ccamacho has quit IRC18:00
mordredno -there is no compute-api in gerrit18:00
corvushttps://review.openstack.org/#/admin/projects/?filter=openstack-attic18:00
corvusbut there are other openstack-attic projects there18:01
*** wolverineav has quit IRC18:01
corvusit's "hidden" in gerrit18:02
corvusif you log in as an admin, you'll see it here: https://review.openstack.org/#/admin/projects/openstack-attic/compute-api18:02
*** derekh has quit IRC18:03
*** rkukura has joined #openstack-infra18:04
mordredcorvus: hrm. so maybe we should unhide it in gerrit?18:04
mordredcorvus: since we don't hide the others18:04
corvusi'm trying to dig into the config history, but that's how i'm leaning18:04
clarkbwould certainly reduce confusion18:05
*** psachin has quit IRC18:05
fungiyeah, i bet that was a very early iteration of our project retirement process18:05
mordredif we do that - we should be able to trigger a forced replication to get it populated in gitea18:05
fungiright, you can ask the replication plugin to just replicate a specific repo by name18:06
mordredyeah18:06
clarkbthinking out loud here we never have empty git repos at the end of the day. We either pull in existing content or make a .gitreview commit18:06
*** wolverineav has joined #openstack-infra18:06
*** wolverineav has quit IRC18:06
clarkbbut we don't create the project in gerrit until after we create it in gitea18:06
*** jtomasek has quit IRC18:06
*** wolverineav has joined #openstack-infra18:06
clarkbso we may need to create in gitea, create in gerrit, then set default branch?18:06
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Validate ansible installations on startup  https://review.openstack.org/63741818:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Make ansible version configurable  https://review.openstack.org/63742218:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Install ansible during executor startup if needed  https://review.openstack.org/64064418:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Support ansible 2.6  https://review.openstack.org/63193118:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: WIP: Support ansible 2.7  https://review.openstack.org/63193218:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Switch default ansible version to 2.7  https://review.openstack.org/63742418:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: DNM: zuul-stream-functional debugging  https://review.openstack.org/64064818:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: DNM: Test concurrent tox remote  https://review.openstack.org/64065418:07
openstackgerritDoug Wiegley proposed openstack/gertty master: Enable review keys in diffs, and close change on review  https://review.openstack.org/64119718:08
*** yamamoto has joined #openstack-infra18:09
corvusmordred: i dug up the history. it looks like there were a bunch of changes made by you through the web ui starting around Sat Oct 17 19:31:05 2015 +0000.  but there's no additional info.18:09
mordredcorvus: yay for me18:09
corvusmordred: so, unless you remember otherwise, we can probably assume it was an accident -- selected the wrong item in the drop down or something18:10
mordredclarkb: oh - I see what you're saying - do the gitea set default branch action after... wait no - that won't work18:10
mordredcorvus: yeah. let's assume that18:10
mordredclarkb: creating the project in gerrit does not equal the project getting replicated to gitea18:10
clarkbmordred: it should18:11
corvusmordred: happily, since then, you've created a system that prevents this from happening.  we just didn't know we had already demonstrated its necessity.  :)18:11
mordredclarkb: so we still might attempt to run the set-default-branch before the project has been replicated18:11
clarkbmordred: however there may be a delay18:11
clarkbya18:11
mordredclarkb: right. I mean the set-default-branch would need to wait for content to exist18:11
*** ginopc has quit IRC18:12
corvusclarkb, mordred: i switched it to read-only in the web ui18:12
corvuswe should probably update projects.yaml, but at least it matches its siblings now18:13
mordredcorvus: it's gerrit replication start openstack-attic/compute-api right?18:13
corvusmordred: i think you drop 'gerrit' now18:14
mordredkk18:14
clarkbya the plugin has its own command namespace iirc18:14
* mordred kicks replication for that18:14
*** yamamoto has quit IRC18:14
clarkbmordred: maybe the thing to do is if project was just created in gitea then push master branch else (if fixup) set branch via settings url18:14
mordredclarkb: yeah18:16
mordredor, rather - have the creation path create empty repo in gitea - but have the fixup playbook do the settings url18:16
* mordred is running the fixup playbook now18:17
mordredbased on previous attempts - I expect it to take what - 1.5 hours?18:17
*** jpena is now known as jpena|off18:19
mordredclarkb: let me push up an amended patch18:19
*** luizbag_ has joined #openstack-infra18:21
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Revert "Revert "Set default branch for repo""  https://review.openstack.org/64110818:22
mordredclarkb: ^^ that should do the logic I think we're talking about18:22
mordredit's going to create repos with default content in them18:22
mordredbut then leave in teh 'set default branch' code - which will normally not do anything on the creation case, but when we run the fixup playbook, it can be useful18:23
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Add foreground option  https://review.openstack.org/63564918:24
*** luizbag__ has joined #openstack-infra18:25
mordredcorvus: patch failed again - I'm taking it over18:25
corvusmordred: thx18:25
clarkbmordred: assuming auto init results in initing to master branch then ya I think that should do what we want18:25
*** luizbag_ has quit IRC18:27
*** luizbag has quit IRC18:27
*** Sundar has quit IRC18:31
*** cmoura has quit IRC18:38
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115818:43
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Revert "Revert "Set default branch for repo""  https://review.openstack.org/64110818:43
*** aojeagarcia has quit IRC18:47
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Update API code to work with Presentation Moderators collection (+N)  https://review.openstack.org/64109118:49
*** markvoelker has joined #openstack-infra18:50
*** jamesmcarthur has quit IRC18:50
*** jamesmcarthur has joined #openstack-infra18:51
*** jamesmcarthur has quit IRC18:55
mordredwatching the project fixup playbook run is very boring18:56
clarkbmordred: care to review https://review.openstack.org/#/q/owner:cboylan+topic:puppet-4+status:open for a slight improvement in the excitement levels?18:57
clarkbthis set of chagnes addresses some stuff I noticed after we upgraded more dev servers to puppet418:57
*** eumel8 has quit IRC18:57
clarkbmordred: out of curiousity are there docs for the gitea settings api?19:00
clarkbthe swagger api stuff doesn't cover it, does something else?19:00
clarkbOr was this capture what the web browser does with debugging tools and use it?19:00
mordredclarkb: that19:06
mordredclarkb: this is a thing where we likel want to go make some patches to add things to the swagger api19:06
*** kaiokmo has quit IRC19:07
mordredclarkb: if (defined('$::project_config_ref') and $::project_config_ref != undef) { is amaing19:08
clarkbmordred: yes...19:09
clarkbmordred: that might be me being overly cautious but I'm 99% sure you can define a variable with an undef value19:09
clarkbwhich is different than having an undefined variable19:09
clarkbin any case puppet complains with a warning so may as well clean it up19:10
mordredyeah. still. jeez19:10
mordredclarkb: +2 and/or +A from me on that stack19:11
clarkbthank you19:11
*** wolverineav has quit IRC19:14
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix test race with Watchdog thread still running  https://review.openstack.org/64147319:18
*** ykarel_ has joined #openstack-infra19:20
*** ykarel|away has quit IRC19:22
*** markvoelker has quit IRC19:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Run tox remote concurrent  https://review.openstack.org/64065419:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: WIP: Support ansible 2.7  https://review.openstack.org/63193219:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Switch default ansible version to 2.7  https://review.openstack.org/63742419:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: DNM: zuul-stream-functional debugging  https://review.openstack.org/64064819:22
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add puppet-vcsrepo as required project on puppet jobs  https://review.openstack.org/64116119:24
*** wolverineav has joined #openstack-infra19:25
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Create a repo for zuul-operator  https://review.openstack.org/64147419:26
*** rfolco|ruck has quit IRC19:26
mordredclarkb: if you were thinking to yourself, "I bet the repo fix playbook is almost done" ... you'd be so wrong19:26
*** wolverineav has quit IRC19:26
*** gfidente has quit IRC19:26
*** wolverineav has joined #openstack-infra19:26
clarkbI was actually thinking to myself: "what do I have for lunch"19:27
clarkbbecause it is snowing outside and I don't feel like going outside to find food19:27
*** electrofelix has quit IRC19:27
*** rfolco has joined #openstack-infra19:27
corvusmmm.  bachelor chow!19:28
fungii ordered a cheeseless thin crust pizza for lunch. they got the cheeseless part right but also made it sauceless, so it's like a dry cracker with loose toppings scattered across the surface19:28
mordredfungi: hahahaha. wow. yeah. that doesnm't sound like food as much as art19:28
corvusfungi: wash it down with some slurm19:29
fungiit's a challenge to consume. slurm would probably help19:29
clarkbalso I've just remembered my new desk is supposed to arrive today so depending on timing of that I may tackle afs01.ord tomorrow19:29
*** wolverineav has quit IRC19:29
fungiwhimmy wham wham wozzle19:29
*** gfidente has joined #openstack-infra19:30
*** wolverineav has joined #openstack-infra19:30
clarkbfungi: corvus any chance you have a moment to be second review on https://review.openstack.org/#/q/owner:cboylan+topic:puppet-4+status:open19:31
fungidone19:33
clarkbthanks19:34
clarkbI guess I should go find food so I can follow up on them once they've merged19:34
openstackgerritMerged openstack-infra/puppet-project_config master: Check if project_config_ref is defined before using it  https://review.openstack.org/64115219:34
*** wolverineav has quit IRC19:35
*** kaiokmo has joined #openstack-infra19:38
*** wolverineav has joined #openstack-infra19:41
*** xek has joined #openstack-infra19:42
*** xek has quit IRC19:44
*** xek_ has joined #openstack-infra19:44
*** xek__ has quit IRC19:45
*** wolverineav has quit IRC19:47
mordredSAD TROMBONE19:48
mordredopenstack/deb-oslotest19:48
mordredalso not a real repo19:49
*** flwang has left #openstack-infra19:50
fungido we have "data quality" issues with our repos? mildly shocked19:50
mordredfungi, clarkb, corvus: as fun as this is - I'd like to just fix this for this time with a sql query if y'all are ok with that19:50
fungimordred: pretend for a moment i haven't been paying super-close attention to scrollback... can you elaborate on "this" with different words?19:52
mordredfungi: the repos in the gitea servers have incorrect settings for default_branch19:52
fungithat this, okay19:53
mordredfungi: we have the create playbook fixed so it won't happen moving forward (at least we think so) - and have been running the sync-gitea-repos playbook to fix the existing ones19:53
fungi(i'm keeping a lot of this'es in my head which need fixing)19:53
clarkbmordred: should we query gerrit for all hidden projects and unhide them so that fixup playbook works in the future at least?19:53
mordredclarkb: this one isn't a hidden project - this one is just flat empty19:53
mordredclarkb: I mean - I think continuing to run the fixup playbook until it runs green is a good idea - so that we know we're in good shape19:54
clarkboh interesting. Thats right yhe deb s tuff defaulted to not master19:54
clarkbmordred: ++19:54
mordredclarkb: hrm. do we have any repos that legit don't want master as their default branch (of things we still care about?)19:55
fungiwell, the deb stuff we imported defaulted to not master on the originals, we "fixed" that (or so we thought) after one or two didn't import correctly19:55
fungiso maybe this was one of those19:55
fungimordred: we made it a policy that the default branch on all our repos is master, but it wasn't until the deb projects import that we added a ci check to make sure we didn't import more in that state19:55
mordredcool19:56
*** wolverineav has joined #openstack-infra19:56
clarkbya I think not doing master introduces so many weird corner cases we should continue to avoid it19:56
fungiso if we find some we didn't fix entirely from the beforetime, we're fine to manually fix those up in my opinion19:56
*** tosky has quit IRC20:00
*** tosky has joined #openstack-infra20:01
*** auristor has quit IRC20:01
*** luizbag__ has quit IRC20:01
mordredclarkb, fungi, corvus: ok. all of the giteas should be fixed with default_branch now20:01
*** e0ne has joined #openstack-infra20:03
fungimordred: if for some reason we need to redo these servers from scratch or add another, they'll get the right default branch on all the repos we sync in from now on right?20:03
fungi(since the playbook is fixed)20:03
mordredfungi: once we land the updated playbook patch20:03
mordredyes20:03
fungiokay, sounds fine to me. thanks!20:03
mordredhttps://review.openstack.org/#/c/641108/ ftr20:04
mordredcorvus: incidentally - your patch is green, and I based my patch on top of it20:04
openstackgerritMerged openstack-infra/system-config master: Install latest puppet-vcsrepo  https://review.openstack.org/64115620:07
openstackgerritMerged openstack-infra/system-config master: Bump puppet/staging to 1.0.1 to satisfy mysql module dep  https://review.openstack.org/64115720:07
openstackgerritMerged openstack-infra/zuul master: Fix test race with Watchdog thread still running  https://review.openstack.org/64147320:10
*** yamamoto has joined #openstack-infra20:11
corvusmordred: thank you!20:14
*** cmoura has joined #openstack-infra20:14
corvus+2s; i'll let clarkb/fungi +320:15
*** yamamoto has quit IRC20:16
*** auristor has joined #openstack-infra20:16
*** markvoelker has joined #openstack-infra20:19
fungione of these days when someone has time, i'm curious to know why we expect status_code: 302 from the gitea api20:20
*** wolverineav has quit IRC20:21
*** e0ne has quit IRC20:21
mordredfungi: well - those calls aren't actually gitea api calls20:25
mordredfungi: those calls are us pretending to be the html of gitea20:25
mordredfungi: in the future, we should add api calls for the places where we're making form submit calls20:26
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Allow operator to generate auth tokens through the CLI  https://review.openstack.org/63619720:26
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul CLI: allow access via REST  https://review.openstack.org/63631520:26
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Add Authorization Rules configuration  https://review.openstack.org/63985520:27
fungiooh20:27
fungiokay, that makes more sense20:27
fungiso after you perform those actions gitea wants to redirect your "browser"20:27
corvusyeah, generally back to the page you were just on20:27
corvusPOST /org/project/settings -> /org/project20:28
fungifun times20:28
clarkbwhich is why we need the csrf tokens too?20:29
*** jamesmcarthur has joined #openstack-infra20:30
corvusyep20:31
*** betherly has joined #openstack-infra20:31
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088420:32
*** wolverineav has joined #openstack-infra20:37
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088420:37
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul Web: add /api/user/actions endpoint  https://review.openstack.org/64109920:38
*** betherly has quit IRC20:41
clarkbthe puppet run just after 2100UTC should apply those puppet things. I'll be watching it to make sure that we get the expected cleanup behavior on those warnings20:44
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Log exception on module failure with empty stdout  https://review.openstack.org/64065020:45
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul Web: add /api/user/actions endpoint  https://review.openstack.org/64109920:46
clarkbI've approved the test gitea chang which will get the fix for branch setting in the gate too20:48
*** eernst has joined #openstack-infra20:50
mordredclarkb: woot20:51
*** betherly has joined #openstack-infra20:51
clarkbI didn't have to resort to bachelor chow either. Reheated some frozen bolognese I made a couple months ago :)20:51
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Log exception on module failure with empty stdout  https://review.openstack.org/64065020:52
*** eernst has quit IRC20:52
*** eernst has joined #openstack-infra20:52
*** markvoelker has quit IRC20:53
*** fdegir has quit IRC20:53
fungifreezers are a marvellous invention20:53
corvusoh i thought he just left it outside20:54
*** fdegir has joined #openstack-infra20:54
*** eernst has quit IRC20:54
fungii think if you do that, the portland pandas are likely to find it20:55
clarkbthankfully I don't live in the part of the wolrd where you can do that (/me looks at smcginnis )20:55
*** betherly has quit IRC20:55
smcginnisWalk out freezers can be very handy!20:56
fungiheck, we did that for the winter 2017 holiday season when we had family coming to stay and the fridge/freezer died on us20:57
fungiand because everybody had decided to buy appliances as gifts, everything was backordered and the delivery folks were backlogged too20:58
fungiworst time of year to have appliances die on you20:58
clarkbaround here its more fridge temp20:58
clarkband I don't trust it for anything but beer20:58
clarkb(but we have had a weird end of winter with it being extra cold)20:59
fungiwe used the garage for a refrigerator and some coolers on the porch for a freezer20:59
fungiit worked out fine20:59
fungipanda-proof coolers20:59
fungii just looked up and realized the day is nearly gone and all i've really "gotten done" is to be in a bunch of meetings21:00
clarkbfungi: you've also got the water which I imagine is a pretty constant temp but probably warmer than fridge temp there?21:00
fungiclarkb: it's fridge temp in the winter. sometimes approaching freezer temperature and that gets really funky with the tides lifting and lowering the ice sheet21:01
clarkbmichigan had an ice tsunami recently21:01
fungithe ice rime that builds up on the pier pilings and cyprus knees is gnarly21:01
fungiworth noting the water *can* get well below 0c since it's briny21:02
*** zbr|ssbarnea has joined #openstack-infra21:07
*** irclogbot_1 has joined #openstack-infra21:08
*** zbr has quit IRC21:10
*** betherly has joined #openstack-infra21:11
*** e0ne has joined #openstack-infra21:13
clarkbpuppet is looking slightly cleaner on review-dev now21:15
clarkbthe staging version bump hasn't made it in yet so on the next run we should see it properly clean up21:15
*** betherly has quit IRC21:15
*** ldnunes has quit IRC21:16
openstackgerritMerged openstack-infra/system-config master: Test gitea project creation playbook  https://review.openstack.org/64115821:17
*** whoami-rajat has quit IRC21:18
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Manage ansible installations within zuul  https://review.openstack.org/63193021:21
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Validate ansible installations on startup  https://review.openstack.org/63741821:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Make ansible version configurable  https://review.openstack.org/63742221:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Install ansible during executor startup if needed  https://review.openstack.org/64064421:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Support ansible 2.6  https://review.openstack.org/63193121:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Run tox remote concurrent  https://review.openstack.org/64065421:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: WIP: Support ansible 2.7  https://review.openstack.org/63193221:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Switch default ansible version to 2.7  https://review.openstack.org/63742421:22
openstackgerritTobias Henkel proposed openstack-infra/zuul master: DNM: zuul-stream-functional debugging  https://review.openstack.org/64064821:22
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: Proposed spec: tenant-scoped admin web API  https://review.openstack.org/56232121:22
*** jamesmcarthur has quit IRC21:24
clarkbfungi: puppet4 has caught that the reply_to_email_address in puppet-storyboard is never defined but used in our storyboard conf erb21:24
*** jamesmcarthur has joined #openstack-infra21:24
clarkbfungi: any chance you know what the intent was there? I'm thinking maybe we just remove it from the config if it has been unsed21:24
openstackgerritGeorge Dimitrov proposed openstack-infra/git-review master: Fix parsing of review over http  https://review.openstack.org/64148421:26
*** irclogbot_1 has quit IRC21:28
*** jamesmcarthur has quit IRC21:28
*** eharney has quit IRC21:29
ianwi wonder what year it will be when i think "wow, it's been a long time since my whole system randomly stopped receiving peripheral input and i had to ssh in and sighup gnome-shell to avoid destroying all my state"21:30
*** betherly has joined #openstack-infra21:31
openstackgerritClark Boylan proposed openstack-infra/puppet-storyboard master: Remove unused puppi dependency  https://review.openstack.org/64148521:32
clarkbfungi: ^ thats a different fix. Happy to push up the removal of that undefined var as well if you think that is appropriate21:33
clarkbianw: the year of the linux desk21:33
clarkber linux desktop21:33
*** betherly has quit IRC21:35
clarkbfungi: SotK looks like SotK added support for that in the template I'm guessing it was just a missed variable in that case?21:37
clarkbI guess I'll define it then21:37
openstackgerritClark Boylan proposed openstack-infra/puppet-storyboard master: Define reply_to_email_address  https://review.openstack.org/64148621:39
clarkbfungi: SotK ^ I think that is the fix21:39
openstackgerritMerged openstack-infra/system-config master: Revert "Revert "Set default branch for repo""  https://review.openstack.org/64110821:40
fungiclarkb: i think it was probably related to the fact that we haven't yet implemented the ability to comment on stories via e-mail21:44
*** betherly has joined #openstack-infra21:46
clarkbianw: did you see my questions about working through afs01.ord soon? also do you think it would be worthwhile to stop afs on it, move the afs config aside, take a snapshot, then test thsi on the snapshot first?21:46
clarkbfungi: my fix for it should support that then21:49
clarkbfungi: I add it as a valid parameter which makes the variable defined21:49
fungiclarkb: i commented in the review, but right now it's resulting in setting the variable to an empty value. wondering if the nil erb is going to turn that undef into will end up as a literal 'nil' string instead?21:50
*** markvoelker has joined #openstack-infra21:50
clarkbfungi: I responded21:50
clarkbfungi: it should remove the section entirely beacuse of the != undef check just above21:50
fungioh! thanks, i missed the if21:51
clarkbunless the != :undef is not hcecking what we want there21:51
clarkbits possible it would still fall through in that case21:51
*** betherly has quit IRC21:51
clarkbcmurphy: ^ do you have a sec to check that? https://review.openstack.org/#/c/641486/1/templates/storyboard.conf.erb21:51
fungioh, yeah i thought undef became nil in erb, and i guess we're specifically matching on :undef21:52
fungibut i've never seen that syntax before so maybe it's valid21:52
clarkbfungi: let me update it to be safe21:53
clarkbactually stackoverflow says the check for :undef is correct when doing scope.lookupvar21:54
clarkb$deity forbid we actually make this simple21:54
clarkbI don't understand how it was falling through before21:56
ianwclarkb: sorry, i didn't; i mean it's probably never a *bad* idea to get the snapshot before updating?21:57
clarkbianw: ya I am leaning towards doing it, I'll get started on stopping afs on the server so that I can make a snapshot21:58
clarkbbut first sorting out this puppet mystery21:58
*** e0ne has quit IRC21:58
*** jcoufal has quit IRC21:59
openstackgerritClark Boylan proposed openstack-infra/puppet-storyboard master: Define reply_to_email_address  https://review.openstack.org/64148621:59
clarkbfungi: ^ we check against nil in other places21:59
clarkband we shouldn't fall through if that :undef check was working so I suspect that check was not working22:00
*** e0ne has joined #openstack-infra22:01
fungithanks22:01
*** e0ne has quit IRC22:01
clarkbhrm what happens when you vos release and one of your secondary RO sites is down?22:02
clarkbdocs in particular has a secondary site on afs01.ord22:02
clarkband snapshotting isn't a fast process so chances are I will atke the service down for a bit22:03
clarkbdoes it just catch up again after the service is running again?22:04
*** apetrich has quit IRC22:08
clarkbvos_release manpage implies it handles it properly but you have to manually rerun vos release to do the catch up22:09
clarkbI think for docs we are likely ok there since we run the cron often22:09
clarkbit should just automatically catch back up again when it can22:09
clarkbinfra-root given ^ any objection to me disabling afs01.ord's afs services in order to make a snapshot?22:09
corvusclarkb: sounds good22:10
*** apetrich has joined #openstack-infra22:11
fungiclarkb: sounds like the safer path22:11
clarkbok I did a bos shutdown, then a service openafs-fileserver stop, then moved /etc/openafs/ to /etc/clarkb.moved.openafs and am running the snapshot now22:15
clarkbwhen snapshot is done I'll move /etc/openafs back, start openafs-fileserver, and the bos status and make sure its happy again22:15
*** betherly has joined #openstack-infra22:17
*** betherly has quit IRC22:22
*** markvoelker has quit IRC22:22
*** pabelanger has quit IRC22:32
*** auristor has quit IRC22:32
*** auristor has joined #openstack-infra22:33
openstackgerritMerged openstack-infra/puppet-storyboard master: Remove unused puppi dependency  https://review.openstack.org/64148522:35
*** anteaya has quit IRC22:37
clarkbsnapshot has finished and I've moved things back and started openafs-fileserver again. bos status says "currently running normally"22:40
clarkbI'm going to pop out for a few before I boot the snapshot but next step in my plan is to boot the snapshot then run the upgrade on it22:40
*** cmoura has quit IRC22:45
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Fix duplicate and reversed artifacts  https://review.openstack.org/64150822:46
*** rcernin has joined #openstack-infra22:52
*** wolverineav has quit IRC22:52
*** wolverineav has joined #openstack-infra22:58
*** mattw4 has quit IRC23:00
*** mattw4 has joined #openstack-infra23:01
*** tkajinam has joined #openstack-infra23:01
*** dave-mccowan has quit IRC23:06
openstackgerritClark Boylan proposed openstack-infra/system-config master: Test afs nodes on xenial  https://review.openstack.org/64151223:08
clarkbrealized I should double check ^ before I get too far down the path of doing upgrades23:09
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Allow soft job dependencies  https://review.openstack.org/64143923:10
clarkbtest server is booting off of the snapshot now23:11
*** wolverineav has quit IRC23:13
*** wolverineav has joined #openstack-infra23:17
*** markvoelker has joined #openstack-infra23:19
*** rlandy is now known as rlandy|bbl23:24
*** kjackal has quit IRC23:24
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Fix duplicate and reversed artifacts  https://review.openstack.org/64150823:28
*** ekultails has quit IRC23:29
clarkbianw dmsimard did we learn anything about booting snapshots? my booted snapshot pings but does not accept ssh connections23:30
ianwclarkb: yes, yes we did23:30
fungimay need to check its console23:30
ianwhttps://etherpad.openstack.org/p/bridge-snapshot-bringup23:31
ianwwhat i learnt, at least for bionic, was that what seems to happen is that the initial boot is cloud-init enabled, and nova-agent correctly writes out the config23:31
ianwbut then we remove cloud-init, and so subsequent boots will not pick up new networking23:31
ianwi think because rax's nova-agent .service files are all tied into cloud-init dependencies23:32
clarkbhrm this may be a reason to not uninstall cloud-init23:32
clarkbmordred: ^ fyi23:32
clarkbianw: step 3 requires web ui?23:33
mordredclarkb: I see that as another reason to start using our own base images23:33
mordredclarkb: but yeah23:33
clarkbmordred: well glean doesn't work in that case either23:33
ianwclarkb: afaik, yes, you need the webui to enter the rescue mode.  it gives you the root password23:33
clarkb(not on rax)23:33
mordredoh - this is for bringing up snapshot23:33
mordredbecause of new networking23:34
clarkbmordred: yes23:34
mordredsorry - I fixated on nova-agent first :)23:34
*** rascasoft has quit IRC23:34
clarkbI mean nova-agent/cloud-init are the actors on my snapshot23:34
clarkbbut we'd get the same behavior on our own images with glean23:34
mordredyeah- totes23:35
mordredclarkb: glean doesnt' run on subsequent boots because we write out a stamp file23:35
mordredright?23:35
* mordred is paging info back in23:35
ianwright, it won't overwrite the existing network files23:35
ianwthey could be cleared out, and it would be re-written ... which i assume cloud-init+nova-agent would do23:36
ianwthat was what i tried first23:36
ianwbut the netplan file was not re-written on boot ... ergo my thought that i think because no cloud-init, nova-agent doesn't correctly know to re-write it23:36
clarkbcloud-init checks the instance uuid against a value it writes23:37
clarkbif they don't match it assumes a new instance and redoes everything23:37
*** rascasoft has joined #openstack-infra23:37
mordredyeah. so - yeah, I guess as much as it pains me to say it - not deleting cloud-init might be the best choice for some of these23:37
ianwhrm, glean does not do that, but certainly could23:37
clarkbianw: ya we could update glean to be smarter23:37
ianwglean also has no support for netplan, which sounds like another pita thing to design for23:37
mordredwhat is netplan?23:38
mordredoh wow23:38
mordrednevermind23:38
mordredI've learned enough23:38
ianw... yeah ... :)  that was me a few days ago looking at this :)23:38
clarkbremember how systemd was going to solve all this?23:38
mordredit's config for the desktop networking systems that have gotten shoved in to server operating systems23:39
clarkbfwiw I think trusty is too old for me to deal with netplan but everything else is likely applicable23:39
* mordred cries23:39
ianwi don't know how to deal with the ephemeral drive thing23:42
fungiwow, netplan looks like ifupdown configuration in yaml with a versioned schema23:42
ianwwhile nofail in the fstab means the system boots, it also means the snapshot has a ephmeral drive without the disk layout it expects23:42
*** rh-jelabarre has quit IRC23:43
fungiwell, sure, it's "ephemeral" after all23:43
ianwi guess, with enough effort, a .service file could be written to do the partitioning if it didn't exist before mounts23:43
clarkbin the case of old trusty machine I need to copy /etc/network/interfaces and not the netplan stuff23:44
clarkbbut same idea23:44
clarkbhuh though it looks like that may already have been done and my only issue then was trying to mount things that well weren't going to mount23:45
ianw"# Cloud-inti supports the creation of simple partition tables and files systems on disks"23:45
ianwof course it does23:45
clarkbya that was actually what precipitated our move away from cloud-init23:45
fungicloud-init will floss your dog and flense your speen23:45
clarkbhpcloud had decided they knew how to partition our disks and iirc partitioned over a disk with contnet23:46
clarkbvia cloud-init config23:46
clarkbianw note that rebooting doesn't seem to exit rescue mode. I've had to click the button in the webui and am trying again23:48
ianwbut yeah, on bridge.o.o -- i feel like dmsimard's resize plan is probably good, and hope for the best that it all just works -- but if it doesn't, it's not so hard to recover23:48
fungianother thing i've discovered about rescue mode is that it will cause any userdata script you originally supplied for the instance to be executed again on entering rescue mode... likely doesn't impact our use though23:53
*** markvoelker has quit IRC23:53

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