Thursday, 2019-02-28

prometheanfirethanks, that works00:00
clarkbcorvus: fwiw this being the first ever tiem I've read that code your change seems to make sense. The redirect is installed when transfering the repo and not something dynamically sorted out on every request right?00:02
*** jamesmcarthur has joined #openstack-infra00:02
clarkbcorvus: in the transfer ownership code you install a redirect (into something the db?) and then whatever web hadnling code happens on the requests will look that up00:02
corvusclarkb: right, that makes a row in the db.  somewhere there's a web router that hits a point that says "there's no repo here, let's see if there's a redirect object here" and pulls that row out of the db.00:03
clarkbthen ya lgtm00:03
*** sdake has joined #openstack-infra00:04
corvusclarkb: here's the db model defn: https://github.com/go-gitea/gitea/blob/master/models/repo_redirect.go#L10-L1500:04
prometheanfireclarkb: fungi corvus https://review.openstack.org/63986800:04
prometheanfireI'm going to be afk, but if it's passing feel free to merge00:04
prometheanfiretonyb should be around too00:04
corvusclarkb: the lucky/well-designed thing is that it references the target repo by id, which means that even though it's currently only used when the target repo is in the same org, it shouldn't necessarily be restricted to that.00:05
clarkbcorvus: ya00:05
corvus(if that were a string instead of a repo id, i was just going to file a bug and call it a day :)00:06
clarkbprometheanfire: I've +2'd can let tonyb be second reviewer and hopefully approve00:06
prometheanfirekk00:06
prometheanfiremay need backports, we'll see there I guess00:06
*** dave-mccowan has quit IRC00:07
*** anteaya has quit IRC00:07
clarkbthat chagne merging should trigger post jobs and we'll know quickl.y My guess is it will work for master changes at that point but then we still want backports for stable merges00:07
corvusprometheanfire: will probably need a backport for each branch you want to fix publishing00:07
corvusclarkb: yeah that.00:07
prometheanfirecorvus: feel free if needed00:07
clarkbmordred: thinking out loud a bit about that image update thing, we may want to suggest to mnaser that they update their bionic image to a version that has that netplan stuff instaleld so that other users don't have to work around it too?00:09
ianwbridge.o.o : 1.4G Feb 28 00:05 puppet_run_cloud_launcher.log ... should probably rotate that00:09
clarkbmnaser: ^ tl;dr is current vexxhost bionic images fail to update packages because of a package thing with netplan. I expect htat newer bionic images have this sorted already00:09
clarkbianw: I thought we did rotate that00:09
clarkbianw: maybe that was only in the puppet nad didn't get ansibled00:10
ianwclarkb: yeah, will look when i find out why graphite's not puppeting :)00:10
ianw   - ze08.openstack.org # clarkb is testing jemalloc00:11
ianw .. did i misremember but didn't we do this on all hosts?00:11
clarkbfungi pushed a chagne to do it on all hosts, did that merge? if so we can remoev that00:11
* clarkb checks00:11
clarkbhttps://review.openstack.org/#/c/638044/ not merged. I will review it now00:11
fungithanks!00:12
fungii have plenty of changes which aren't merged, but i expect that goes for all of us00:12
clarkbfungi: ianw I just approved it after double checking the new defaults file contents match what I put in place on ze0800:13
*** mriedem has quit IRC00:13
fungiawesome00:13
fungipresumably we'll need executor restarts for it to take effect00:14
clarkbyup00:14
ianwclarkb: oh you know what, i think that's an old cloud_launcher file, it's all in /var/log/ansible which we do rotate00:14
clarkbianw: ah00:14
* tonyb cap in hand ... can someone approve my message to openstack-discuss 'Too many recipients to the message' #oops00:14
clarkbfungi: I expect taht zuul will end up wanting another one of those soon as I think people want a release as soon as we base64 encode the commit message in the job vars00:14
fungiyup00:15
clarkbtonyb: how do we know you aren't spamming:P00:15
fungitonyb: i already did00:15
clarkbwith mailman not deduping I've been trying to remove those extra recipients00:15
fungitonyb: if your mua has a reply-to-list option, i recommend it. i couldn't get by without mine00:15
prometheanfireL00:16
tonybfungi: Ahh thanks00:16
ianwFAILED! => {... "msg": "puppet did not run", ...} ... thanks ansible :)00:16
clarkbianw: check syslog on the target host00:16
clarkbianw: I like sudo grep puppet-user /var/log/syslog00:16
tonybfungi: I humharred about it but figured it was worth going to all the lists00:16
clarkbthis doesn't work on centos because its puppet-something-else iirc00:17
tonybfungi: which was probably a mistake00:17
*** jamesmcarthur has quit IRC00:17
fungiyeah, i trimmed the recipient list all the way down to just openstack-discuss on my reply, for good measure00:17
ianwclarkb: yeah ... and doh i didn't update the heira vals, so that explains that00:17
clarkbianw: is it safe for me to edit the emergency file? I don't want ot stomp on your changes. BUt i can remove kdc01 from that file now and we can remoev ze08 as soon as fungi's change merges00:18
clarkbianw: oh I ran into that with health too00:18
*** jamesmcarthur has joined #openstack-infra00:18
fungii doubt the starlingx, kata, airship and zuul discussion lists care about this ongoing row00:18
clarkbwhich reminds me I can also delete the old host_vars/health.openstack.rog as soon as you aren't editing things00:18
ianwclarkb: yeah, i'm not using it, thanks00:18
clarkbok updating emergency now00:18
fungiclarkb: did i have changes in the emergency file?00:18
clarkbdone with emergency, let me know when I can edit ansible vars00:18
clarkbfungi: ya old openstackid.org is in there00:19
clarkbfungi: I am done with the file if you want to remove that00:19
fungioh, i'll remember to clean that up as soon as i delete the server itself00:19
*** mriedem has joined #openstack-infra00:19
fungiwanted to be sure smarcet had a chance to copy anything he might have left on it00:19
clarkbsounds good00:20
ianwclarkb: ok, i just made a copy and similar will remove the old one when it's retired00:20
mnaserlarsks: are you telling me that uh00:20
mnasererr sorry for that highlight ^00:20
clarkbianw: I'm good to make my changes now then?00:20
ianwclarkb: yep, i'm done for now, thanks00:20
*** rascasoft has joined #openstack-infra00:20
mnaserclarkb: are you telling me that ubuntu 18.04 images magically started shipping with netplan at some arbitrary point?00:20
fungiand in an un-upgradeable state, it seems like00:21
clarkbmnaser: aiui netplan is a dep of network config things00:21
clarkbmnaser: so yes I expect that current bionic images will ship with an up to date an upgradeable netplan as they are past the point of needing interaction00:21
clarkbI haven't confirmed this though00:21
mnaserhttps://i.ytimg.com/vi/xz3PbpPR6DY/hqdefault.jpg00:22
*** jamesmcarthur has quit IRC00:22
clarkbI think it was added as a dep to a preexisting package?00:23
clarkbsomething like that which forces you to say yes its ok to install00:23
fungiright, it needs apt full-upgrade (formerly apt or apt-get dist-upgrade which is more recently deprecated)00:24
*** rascasoft has quit IRC00:27
openstackgerritMerged openstack/diskimage-builder master: Update gentoo-releng gpg key  https://review.openstack.org/63983300:33
openstackgerritClark Boylan proposed openstack-infra/zuul-website master: Add a promotional message banner and events list  https://review.openstack.org/63987100:35
openstackgerritMerged openstack-infra/puppet-zuul master: Use jemalloc on Zuul v3 executors  https://review.openstack.org/63804400:40
ianwoh my god, my cable/pole situation has gone from mildly annoying to kafkaesque farce00:40
ianwdue to the "national broadband network" taking over the coax cable outside my house at some unspecified future point, the telecommunications company who presently owns and maintains it won't work on it00:42
clarkbianw: I think you need a Grade 34 bureaucrat00:43
ianwsomebody could come and plug in, run and terminate the cable to the house in 15 minutes and be done.  but now there are at least 4 people somehow involved, apart from the people they have already physically sent out00:43
ianwwho weren't aware of the situation, but weren't qualified to work on the pole00:44
*** ijw has joined #openstack-infra00:47
*** agopi has joined #openstack-infra00:48
fungiget harry tuttle. he doesn't care if you've filed a twenty-seven b stroke six00:57
*** sdake has quit IRC01:08
ianw++01:11
*** yamamoto has quit IRC01:11
ianw"[remote 1.43.155.180:27803] ImportError: No module named whisper ..." ... guess graphite puppet needs some debugging01:15
*** ijw has quit IRC01:15
ianwailed to call refresh: Could not find command 'pip' ... interesting, this might be similar to clarkb's issues01:16
*** betherly has joined #openstack-infra01:17
*** ijw has joined #openstack-infra01:17
clarkbianw: my low tech fix was to delete the repos and let next puppet run find the pip that was indtalled01:18
clarkband my complictaed real fix is pushed to the puppet health repo01:18
ianwgcc is also not installed01:20
ianwhow do the rspec test work :/01:21
*** betherly has quit IRC01:22
ianwso, breaking it down, first issue is that whisper install doesn't depend on the install::pip -> https://git.openstack.org/cgit/openstack-infra/puppet-graphite/tree/manifests/init.pp#n10101:23
*** hwoarang has quit IRC01:23
ianwthat "install ::pip" doesn't bring in build-essential is interesting01:25
clarkbrspec worls because we install that stuff in our images aiui01:26
*** hwoarang has joined #openstack-infra01:29
tonybcorvus, clarkb: for the record setting CR=0,W=0 didn't work on https://review.openstack.org/#/c/639011/6 gerrit stilled -2'd the new patchset01:32
openstackgerritIan Wienand proposed openstack-infra/puppet-graphite master: Fix pip ordering and build requirements  https://review.openstack.org/63987701:33
ianwclarkb: yeah, two points that we miss in testing of course hit in production :)01:34
fungiit should be a fundamental law of ci01:35
*** gyee has quit IRC01:38
clarkbtonyb that is quite weird I wonder what happens if someone else pushes a patch01:41
tonybclarkb: Yeah01:42
* tonyb tries to re-create it in the sandox01:42
tonybsandbox even01:42
*** yamamoto has joined #openstack-infra01:47
tonybclarkb: Yup https://review.openstack.org/#/c/639880/ is doing the same thing01:52
*** yamamoto has quit IRC01:52
tonybclarkb, corvus: it does have soemthing to do with switching from 0,0 -> -2,-1 -> 0,-101:52
tonybclarkb: clarkb I messed up https://review.openstack.org/#/c/639880/ and it doesn't do it01:53
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: Add API endpoint to get frozen jobs  https://review.openstack.org/60707701:59
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: Get executor job params  https://review.openstack.org/60707801:59
*** rascasoft has joined #openstack-infra02:00
*** sdake has joined #openstack-infra02:09
*** betherly has joined #openstack-infra02:10
*** rascasoft has quit IRC02:11
*** betherly has quit IRC02:14
*** yamamoto has joined #openstack-infra02:27
*** roman_g has quit IRC02:28
*** bhavikdbavishi has joined #openstack-infra02:29
*** bhavikdbavishi has quit IRC02:31
*** hongbin has joined #openstack-infra02:32
*** yamamoto has quit IRC02:39
*** rlandy|bbl is now known as rlandy02:45
*** sdake has quit IRC02:46
openstackgerritIan Wienand proposed openstack-infra/puppet-graphite master: Fix pip ordering and build requirements  https://review.openstack.org/63987702:46
hogepodgefungi: I can send a patch up to disable loci jobs except for requirements. We're not doing functional testing of loci containers so we don't even know if the artifacts work (well, those running in production know but it's not the same as a gate job to be sure)02:55
fungihogepodge: do you happen to know which projects it was running on? the alternative i think is to move the job definition and playbooks into the openstack-infra/project-config repo02:56
fungithough that may still require a job rename dance touching all those repos since zuul won't let us remove an in-use job nor create a non-variant job with the same name02:57
*** psachin has joined #openstack-infra03:02
*** wolverineav has quit IRC03:06
*** dave-mccowan has joined #openstack-infra03:06
tonybhogepodge: https://review.openstack.org/#/c/639868/ we're in the process of removing it from requirements ATM happy to put it back when we can see it working03:10
*** ijw has quit IRC03:13
*** ijw_ has joined #openstack-infra03:13
*** udesale has joined #openstack-infra03:15
*** yamamoto has joined #openstack-infra03:15
*** apetrich has quit IRC03:15
hogepodgetonyb: I've been afk for a bit tonight, so I need to catch up on exactly what's happening.03:16
clarkbhogepodge: its the same thing as with the cinder image build job03:16
hogepodgerequirements is the one job we actually do want to run and publish since everything else depends on it03:16
clarkbjust a different job03:17
hogepodgeis there a writeup of the issue or a link to where the discussion happened in IRC?03:17
clarkbprometheanfire: made a bug let me find a link03:18
clarkbhogepodge: https://storyboard.openstack.org/#!/story/200510803:19
hogepodgeOk, so if I understand, we won't automatically trigger a loci-requirements build with the requirements repo updates, but our regular loci requirements build is ok?03:19
tonybhogepodge: I believe that is correct03:19
hogepodgethis kind of fits in with how I want to do builds anyway, rather than segment across projects I'd rather segment across distros and add functional testing into the mix also somehow.03:20
fungiyes, running it on loci changes is fine03:20
hogepodgeI'm cool with all those changes03:20
clarkb yes and we werent triggering it anyway for just over a month03:20
clarkbsince the zuul security fix went in03:20
hogepodgethat explains the sense of... emptiness I was feeling ;-) (kidding)03:20
*** sdake has joined #openstack-infra03:22
*** sdake has quit IRC03:23
*** bhavikdbavishi has joined #openstack-infra03:24
*** takamatsu has quit IRC03:25
*** sdake has joined #openstack-infra03:27
*** dave-mccowan has quit IRC03:32
*** mriedem has quit IRC03:32
*** ijw_ has quit IRC03:38
*** sdake has quit IRC03:40
*** wolverineav has joined #openstack-infra03:40
*** wolverineav has quit IRC03:47
*** markvoelker has joined #openstack-infra03:48
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: WIP: add job filter to trigger event  https://review.openstack.org/63990503:49
*** rascasoft has joined #openstack-infra03:58
openstackgerritIan Wienand proposed openstack-infra/puppet-graphite master: Fix pip ordering and build requirements  https://review.openstack.org/63987704:08
*** wolverineav has joined #openstack-infra04:08
*** rascasoft has quit IRC04:08
*** spsurya has joined #openstack-infra04:14
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Get executor job params  https://review.openstack.org/60707804:16
*** wolverineav has quit IRC04:17
*** yamamoto has quit IRC04:19
*** ykarel|away has joined #openstack-infra04:19
*** ykarel|away is now known as ykarel04:20
*** markvoelker has quit IRC04:20
*** betherly has joined #openstack-infra04:26
*** betherly has quit IRC04:31
*** eernst has joined #openstack-infra04:42
*** rlandy has quit IRC04:50
tonyb:qa!04:56
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add initial driver and event  https://review.openstack.org/55515305:00
*** armax has quit IRC05:00
*** raukadah has quit IRC05:03
*** chandankumar has joined #openstack-infra05:04
*** takamatsu has joined #openstack-infra05:05
*** takamatsu has quit IRC05:10
*** wolverineav has joined #openstack-infra05:14
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add web route and rpclistener  https://review.openstack.org/55483905:15
*** hongbin has quit IRC05:16
*** markvoelker has joined #openstack-infra05:17
*** wolverineav has quit IRC05:19
*** sdake has joined #openstack-infra05:30
*** rascasoft has joined #openstack-infra05:43
*** ramishra has joined #openstack-infra05:45
*** yamamoto has joined #openstack-infra05:46
*** rascasoft has quit IRC05:47
*** markvoelker has quit IRC05:51
*** psachin has quit IRC05:58
*** psachin has joined #openstack-infra06:00
*** ramishra has quit IRC06:01
*** sdake has quit IRC06:02
*** ramishra has joined #openstack-infra06:08
*** janki has joined #openstack-infra06:15
*** yamamoto has quit IRC06:19
*** bhavikdbavishi has quit IRC06:19
*** bhavikdbavishi has joined #openstack-infra06:20
*** rascasoft has joined #openstack-infra06:20
*** yamamoto has joined #openstack-infra06:22
*** sdake has joined #openstack-infra06:22
*** gregoryo has joined #openstack-infra06:24
*** sdake has quit IRC06:26
*** rascasoft has quit IRC06:32
*** ijw has joined #openstack-infra06:32
*** sdake has joined #openstack-infra06:42
*** markvoelker has joined #openstack-infra06:48
openstackgerritMerged openstack-infra/grafyaml master: Add capability to use grafana folderid parameter  https://review.openstack.org/63752006:54
*** eernst has quit IRC07:00
*** tkajinam_ has joined #openstack-infra07:02
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add build button to trigger job  https://review.openstack.org/63571607:04
*** tkajinam has quit IRC07:05
*** dpawlik has joined #openstack-infra07:08
openstackgerritMerged openstack-infra/zuul-jobs master: Explanatory comment for Editable Git install  https://review.openstack.org/63964907:09
*** mandre is now known as mandre_away07:09
*** rascasoft has joined #openstack-infra07:09
*** apetrich has joined #openstack-infra07:12
*** sdake has quit IRC07:15
*** markvoelker has quit IRC07:20
*** aojea has joined #openstack-infra07:25
*** ijw has quit IRC07:25
*** ijw has joined #openstack-infra07:25
prometheanfireclarkb: fungi ping me if backport needed, sleep now, nn07:28
*** sshnaidm|afk is now known as sshnaidm07:32
*** kopecmartin|off is now known as kopecmartin07:34
*** ijw has quit IRC07:34
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: switch jobs list to a tree view  https://review.openstack.org/63343707:41
*** quiquell is now known as quiquell|brb07:48
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add jobs list filter  https://review.openstack.org/63365207:48
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method  https://review.openstack.org/63365307:48
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add tags to jobs list  https://review.openstack.org/63365407:48
*** roman_g has joined #openstack-infra07:52
*** ccamacho has joined #openstack-infra07:59
*** jpich has joined #openstack-infra08:00
*** jtomasek has joined #openstack-infra08:02
*** ginopc has joined #openstack-infra08:13
*** quiquell|brb is now known as quiquell08:14
*** markvoelker has joined #openstack-infra08:18
zbrprometheanfire: hi! can you give a kick to https://review.openstack.org/#/c/635590/ ?08:27
*** pcaruana has joined #openstack-infra08:32
*** helenafm has joined #openstack-infra08:37
*** bhavikdbavishi has quit IRC08:38
*** tkajinam_ has quit IRC08:38
*** bhavikdbavishi has joined #openstack-infra08:38
*** tosky has joined #openstack-infra08:39
*** ykarel is now known as ykarel|lunch08:43
*** markvoelker has quit IRC08:50
*** jpena|off is now known as jpena08:54
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum and dnf atoms for platforms  https://review.openstack.org/63995108:58
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum and dnf atoms for platforms  https://review.openstack.org/63995109:00
*** gregoryo has quit IRC09:05
*** jaosorior has joined #openstack-infra09:06
*** iurygregory_mtg is now known as iurygregory09:06
*** electrofelix has joined #openstack-infra09:11
*** elod has quit IRC09:12
*** sdake has joined #openstack-infra09:15
*** olivierbourdon38 has joined #openstack-infra09:20
*** olivierbourdon38 has quit IRC09:22
*** iurygregory has quit IRC09:24
*** iurygregory has joined #openstack-infra09:28
*** udesale has quit IRC09:28
*** udesale has joined #openstack-infra09:29
*** e0ne has joined #openstack-infra09:32
*** derekh has joined #openstack-infra09:35
*** yboaron has joined #openstack-infra09:45
*** gfidente has joined #openstack-infra09:45
*** sdake has quit IRC09:46
*** markvoelker has joined #openstack-infra09:47
*** rcernin has quit IRC09:48
*** ykarel|lunch is now known as ykarel09:48
*** dtantsur|afk is now known as dtantsur09:49
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Adds task title to new story modal  https://review.openstack.org/63996209:50
*** panda|ruck|off is now known as panda|ruck09:51
openstackgerritM V P Nitesh proposed openstack/diskimage-builder master: Adding 'oel' as a new dib element  https://review.openstack.org/62550109:56
*** iurygregory has quit IRC10:03
*** yamamoto has quit IRC10:04
dulekHi folks! Do we happen to have infra mirrors of k8s builds?10:06
dulekI'm working on running upstream k8s tests on kuryr-k8s gates.10:06
*** yboaron_ has joined #openstack-infra10:09
*** iurygregory has joined #openstack-infra10:09
*** elod has joined #openstack-infra10:10
*** yboaron has quit IRC10:12
*** markvoelker has quit IRC10:21
*** udesale has quit IRC10:24
*** udesale has joined #openstack-infra10:24
*** sdake has joined #openstack-infra10:37
*** luizbag has joined #openstack-infra10:38
*** yamamoto has joined #openstack-infra10:38
*** iurygregory has quit IRC10:40
*** iurygregory has joined #openstack-infra10:41
*** wolverineav has joined #openstack-infra10:42
*** iurygregory has quit IRC10:42
*** iurygregory has joined #openstack-infra10:42
*** ccamacho has quit IRC10:43
*** yamamoto has quit IRC10:43
*** wolverineav has quit IRC10:46
*** cmoura has quit IRC10:47
*** ccamacho has joined #openstack-infra10:49
*** dpawlik has quit IRC11:00
openstackgerritMerged openstack-infra/nodepool master: Support requests for specific providers  https://review.openstack.org/63941811:02
*** bhavikdbavishi has quit IRC11:03
*** agopi has quit IRC11:06
*** mhu has joined #openstack-infra11:10
openstackgerritMerged openstack-infra/zuul master: Request child nodes of paused jobs at higher priority  https://review.openstack.org/63879611:10
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Add AuthorizationRule Configuration Object, claim validation  https://review.openstack.org/63985511:11
*** yboaron_ has quit IRC11:15
*** markvoelker has joined #openstack-infra11:18
*** sdake has quit IRC11:18
*** yboaron_ has joined #openstack-infra11:19
*** sdake has joined #openstack-infra11:20
openstackgerritM V P Nitesh proposed openstack/diskimage-builder master: Adding 'oel' as a new dib element  https://review.openstack.org/62550111:22
*** e0ne has quit IRC11:30
*** rossella_s has quit IRC11:31
*** rossella_s has joined #openstack-infra11:32
*** e0ne has joined #openstack-infra11:36
*** jpich has quit IRC11:47
*** markvoelker has quit IRC11:50
openstackgerritMerged openstack-infra/bindep master: Adds job to test bindep on Fedora  https://review.openstack.org/63559011:51
*** dpawlik has joined #openstack-infra11:52
*** sdake has quit IRC11:55
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum, dnf and zypper atoms for platforms  https://review.openstack.org/63995111:57
*** gfidente has quit IRC12:07
*** bhavikdbavishi has joined #openstack-infra12:12
*** yboaron_ has quit IRC12:14
*** bhavikdbavishi has quit IRC12:14
*** bhavikdbavishi has joined #openstack-infra12:14
*** yamamoto has joined #openstack-infra12:18
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum, dnf and zypper atoms for platforms  https://review.openstack.org/63995112:18
*** bhavikdbavishi1 has joined #openstack-infra12:19
*** bhavikdbavishi has quit IRC12:19
*** bhavikdbavishi1 is now known as bhavikdbavishi12:19
*** yamamoto has quit IRC12:22
*** gfidente has joined #openstack-infra12:31
*** ccamacho has quit IRC12:38
*** jpena is now known as jpena|lunch12:41
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum, dnf and zypper atoms for platforms  https://review.openstack.org/63995112:41
*** ramishra_ has joined #openstack-infra12:41
*** ramishra has quit IRC12:44
*** panda|ruck is now known as panda|ruck|lunch12:44
*** markvoelker has joined #openstack-infra12:47
*** rh-jelabarre has joined #openstack-infra12:52
*** yamamoto has joined #openstack-infra12:58
*** trown|outtypewww is now known as trown12:58
*** luizbag has quit IRC12:58
*** yamamoto has quit IRC13:03
*** ccamacho has joined #openstack-infra13:06
*** dave-mccowan has joined #openstack-infra13:10
*** yamamoto has joined #openstack-infra13:11
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Add AuthorizationRule Configuration Object, claim validation  https://review.openstack.org/63985513:14
*** luizbag has joined #openstack-infra13:16
*** jamesmcarthur has joined #openstack-infra13:16
*** rtjure has joined #openstack-infra13:17
*** markvoelker has quit IRC13:20
*** jamesmcarthur has quit IRC13:23
*** rossella_s has quit IRC13:24
*** rossella_s has joined #openstack-infra13:24
*** jamesmcarthur has joined #openstack-infra13:26
*** yboaron_ has joined #openstack-infra13:26
guilhermesphey there! I've asked a while ago a hold on https://review.openstack.org/#/c/627782/ which has 23.253.218.253 ip13:29
guilhermespjust to let you know it can be destroyed13:29
*** panda|ruck|lunch is now known as panda|ruck13:30
*** jamesmcarthur has quit IRC13:31
*** rh-jelabarre has quit IRC13:31
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Update dns message to make sense for opendev  https://review.openstack.org/64002613:34
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002713:34
*** rlandy has joined #openstack-infra13:35
*** quiquell is now known as quiquell|lunch13:38
*** Shrews_ is now known as Shrews13:43
*** jpena|lunch is now known as jpena13:43
fungithanks guilhermesp! i've released it back to the pool for deletion13:45
*** jamesmcarthur has joined #openstack-infra13:46
*** yamamoto has quit IRC13:46
*** yamamoto has joined #openstack-infra13:48
*** yamamoto has quit IRC13:48
*** yamamoto has joined #openstack-infra13:49
*** dpawlik has quit IRC13:51
*** quiquell|lunch is now known as quiquell13:52
*** yamamoto has quit IRC13:53
*** rfolco|rover is now known as rfolco|afk13:54
*** ekultails has joined #openstack-infra14:01
*** betherly has joined #openstack-infra14:02
*** jamesmcarthur has quit IRC14:04
*** yamamoto has joined #openstack-infra14:04
*** yamamoto has quit IRC14:04
*** yamamoto has joined #openstack-infra14:04
*** jamesmcarthur has joined #openstack-infra14:05
*** jamesmcarthur has quit IRC14:10
*** jistr is now known as jistr|call14:11
*** mriedem has joined #openstack-infra14:13
*** dpawlik has joined #openstack-infra14:14
*** ykarel has quit IRC14:15
*** ykarel has joined #openstack-infra14:15
zbrAJaeger: finished implementing your bindep comments on https://review.openstack.org/#/c/639951/ -- looks ok now?14:16
*** markvoelker has joined #openstack-infra14:18
*** wolverineav has joined #openstack-infra14:18
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum, dnf and zypper atoms for platforms  https://review.openstack.org/63995114:18
*** yboaron_ has quit IRC14:19
*** yboaron_ has joined #openstack-infra14:19
*** ianychoi has joined #openstack-infra14:20
*** yboaron_ has quit IRC14:21
*** yboaron_ has joined #openstack-infra14:21
*** wolverineav has quit IRC14:22
*** bogdando has joined #openstack-infra14:23
bogdandohi folks14:25
bogdandocorvus: I know your time was spent enough on the ci pipelines topic, but some updates and numbers I posted probably worth reading and re-evaluating14:26
bogdandohttp://lists.openstack.org/pipermail/openstack-discuss/2019-February/003336.html14:26
bogdandoand a few more on top of that14:27
bogdandothanks14:27
bogdandoclarkb: ^^14:27
*** bogdando has left #openstack-infra14:29
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Build ubuntu bionic minimal images for control plane  https://review.openstack.org/64004414:30
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Build ubuntu bionic minimal images for control plane  https://review.openstack.org/64004414:30
mordredinfra-root: I pushed up a couple of patches ^^ that contain a radical approach - namely, using nodepool to build minimal base images for our control plane14:31
pabelangermordred: I'm behind that, last we discussed (many years ago) there was some concerns around nodepool.o.o using passwords to control plane. At one point we might have discussed a 2nd nodepool just for that. However, +1 from me for doing it14:32
fungimordred: having consistent images across providers for our persistent servers sounds great, though i do wonder what fun gotchas we'll encounter once we're using them for systems we don't delete in an hour14:32
mordredinfra-root: this was spurred by 2 things - a) we keep talking about eventually building our own images b) I spent all of yesterday trying to boot a new server and kept failing on clarkb's unattended-upgrades - no matter what I tried something kept bombing out. I manually built a minimal base image and uploaded it to see if that worked better - and it worked GREAT14:32
mordredpabelanger: yeah - I thnik the thing that clicked for me is that the now the builders are separate from the launchers and have a separate config, so we can pretty safely configure only the builders to know about the control plane14:33
mordredfungi: ++ ... if we are ok with this- I'd like to update launch-node to know how to look for an image by its logical name assuming it's been uploaded by nodepool - so basically get the image list and look for the newest image prefixed with the requested name14:34
mordredfungi: since step 1 of using launch-node today is "openstack image list" to find what image is the right one to use14:34
fungimore wondering, e.g., where our model for things like glean may fall down for long-lived systems14:35
mordredfungi: the nice thing is that it means that when we boot a server it'll have the latest packages already14:35
mordredfungi: yeah - although I think as long as glean survives reboots we should be fine (and I think we already require this)14:36
fungiit's also the case that we can set up access controls so that the credentials nodepool uses can only upload and delete images, the actual security risks i think come in where a compromised nodepool builder is able to compromise the images we use to boot any new server14:37
*** jamesmcarthur has joined #openstack-infra14:38
mordredfungi: I'm not sure we have that ability actually -at least not across all of our clouds14:38
fungioh, good point, i may be conflating rackspace's account access controls with openstack's14:39
mordredyeah. heaven forbid we have such acl support in openstack :)14:40
mordredto be fair, the app-creds work aims to provide such a thign in a sane manner14:40
cmurphyit's this close -->||<--14:42
mordredinfra-root: in any case, we don't specifically have to agree to that approach today - but at the very least I'd like either an ack from folks on using manually uploaded minimal base images, because otherwise I'm pretty dead in the water with the gitea servers14:42
mordredor - I said either - I guess the or would be "or would love to hear an alternate suggestion for getting unstuck"14:43
*** bhavikdbavishi has quit IRC14:43
*** bhavikdbavishi has joined #openstack-infra14:44
*** jamesmcarthur has quit IRC14:44
mordredalthough I guess if we do decide we want to do that, I should expand those patches to include the control plane accounts on all of our clouds14:46
*** kgiusti has joined #openstack-infra14:46
mordredalso - my manually built image used the elements listed in that definition14:48
mordredfungi, pabelanger: https://review.openstack.org/#/c/640026/ should be a safe review to land14:48
*** markvoelker has quit IRC14:50
*** lujinluo has joined #openstack-infra14:50
fungimordred: i'm certainly cool with us uploading images for our servers to get consistency, whether we build them ourselves or fetch them from an image provider14:50
mordredcool. if we're not ok with using nodepool builder - we could make a script that could be run manually to ensure we get consistently built images14:51
fungii even do that for my personal servers because i don't trust clouds to have the exact same build of debian image, even if they all download their debian images from the same place14:52
mordredfungi: ++14:52
fungi(i don't build them myself... i could but my needs aren't quite that unusual)14:52
*** rh-jelabarre has joined #openstack-infra14:54
*** lujinluo has quit IRC14:55
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002715:00
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Clean up boot-from-volume volumes on error  https://review.openstack.org/64005415:00
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Change the default for config-drive to True  https://review.openstack.org/64005515:00
mordredinfra-root: https://review.openstack.org/#/q/topic:infra-base-images updated a few things and made a topic15:02
*** rossella_ has joined #openstack-infra15:06
*** rossella_s has quit IRC15:06
*** gfidente has quit IRC15:06
mordredfungi, clarkb, corvus, pabelanger: while I'm bugging people - in zone-opendev.org we seem to be half aligning rocketships and half not. I do not care at all which thing we do - but should we pick one? if we pick one - I think I'd vote for not aligning rocketships15:07
fungimordred: i think it depends on what your tabstop is set to15:09
fungisince it's all hard tabs15:09
fungibut we could certainly just treat it as tab-separated fields rather than tab-aligned fields15:10
mordredfungi: yeah - some of the lines seem to just be tab-separated, some seem to be tab aligned - but even the tab aligned have chosen different arbitrary alignments15:13
mordredfungi: I'll leave it for now, but maybe do a cleanup after the gitea servers go in15:14
fungidoesn't surprise me15:14
fungiif we do decide it's important, then we should make a simple linter15:14
fungithough there's also nslint i used to use when i had a bunch of operators managing hundreds of bind zonefiles15:14
mordredyeah. I mean - I mostly don't think it's important - but thought I'd ask :)15:15
*** e0ne has quit IRC15:16
*** jamesmcarthur has joined #openstack-infra15:18
*** e0ne has joined #openstack-infra15:19
*** jamesmcarthur has quit IRC15:22
*** amorin has joined #openstack-infra15:23
corvusmordred: generally it's tab-aligned, but some of the names got long and stick out a bit.  i'd be fine rounding up to the next tab so everything is aligned again.15:27
mordredcorvus: nod. so the intent was tab-aligned15:27
corvusmordred: i think uploading a base image is fine.  i also like the idea of auto-building them via some method in general.  but i haven't looked at your patches yet.15:29
*** gfidente has joined #openstack-infra15:29
mordredcorvus: cool. so - for now I'm booting the gitea servers using a manually built and uploaded image - but the image was built using the same elements as the proposed patches, so if we decide to move forward with that they shouldn't be terribly divergent15:30
*** pgaxatte has quit IRC15:33
*** kashyap has joined #openstack-infra15:36
corvusmordred: re boot from volume -- is there not an argument to tell it to auto delete volumes when the server is deleted?15:39
corvusmordred: (i thought both that there was, and we were using it)15:40
mordredcorvus: no - there is for ips, and there is a way to tell openstack when you _create_ a root volume via boot-from-volume to delete the volume when the server is deleted15:40
*** yboaron_ has quit IRC15:40
mordredbut we don't set that flag because it would mean we wouldn't be able to delete the server keeping the root volume and boot a new server using it as a root volume15:41
*** yboaron_ has joined #openstack-infra15:41
corvusoh.  i don't recall ever doing that, but i could see that being useful to "resize" a server...15:42
*** jamesmcarthur has joined #openstack-infra15:42
mordredyeah15:42
corvusmordred: i'm +2 on your first 2 patches15:42
mordred\o/15:43
*** jamesmcarthur has quit IRC15:46
*** armax has joined #openstack-infra15:46
*** jamesmcarthur has joined #openstack-infra15:47
*** markvoelker has joined #openstack-infra15:47
*** janki has quit IRC15:52
clarkbI'm not opposed to building our own images though we may not want to use the nodepool builders for that? Also some have recently suggested we move away from custom images entirely which would be complicated by doubling down on building our own images for the control plane (so we should consider if going back to upstream images for testing is actually something to consider)15:54
*** dpawlik has quit IRC15:55
corvusclarkb: heh, i'm in favor of *simplifying* our nodepool images, but i'm not in favor of going to upstream images.  we've been there.  we wrote an entire program because of how bad that was.15:56
clarkbAnother thing to consider if we use glean in the control plane is that we only support static ipv6 on debuntu and gentoo iirc. So there will be work there to get non debuntu control plane servers running in various regions with ipv615:56
corvusin fact, i mean, we're talking about using that program now because upstream images don't even work for a *regular server* much less a ci system :)15:56
clarkbcorvus: ya thats sort of where I sit today too. Also we publish our custom images to make it easier for people to reconsume them for local reproduction (which is the case that I think started the discussion from dmsimard)15:56
*** ykarel is now known as ykarel|away15:58
mordredclarkb: we don't really have any need of non-debuntu servers in control plane currently, do we?15:59
mordredclarkb: and if we did, updating glean to properly support them wouldn't be terrible15:59
*** jistr|call is now known as jistr16:00
mordredalso - I am also not in favor of ditching custom images. and by not in favor, I mean categorically opposed16:00
clarkbmordred: we do not, just pointing out the corner cases of the proposed plan16:00
mordredtotes16:00
*** armstrong has joined #openstack-infra16:00
mordred(I also agree that simplifying our nodepool images is not a bad idea)16:01
corvusmordred: i'm +1 on the nodepool plan, but sounds like we should wait for others to weigh in, and maybe consider alternatives if there are concerns.  but meanwhile, i left comments on how to fix the test job.16:02
mordredcorvus: sweet.I will read them16:03
*** rfolco|afk is now known as rfolco|rover16:10
*** udesale has quit IRC16:13
*** yamamoto_ has joined #openstack-infra16:15
clarkbI'm catching up on email (in particular the reorg test jobs stuff) but will review changes after16:15
*** e0ne has quit IRC16:16
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum, dnf and zypper atoms for platforms  https://review.openstack.org/63995116:17
*** yamamoto has quit IRC16:17
*** dave-mccowan has quit IRC16:17
*** SpamapS has quit IRC16:18
openstackgerritSorin Sbarnea proposed openstack-infra/bindep master: Adds yum, dnf and zypper atoms for platforms  https://review.openstack.org/63995116:18
*** bhavikdbavishi has quit IRC16:18
*** bhavikdbavishi has joined #openstack-infra16:19
*** wolverineav has joined #openstack-infra16:19
*** SpamapS has joined #openstack-infra16:19
*** quiquell is now known as quiquell|off16:20
*** markvoelker has quit IRC16:21
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Update dns message to make sense for opendev  https://review.openstack.org/64002616:22
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Clean up boot-from-volume volumes on error  https://review.openstack.org/64005416:22
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002716:22
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Change the default for config-drive to True  https://review.openstack.org/64005516:22
mordredcorvus: ^^ I had to fix a line in the first patch - then I fixed your review comments on the third16:22
*** e0ne has joined #openstack-infra16:23
*** gfidente has quit IRC16:23
*** quiquell|off is now known as quique16:23
*** quique is now known as quiquell|off16:23
*** wolverineav has quit IRC16:23
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884016:26
openstackgerritThierry Carrez proposed openstack-infra/project-config master: Use publish-to-pypi for instack-undercloud  https://review.openstack.org/63845916:27
*** IvensZambrano has joined #openstack-infra16:27
*** psachin has quit IRC16:28
*** dpawlik has joined #openstack-infra16:30
clarkbI have removed ze08 from the ansible emergency file16:30
clarkbonce caught up on mordred's changes I'm going to try and do a cleaning pass of our ansible vars/hieradata since I've got a bunch of that context paged in after doign server rebuilds16:30
openstackgerritThierry Carrez proposed openstack-infra/project-config master: Use publish-to-pypi for instack-undercloud  https://review.openstack.org/63845916:31
corvusclarkb: ooooh nice16:31
* mordred supports this activity from clarkb16:32
*** chandankumar is now known as raukadah16:34
*** dpawlik has quit IRC16:34
corvusi'm launching a zuul-preview server16:35
corvus(in rax)16:35
*** sthussey has joined #openstack-infra16:35
*** hamzy has quit IRC16:40
clarkbmordred: comment on https://review.openstack.org/#/c/640026/216:41
mordredclarkb: hrm. that's a good question. will those rdns command work in rax for a domain not hosted in their dns?16:43
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Adds task title to new story modal  https://review.openstack.org/63996216:43
clarkbmordred: for reverse dns, yes16:44
mordredclarkb: kk. yeah - let's do that - I'll do a followup patch16:44
fungii need to disappear for lunch now that all my meetings have wrapped up, but will be back in an hour or so16:45
*** bhavikdbavishi1 has joined #openstack-infra16:45
*** bhavikdbavishi has quit IRC16:46
*** bhavikdbavishi1 is now known as bhavikdbavishi16:46
corvusmordred, clarkb: what should we call the zuul preview server?  "preview.opendev.org" ?16:46
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: Adds task title to new story modal  https://review.openstack.org/63996216:47
clarkbcorvus: preview might be too generic? aritfact-preview.opendev.org ?16:47
corvusremember that the visible hostnames will be site.75031cad206c4014ad7a3387091d15ab.openstack.preview.opendev.org16:48
corvusso: site.75031cad206c4014ad7a3387091d15ab.openstack.artifact-preview.opendev.org16:48
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Print reverse dns instructions for rackspace servers  https://review.openstack.org/64009416:49
corvuscould also call it "zuul-preview".  that might be better marketing.  :)16:49
clarkbI'd be fine with that. Mostly thinking about how status.openstack.org caused confusion in dns the other day16:49
clarkbgranted we'll have review now so less of a concern16:49
clarkbmordred: I also left a note that config drive implies no live migration in older clouds. I don't think this is a big issue for us16:50
clarkbmordred: but may be unexpected side effect of that chagne for some16:51
*** lujinluo has joined #openstack-infra16:51
mordredclarkb: I don't expect other people who are not us to be using launch-node ... are they?16:51
clarkbmordred: no I think its mostly for our info16:52
mordredcoolio16:52
openstackgerritMonty Taylor proposed openstack-infra/zone-opendev.org master: Add gitea servers and load balancer  https://review.openstack.org/64009516:52
mordredclarkb, corvus: ^^ done booting gitea servers and lb16:52
*** helenafm has quit IRC16:53
corvusmordred: cool, we'll need one more change to add the gitea servers to the load balancer config; there should be a placeholder for them in system-config hostvars16:53
clarkbinfra-root I'm taking the hieradata "lock" now16:54
*** dtantsur is now known as dtantsur|afk16:55
mordredcorvus: yup. and also to add the servers to the inventory. that change is on its way16:55
corvusoh yeah, i forgot that for zp01. :)16:55
mordredyou know - we shoudl update launchnode to include a yaml snippet16:55
mordredwhen it prints things16:55
openstackgerritMerged openstack-infra/zuul master: Request dependent job nodes from paused parent provider  https://review.openstack.org/63946716:56
*** lujinluo has quit IRC16:56
corvusi thought someone did that?16:56
openstackgerritJean-Philippe Evrard proposed openstack-infra/zuul-jobs master: Add tag in upload image name  https://review.openstack.org/64009616:56
evrardjpcorvus: I am back16:56
evrardjpDo you think https://review.openstack.org/640096 is a reasonable approach ?16:57
evrardjpnot tested16:57
corvusclarkb, fungi: i thought someone wrote a change to launch-node to spit out an inventory blob but i can't find it.  is that ringing a bell?16:57
corvusevrardjp: https://review.openstack.org/63977916:58
clarkbcorvus: I think it was mordred?16:58
clarkbcorvus: I think it was a script that would write out a full inventory not just the delta16:58
evrardjpcorvus: haha sorry16:58
corvusevrardjp: np, and thanks :)16:59
*** betherly has quit IRC16:59
*** electrofelix has quit IRC16:59
corvusclarkb, mordred: huh.  i can't find any evidence of that.  must be a glitch in the matrix.17:00
evrardjpcorvus: different approach I see17:00
*** ginopc has quit IRC17:02
openstackgerritMerged openstack-infra/zone-opendev.org master: Add gitea servers and load balancer  https://review.openstack.org/64009517:02
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add gitea servers to inventory and load-balancer hostvars  https://review.openstack.org/64010017:02
corvusi unwisely decided to align the tabs right before mordred proposed his zone change :)17:04
mordredcorvus, clarkb: it's worth noting - given how we do hostnames now - and how we're doing dns ... we could stop putting the ip addresses into our inventory17:04
mordredit really COULD just be gitea01.opendev.org:17:04
mordredwith no additional info17:04
*** e0ne has quit IRC17:05
clarkbmordred: we actually use the ip addr as the connection iirc17:05
clarkbmordred: which is nice beause you can bootstrap things before changing IPs17:05
clarkber changing dns17:05
mordredgood point17:05
mordredkk. that's a good reason to keep it then :)17:05
clarkbI think if we continue to support that use case too then I would be fine with simplifying in general17:05
openstackgerritJames E. Blair proposed openstack-infra/zone-opendev.org master: Add zuul-preview  https://review.openstack.org/64010217:06
corvusmordred: did you mean to put tabs in gitea-lb.yaml?17:06
corvusmordred: of course not -- yaml forbids tabs!17:07
mordredugh17:07
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add gitea servers to inventory and load-balancer hostvars  https://review.openstack.org/64010017:08
mordredcorvus: silly me :)17:08
clarkbfungi: ttx: odsreg.openstack.org is not a service/server anymore right?17:09
openstackgerritMerged openstack-infra/zuul master: Fix multiple jobs with provides/requires  https://review.openstack.org/63947217:09
*** ykarel|away has quit IRC17:09
clarkbThere is no DNS A record for it and I would like ot clean out its hieradata file17:09
clarkbsimilarlry pholio01.openstack.org does not exist right?17:09
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Encode zuul.message with base64  https://review.openstack.org/63880117:09
clarkbinfra-root ^17:09
mordredclarkb: I believe pholio01.openstack.org is not a thing anymore17:10
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add zp01 to inventory  https://review.openstack.org/64011117:10
clarkband proposal.slave.openstack.org has been replaced with zuul secrets17:11
clarkbthis is fun doing spring cleaning17:11
clarkbDelete all the things17:11
corvusmordred: ah crap, sorry, i led you astray.  one more thing to fix in 64010017:12
mordredclarkb: so much delete. many many things17:12
mordredcorvus: on it17:12
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add gitea servers to inventory and load-balancer hostvars  https://review.openstack.org/64010017:13
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add zp01 to inventory  https://review.openstack.org/64011117:13
mordredcorvus: ^^ - and rebased yours17:13
corvusthx17:13
mordredcorvus, clarkb: feel like revoting on https://review.openstack.org/#/c/640026/2 (and or voting for the first time?)17:14
corvusclarkb: https://review.openstack.org/640102 https://review.openstack.org/640100 and https://review.openstack.org/640111 are ready for you now :)17:14
mordredcorvus: https://review.openstack.org/#/c/640054/2 had a zuul sad17:14
*** dpawlik has joined #openstack-infra17:15
corvusmordred: re 640026, you gonna do the reverse dns fix?17:15
corvusi think clarkb is right, we still want to run those17:15
clarkbcorvus: there was a followup change for that17:16
mordredcorvus: yes - I did a followup at the end of the stack17:16
mordredalthough I guess I could just squash them17:16
mordredcorvus: ImportError: No module named 'six'\n" was the zuul sad17:16
*** luizbag has quit IRC17:16
mordredwas that maybe just an unlucky blip of something updating?17:16
*** luizbag has joined #openstack-infra17:16
clarkbI've just rechecked https://review.openstack.org/#/c/639740/1 since cryptography has wheels again. That change adds secrets editing script to bridge so that we don't have the gpg-agent problem (as much)17:17
*** kopecmartin is now known as kopecmartin|off17:17
* clarkb goes back to cleaning out hieradata17:17
corvusmordred: yeah, seems likely.  maybe an ansible release :)17:17
openstackgerritAlex Litvinov proposed openstack-infra/project-config master: add upstream reference  https://review.openstack.org/64011817:17
mordred\o/17:17
corvusno ansible release.  maybe a zuul install.  weird.  doesn't usually do that.17:18
mordredyeah17:18
*** markvoelker has joined #openstack-infra17:18
clarkbfungi: ttx similarly is design-summit-prep.openstack.org replaced by the forum CFP thing?17:18
mordredI rechecked. we can keep our eyes out17:18
corvusmordred: +2s on the stack17:19
corvusmordred: though... tbh... i kinda  think you should squash it17:19
*** dpawlik has quit IRC17:19
mordredcorvus: ok. will do real qu-ick17:19
corvusmordred: it's a regression if the first change lands and it takes us a while to get the middle of the stack in17:19
corvus(and since the middle needs rechecking anyway)17:20
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Update dns message to make sense for opendev  https://review.openstack.org/64002617:20
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Clean up boot-from-volume volumes on error  https://review.openstack.org/64005417:20
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002717:20
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Change the default for config-drive to True  https://review.openstack.org/64005517:20
mordredclarkb, corvus: ^^ there's the squished version17:21
openstackgerritAlex Litvinov proposed openstack-infra/project-config master: add cinder-backup-swift charm and interface  https://review.openstack.org/63983117:21
*** ccamacho has quit IRC17:22
*** gfidente has joined #openstack-infra17:22
corvusmordred: on 640027 you're still missing the files i mentioned in the commit message comment on ps 217:23
clarkbfungi: can I remove group_vars/signing.yaml ? git grep for signing in system-config doesn't show that used in an obvious way17:25
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Print yaml inventory instructions  https://review.openstack.org/64012617:26
mordredcorvus: really? crappit17:27
*** jamesmcarthur has quit IRC17:27
*** wolverineav has joined #openstack-infra17:27
clarkbinfra-root double check me on this. I'm going to delete host_vars/review.openstack.org whcih has the same content as group_vars/review.yaml because gerrit currently runs on review01.openstack.org which doesn't match host_vars/review.openstack.org.yaml17:27
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002717:28
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Change the default for config-drive to True  https://review.openstack.org/64005517:28
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Print yaml inventory instructions  https://review.openstack.org/64012617:28
mordredcorvus: maybe this time I've  learned how to use git add17:28
mordredclarkb: yes. I believe that is correct17:29
*** wolverineav has quit IRC17:31
clarkbok I'm making that change now17:32
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002717:36
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Change the default for config-drive to True  https://review.openstack.org/64005517:36
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Print yaml inventory instructions  https://review.openstack.org/64012617:36
corvusmordred: see what i did in the last patchset on 64002717:37
*** jamesmcarthur has joined #openstack-infra17:37
*** wolverineav has joined #openstack-infra17:37
clarkbfungi: ttx: ok I think I've completed by hieradata spring cleaning except for group_vars/signing.yaml and host_vars/design-summit-prep.openstack.org.yaml and host_vars/odsreg.openstack.org.yaml. IF we can confirm some or all of those can be removed I will clean them up17:37
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: removes # for cards in automatic worklists  https://review.openstack.org/64012817:38
clarkbinfra-root I tried to be organized in my cleanup so that any chagnes can be easily reverted without bringin in a bunch of other cleanups should something go unexecptedly wrong17:38
*** hwoarang has quit IRC17:38
*** hwoarang has joined #openstack-infra17:39
ttxclarkb: I think it's safe to drop them now. Forum org moved to the CFP tooling and that worked out pretty well all considered17:41
*** panda|ruck is now known as panda|ruck|off17:42
clarkbttx: thanks17:43
*** gyee has joined #openstack-infra17:44
*** jamesmcarthur has quit IRC17:45
*** lujinluo has joined #openstack-infra17:46
*** derekh has quit IRC17:46
toskyfrom time to time, when I look at jobs in my gerrit dashboard, I see jobs whose update time does not match the last recorded event when I open that review17:47
toskyfor example, right now the "Updated" time in the table for this job: https://review.openstack.org/639774 is 6:23 PM17:48
*** jamesmcarthur has joined #openstack-infra17:48
corvustosky: when you say "job" do you mean "change"?17:48
toskycorvus: ups, sorry; yes, s/job/change/ or s/job/review/17:49
toskybut when I open the review, the last event happened at 4:06 PM17:49
clarkbtosky: my hunch is that gets updated when gerrit does background things like finding a new conflicts with change17:49
corvuswell, let's be really specific about terms here, because leaving a "review" on a "change" updates its last updated time, so it could be important.  :)17:49
toskycorvus: you are totally right, my bad17:49
corvusbut anyway, i think i understand the question now :)17:50
clarkbtosky: so nothing has been updated in your change, the nsomeone else pushes another change which conflicts with your change. Gerrit updates both sides of that relationship but there is no review commit to go along with it17:50
*** bhavikdbavishi has quit IRC17:50
toskybut is this something that has been going on for a while, or a new behavior? I think I noticed only recently, but maybe I didn't pay attention :)17:50
*** markvoelker has quit IRC17:51
corvuswhen was tom barron added as a reviewer?17:51
toskyoh, good question17:51
toskyI don't know, I didn't receive a notification for that17:51
toskyso that's probably the event17:51
*** bhavikdbavishi has joined #openstack-infra17:52
corvusyeah, i think i've seen newer gerrit record that event, but i think ours doesn't?  but it still might update the timestamp?17:52
*** jamesmcarthur has quit IRC17:52
*** gfidente has quit IRC17:53
*** jamesmcarthur has joined #openstack-infra17:53
toskythere was another review this morning where I'm sure I didn't see any additional reviewers added, but it was updated since then, so it can't be used to check17:53
toskyanyway, not really important, I was just confused because the default ordering is by timestamp and I was weird for me to see reviews moving up without any apparent change17:54
toskynow  I know that there are silent events17:54
openstackgerritMerged openstack-infra/zuul master: Fix missing semaphore release around dequeue  https://review.openstack.org/63902317:54
corvustosky: thanks, i hadn't noticed either :)17:55
*** luizbag has quit IRC17:56
*** gfidente has joined #openstack-infra17:56
*** yboaron_ has quit IRC17:57
*** yboaron_ has joined #openstack-infra17:57
*** lujinluo has quit IRC17:59
openstackgerritMerged openstack-infra/zuul master: Fix rare semaphore leak during reconfiguration  https://review.openstack.org/63911817:59
*** trown is now known as trown|lunch18:03
*** jmorgan1 has quit IRC18:04
openstackgerritMerged openstack-infra/zuul master: Encode zuul.message with base64  https://review.openstack.org/63880118:06
*** rossella_ has quit IRC18:07
*** bhavikdbavishi has quit IRC18:09
*** aojea has quit IRC18:10
*** jpena is now known as jpena|off18:12
*** hamzy has joined #openstack-infra18:13
*** ykarel|away has joined #openstack-infra18:16
openstackgerritMerged openstack-infra/system-config master: Update dns message to make sense for opendev  https://review.openstack.org/64002618:16
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Print yaml inventory instructions  https://review.openstack.org/64012618:21
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Add control plane clouds to nodepool builder clouds.yaml  https://review.openstack.org/64002718:21
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Change the default for config-drive to True  https://review.openstack.org/64005518:21
mordredclarkb: can has re-review on ^^ 640027 (missed adding file to the linter script) and review on https://review.openstack.org/#/c/640100/ ?18:22
*** bhavikdbavishi has joined #openstack-infra18:22
mordredcorvus: (you too on re-review)18:22
clarkbmordred: the cloud names are different in control_plane_cloudns and the nodepool clouds group files. I *think* this is ok because we'll just load the bits out of both when we need them in the nodepool builder ndoes?18:26
clarkbthey aren't exclusive right?18:27
pabelangermordred: re: launch-node, I've been trying to think of a better way of managing out SSH host keys on bridge, what are you thoughts on maybe adding the ssh-ed25519 host key for each host, as an inventory host variable, then populating SSH known_hosts file from it? This is to work around step we need to manually accept the hostkey on bridge for each new server18:27
pabelangers/out/our18:28
mordredclarkb: that's right. I think we could also just use the clouds. dict in the builder file - it has all of the things in it18:28
*** ramishra_ has quit IRC18:29
*** wolverineav has quit IRC18:30
*** wolverineav has joined #openstack-infra18:30
clarkbmordred: +2. Haven't approved as I think we should consider if we want a dedicate builder for this, but looks to be put together properly if we reuse the builders18:32
*** bhavikdbavishi has quit IRC18:32
mordredclarkb: sweet18:32
clarkbpabelanger: I think a simpler option is to just have launch-node accept the key for root18:33
*** gfidente has quit IRC18:33
clarkbwe are already running the script as root so this may automagicvally be happening18:33
*** wolverineav has quit IRC18:33
*** wolverineav has joined #openstack-infra18:33
IvensZambranoAJaeger: ping18:35
*** jamesmcarthur has quit IRC18:37
mordredclarkb, pabelanger: it's already happening best I can tell18:37
mordredalthough - it could be improved so that we don't accept the ssh hostkey for real until the script is successful - accepting the hostkey, having the server boot, fail, then later booting a different server that gets the same ip is ... fun18:38
mordredbut mostly only a problem when iterating on launch-node issues18:38
*** rh-jelabarre has quit IRC18:40
*** rh-jelabarre has joined #openstack-infra18:41
*** sdake has joined #openstack-infra18:45
*** markvoelker has joined #openstack-infra18:47
*** armstrong has quit IRC18:50
*** e0ne has joined #openstack-infra18:51
zbrAJaeger: pabelanger : please have a look at https://review.openstack.org/#/c/639951/ - thanks.18:54
fungitime to find out what i missed in here'18:55
fungithat's rather more scrollback than i expected!18:55
*** luizbag has joined #openstack-infra18:57
fungiclarkb: yes i believe odsreg, pholio01 and proposal.slave can all go away18:58
fungiclarkb: also yes to cleaning up design-summit-prep18:59
fungiclarkb: i may be mistaken, but i think i spotted a phabricator in dns earlier this week too18:59
fungiclarkb: and yes, group_vars/signing.yaml can be deleted18:59
fungiwe have a zuul job for all of those bits now19:00
fungiand i also agree host_vars/review.openstack.org should be safe to delete19:00
*** ykarel|away has quit IRC19:01
*** luizbag_ has joined #openstack-infra19:02
*** e0ne has quit IRC19:03
mordredfungi: wanna review https://review.openstack.org/#/c/640100/ ?19:03
*** luizbag has quit IRC19:04
mordred(and the one after it too)19:04
*** IvensZambrano has quit IRC19:05
*** kgiusti has quit IRC19:05
openstackgerritMerged openstack-infra/system-config master: Clean up boot-from-volume volumes on error  https://review.openstack.org/64005419:05
*** ijw has joined #openstack-infra19:06
*** iurygregory has quit IRC19:08
fungimordred: oh, you know i always want to review your changes, don't you? ;)19:09
mordredfungi: oh I do. I Do!19:09
fungifire in the hole19:10
mordredfungi: wcpgw?19:12
funginot much since it's not a production service yet anyway19:12
mordredgood point19:12
fungimy main review criteria for system-config changes is "can this break anything we care about?"19:12
fungi(i mean, i review for more than _just_ that, but...)19:13
mordredfungi: I just review for "has clarkb already +2d it"19:14
* mordred hides19:14
*** pcaruana has quit IRC19:14
fungiwell, to be fair, that's on my list of criteria too ;)19:15
*** jtomasek has quit IRC19:16
*** dpawlik has joined #openstack-infra19:16
clarkbfungi: ok signing has been removed19:18
clarkbI think I've cleaned up everything that can be cleaned up now19:19
*** dpawlik has quit IRC19:20
*** whoami-rajat has quit IRC19:20
*** markvoelker has quit IRC19:21
openstackgerritMerged openstack-infra/system-config master: Print yaml inventory instructions  https://review.openstack.org/64012619:21
*** trown|lunch is now known as trown19:22
mordredinfra-root: I've been up and working since 4am local time (for no good reason) - so I'm going to step away for a bit19:22
fungiclarkb: thanks!19:28
fungimordred: enjoy!19:28
*** ijw has quit IRC19:35
*** wolverineav has quit IRC19:39
*** wolverineav has joined #openstack-infra19:40
mriedemsilly question, but if a job only runs on nova changes, is there value in it being in the gate queue?19:41
mriedemor just voting + check?19:41
mriedemmultiple nova changes get merged in the gate queue and would run the job right?19:41
mriedemso value+ ?19:41
clarkbmriedem: the value is in checking the merge results yes19:42
mriedemok i thought so19:42
clarkbmriedem: basically it ensures that the order we've ended up in still works19:42
mriedemi don't have to think about this very often...19:42
fungithe risk is that if the change passed that job a month ago but the state of the repo has changed since such that a recheck would show that job broken for that change, then approving it would allow it to merge without a more recent exercise of that job and render the job broken for all other changes benig checked19:43
fungior one of the risks anyway19:43
mriedemgate queue it is19:43
fungimore generally, the main risk of running jobs only in check and not in gate is that it becomes much easier to merge changes which wedge that check job19:44
fungibecause you only know that it passed that job some time before and in relative isolation19:44
funginot that it actually still passes that job at the time it merges19:44
fungior rather, represents a state of the repository which will be capable of passing that job19:45
*** wolverineav has quit IRC19:48
*** wolverineav has joined #openstack-infra19:48
openstackgerritMerged openstack-infra/system-config master: Add gitea servers to inventory and load-balancer hostvars  https://review.openstack.org/64010019:49
openstackgerritMerged openstack-infra/system-config master: Add zp01 to inventory  https://review.openstack.org/64011119:49
*** dpawlik has joined #openstack-infra19:50
*** ijw has joined #openstack-infra19:51
clarkbinfra-root can I get reviews on https://review.openstack.org/#/c/639740/ to add a script to edit the secrest file? It passes tests now19:51
*** wolverineav has quit IRC19:53
*** dpawlik has quit IRC19:54
*** ijw has quit IRC19:56
*** jamesmcarthur has joined #openstack-infra19:56
fungii'll let someone who uses emacs for that review it. i still just back up the original, decrypt to a mode 0600 file, edit that quickly while confirming nobody else is making changes to the list, encrypt it and then shred and delete the plaintext copy19:58
fungii haven't actually looked at emacs's pgp mode to see what it does under the covers19:59
clarkbfungi: the thing that convinced me to let emacs do it for me is it decrypts in memory only20:01
clarkbwhcih there are vim plugins for but I figured emacs already does it so good enough20:01
fungithat's an excellent feature. granted it's still no guarantee it won't be written to disk unless you disable or encrypt swap for the server20:02
clarkbyou can also mlockall though I have no idea if emacs does that for this use case20:03
corvusemacs will also tell you if someone is editing it at the same time, or if it changed out from under you.20:05
fungialso convenient. maybe it's time i learned to ctrl-x20:05
*** sdake has quit IRC20:07
corvusswitching between openstack, docker, k8s is starting to muddle a few things in my mind... i just typed "docker show foo" and "docker examine foo" before i finally hit on "docker inspect foo"20:09
fungican't we get a unified command-line client?20:09
corvusfungi: yes.  we'll have one written in go.  and python.  and rust.20:10
*** jtomasek has joined #openstack-infra20:10
fungiwait, can't we get a unified programming language?20:11
clarkbingy.net wrote one of those in yaml20:11
clarkb(I'm not even joking, compiles your yaml to js/ruby/perl/python)20:12
fungii wrote one on a bar napkin but promptly lost it. in retrospect, it was likely crummy and full of bugs20:12
*** luizbag_ has quit IRC20:13
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: removes # for cards in automatic worklists  https://review.openstack.org/64012820:15
*** e0ne has joined #openstack-infra20:16
*** wolverineav has joined #openstack-infra20:19
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: removes # for cards in automatic worklists  https://review.openstack.org/64012820:22
*** wolverineav has quit IRC20:24
fungiinfra-root: i've generated a new openstack release signing key for use during the upcoming train development cycle: https://sks-keyservers.net/pks/lookup?op=vindex&search=0xcdc08088c3cb45a9be08332b2354069e5b504663&fingerprint=on please follow our attestation directions at your earliest convenience: https://docs.openstack.org/infra/system-config/signing.html#attestation20:25
ianwwhile we're infra-rooting, could i get some eyes on the graphite puppet fixes in https://review.openstack.org/#/c/639877/20:30
*** wolverineav has joined #openstack-infra20:30
*** wolverineav has quit IRC20:37
*** wolverineav has joined #openstack-infra20:38
*** wolverineav has quit IRC20:38
*** wolverineav has joined #openstack-infra20:38
*** hamzy has quit IRC20:39
*** hamzy has joined #openstack-infra20:43
openstackgerritMerged openstack-infra/project-config master: Use publish-to-pypi for instack-undercloud  https://review.openstack.org/63845920:45
*** kgiusti has joined #openstack-infra20:51
openstackgerritRiju Khatri proposed openstack-infra/storyboard-webclient master: allows editing for any length of story title  https://review.openstack.org/64018720:52
*** kgiusti has quit IRC20:58
*** sdake has joined #openstack-infra21:02
*** dave-mccowan has joined #openstack-infra21:10
*** gondoi has joined #openstack-infra21:14
*** kgiusti has joined #openstack-infra21:18
*** wolverineav has quit IRC21:18
*** e0ne has quit IRC21:19
*** gondoi has left #openstack-infra21:19
*** dave-mccowan has quit IRC21:22
*** sdake has quit IRC21:23
*** kjackal has joined #openstack-infra21:23
*** wolverineav has joined #openstack-infra21:25
*** sdake has joined #openstack-infra21:25
*** wolverineav has quit IRC21:28
*** wolverineav has joined #openstack-infra21:28
*** wolverineav has quit IRC21:29
*** ijw has joined #openstack-infra21:32
clarkbOh hey today was the day I said I would merge the letsencrypt spec if there wasn't any major concerns21:33
clarkbif you haven't yet and would like to rereview that spec please do so now. I'll get to that towards the end of my day21:33
openstackgerritClark Boylan proposed openstack-infra/system-config master: Add zuul user to bridge.openstack.org  https://review.openstack.org/60492521:34
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage user ssh keys from urls  https://review.openstack.org/60493221:34
clarkbRebased the CD prereq changes (adding zuulcd user to bridge)21:35
*** wolverineav has joined #openstack-infra21:35
*** kgiusti has quit IRC21:36
*** jtomasek has quit IRC21:42
*** e0ne has joined #openstack-infra21:42
openstackgerritIan Wienand proposed openstack-infra/puppet-graphite master: graphite.wsgi : update for django 1.7  https://review.openstack.org/64019921:43
*** kjackal has quit IRC21:44
*** e0ne has quit IRC21:44
*** e0ne has joined #openstack-infra21:45
*** hamzy has quit IRC21:47
*** eernst has joined #openstack-infra21:48
*** markvoelker has joined #openstack-infra21:48
clarkbianw: 'READ THIS" ok I guess :)21:50
openstackgerritMerged openstack-infra/puppet-graphite master: Fix pip ordering and build requirements  https://review.openstack.org/63987721:51
*** dpawlik has joined #openstack-infra21:51
*** wolverineav has quit IRC21:51
ianwclarkb: heh, i would say that i didn't put that there :)21:52
*** eernst has quit IRC21:52
ianwthere's still something wrong, and whatever should be on port 2003 isn't on the new server ... debugging21:52
clarkbfwiw we don't ahve that WSGIScriptImport in our vhost but that isn't a new regression21:52
*** zul has quit IRC21:53
*** dpawlik has quit IRC21:55
ianwyeah, not sure what that's about21:56
*** slaweq has quit IRC21:58
*** wolverineav has joined #openstack-infra22:01
*** e0ne has quit IRC22:01
*** slaweq has joined #openstack-infra22:04
ianwcan we clone cinder volumes in rax?22:05
clarkbI'm not aware of such a feature22:05
fungii can't say i've tried22:05
fungiwe can definitely snapshot them22:05
fungiand i can't imagine what good that does you if you can't make a new volume from a snapshot22:06
ianwfungi: now i've hit a couple of things and there's a few new versions of django in the way, i'm not thrilled about just detaching and reattaching the stats data22:06
fungiso in a sense you could make a copy of a cinder volume but you may need a lengthy outage to do so22:06
ianwso i think either your rsync idea, or if i can clone them and then rsync that would be even better22:07
ianwoh, i'm not sure i've snapshotted; it's not an atomic op where the parent just continues on?22:07
*** wolverineav has quit IRC22:09
*** slaweq has quit IRC22:09
ianweven still a volume-level snapshot might still be inconsistent given a busy host like statsd.  i think this is pointing to a userspace rsync solution22:10
clarkbdoes whsiper do a merge?22:11
clarkbwe might be able to start writing to the new host, then merge in the content of the old host?22:11
*** slaweq has joined #openstack-infra22:11
ianwhrm, maybe, they're pretty flat files, i wouldn't be surprised if you can cat them together22:11
ianwpython-whisper : whisper-merge22:12
*** rh-jelabarre has quit IRC22:12
*** sdake has quit IRC22:13
*** slaweq has quit IRC22:16
fungiif you're committed to doing some rsync, i'd just do it all via rsync. it's only 0.45tb and 2.1m inodes in /var/lib/graphite/storage22:17
clarkboh thats less than I thoguht it was, ya not too bad22:18
fungithe logical volume is 2tb but it's only 22% used22:18
fungiin theory, at least, subsequent rsync after the initial pass or two should result in a fairly brief lag to grab the remaining delta22:19
fungiso we shouldn't need to shut down writing services on it for long22:19
openstackgerritIan Wienand proposed openstack-infra/puppet-graphite master: graphite.wsgi : update for django 1.7  https://review.openstack.org/64019922:19
fungiplus, i think we need to reload iptables on everything it's polling anyway, right?22:20
ianwi don't think so, it's ports are open for the servers that want to talk to it22:20
clarkbya it doesn't poll22:20
fungioh, nm. i was thinking cacti for a sec22:21
ianwi will take the rsync approach, i'll add some volumes and do an initial sync now22:21
clarkbas long as things lookup new dns values we should be fine22:21
fungiianw: fair warning, i dunno what our cinder quota looks like these days but you may need to hunt for things to clean up first22:21
clarkb(I have no idea if they will do that though)22:21
ianwif everything's in place, i can probably do this on my monday, which is the quietest time22:21
ianwthis being cutting over dns and final rsync22:21
clarkbso we may need to restart services, but otherwise should work22:21
fungiianw: alternative, just do a 1tb volume initially and we can expand it later (but maybe format it for a larger inode percentage just to be safe)22:22
*** markvoelker has quit IRC22:22
ianwrecovery of graphite.o.o partitions.  noticed during pypi testing22:22
ianw  that it had a volume in indeterminite state.  rebuild cinder22:22
ianw  volumes, resync.  in progress22:22
fungier, nevermind about the inodes. looks like we're at 2% inode utilization on the existing volume anyway22:23
ianwmy notes from 2017-08-04 ... maybe the second volume was only in service of fixing it at the time?22:23
ianwgraphite.o.o recovery.  take different approach of swapping out22:24
ianw  failed disk in lvm.  cleanup other cinder volumes22:24
fungiwell, we extended the logical volume to use both physical volumes22:24
fungior so it appears22:24
fungiyeah, /dev/xvdc1 and /dev/xvdf1 are both present in the main vg22:24
ianwso i'm not sure if it started at 1tb and got an extra disk during all that for no particular reason22:25
ianwi think probably a single 1tb lvm is fine, can always grow it22:25
*** slaweq has joined #openstack-infra22:25
fungimy thinking as well22:25
clarkband if you attach it to the old host you can rsync "locally" right?22:26
fungiwe won't even be at 50% utilization on a 1tb volume anyway22:26
fungiclarkb: it's still going over their iscsi network which i haven't observed being especially faster22:26
fungibut sure, it's an option22:27
fungicould also use the private addresses instead of public? also no idea if that will be appreciably better throughput though22:27
ianwi think i'll just do it on host to avoid having to move lvm pvs's around etc22:27
*** wolverineav has joined #openstack-infra22:28
fungiyeah, i would prioritize your own time spent fiddling with stuff over time spent waiting for the rsync to complete when you can just work on other things anyway22:29
*** wolverineav has quit IRC22:29
*** wolverineav has joined #openstack-infra22:29
fungier, that is to say, i favor simplicity over complex solutions to make it happen faster22:29
*** dpawlik has joined #openstack-infra22:29
*** slaweq has quit IRC22:30
clarkb++22:30
*** ijw has quit IRC22:32
*** ijw has joined #openstack-infra22:33
*** wolverineav has quit IRC22:33
*** dpawlik has quit IRC22:34
*** ijw has quit IRC22:35
*** ijw has joined #openstack-infra22:35
corvusclarkb, mordred: i found an issue with my gitea patch and made a followup: https://github.com/go-gitea/gitea/pull/6216  -- but with that, my rename playbook works no matter what crazy ordering i throw at it.22:35
mordredcorvus: neat~!22:35
corvusclarkb: would you mind +3ing https://review.openstack.org/640102 ?22:36
clarkbcorvus: looking22:36
clarkbyou can have globs in zone files? how have I never known this22:37
clarkbI mean it makes sense22:37
corvusit's a wildcard22:37
corvusfungi: ^ maybe you want to double check that too, it's our first wildcard entry :)22:37
openstackgerritMerged openstack-infra/zone-opendev.org master: Add zuul-preview  https://review.openstack.org/64010222:37
clarkboh I just approve it22:37
clarkb:)22:37
mordredfungi: no doublecheck for you!22:38
corvusthere's still plenty of time to revert :)22:38
fungiindeed, i'm looking now22:38
fungithe reindenting makes it a little hard to follow22:39
fungii guess it's only whitespace changes except for the serial and the last record?22:39
clarkbfungi: last 3 records22:40
fungiyep, i see that now22:40
corvussorry about that... there was a conversation about tabs and i just wanted to get it out of the way.  i think i did drop a couple of ".opendev.org" from the right hand side.22:40
openstackgerritMatt Riedemann proposed openstack-infra/openstack-zuul-jobs master: Drop legacy-grenade-dsvm-neutron-multinode-live-migration  https://review.openstack.org/64020922:40
corvusgraphite and pbx22:40
fungiyep, lgtm22:41
fungitimes were i wouldn't advocate for a cname rr to another record in the same zone, but this is sort of a special case and these days nameservers know to provide additional results when they're likely to be iteratively requested otherwise22:42
fungiso it's not actually likely to result in many additional queries, if any22:43
funginot to mention, we're not in any danger of blowing up these nameservers with requests in the first place22:43
clarkbfungi: ya and the dns over tls is gonna hold a connection open forever anyway22:44
fungidns over tls?22:44
fungii've really been out of the loop22:44
*** wolverineav has joined #openstack-infra22:45
clarkbfungi: there is also dns over https but I'm a little annoyed browsers thinkt they should ignore my system settings for dns22:45
*** slaweq has joined #openstack-infra22:45
fungioh, is dns over tls for securing the last hop between the local resolver and your configured recursive resolver?22:46
clarkbfungi: ya to prevent your isp from snooping or injecting things (though with dnssec its mostly the snooping)22:46
clarkbinfra-root I'm not sure if ya'll follow the tc channel but not too long ago a request from the internets came in to hand over the "nimble" package on pypi. Looking at it we renamed our nimble project to mogan due to a name conflict with a contributing company. We only ever published one version 0.0.1 to nimble on pypi. Our nimble was never an official project and renamed anyways so I think we can22:47
clarkbhand it over22:47
clarkb(request was sent to the openstack-discuss list if you want to read it)22:47
clarkbcan you all think of a reason not to hand it over?22:47
corvusclarkb: did anyone ask https://pypi.org/user/Zhenguo/ ?22:49
*** wolverineav has quit IRC22:49
clarkbI don't think anyone has yet, but smcginnis asked on the huawei side.22:50
clarkbBut maybe we respond to the request and cc zhenguo and see what happens22:50
*** slaweq has quit IRC22:51
corvusmy thinking is that since it *wasn't* an official project, we should defer to the original registrant first.  but if they are awol after earnest attempts to contact them, i think we can then defer to the tc as the next best placed entity to make a determination of non-interest.22:51
clarkbok in that case I'll respond to the mailing list request with a cc to zhenguo and see if they can ack/nak22:52
fungiyeah, that sounds fine22:53
*** tkajinam has joined #openstack-infra22:55
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add gitea to project rename playbook  https://review.openstack.org/64021822:56
smcginnisNo real reason to wait. It was nimble, then renamed to mogan, then died.22:57
smcginnisNimble definitely is not needed anymore.22:57
smcginnisBut maybe good to copy zhenguo on the response just as a courtesy I guess.22:57
clarkbsmcginnis: ya I'm gonna write an email offering to do the legwork is zhenguo niu responds with an ack and leave an out if we don't get a response in a reasonable amount of time22:58
clarkbgenerally I think we should follow a process like this so that it isn't ambiguous should it happen again.22:58
smcginnisSounds like a good plan.22:58
mordredcorvus: there isn't any gitea running on the gitea servers22:58
mordreddid I miss something in the patches earlier?22:58
mordredcorvus: also - should we add admins to the docker group?22:59
*** wolverineav has joined #openstack-infra23:01
corvusmordred: i also didn't see zuul-preview running, but i haven't dug into it yet23:01
mordredcorvus: oh! I see a failed=1 in the run_all logs23:01
clarkbmordred: iirc you are strongly recommended to use sudo23:01
clarkb(rather than be in the docker group)23:02
clarkbI think because you get sudo accounting? I dunno if we really care23:02
mordredclarkb: oh - ok. if using sudo is right, totes cool23:02
mordredcorvus: fatal: [gitea08.opendev.org]: FAILED! => {"changed": false, "msg": "AnsibleUndefinedVariable: 'gitea_root_db_password' is undefined"}23:02
mordredcorvus: fatal: [zp01.opendev.org]: FAILED! => {"changed": false, "module_stderr": "/bin/sh: 1: /usr/bin/python: not found\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 127}23:04
mordredcorvus: is the other one23:04
clarkbfor zp01 probably need to tell ansible to use python323:04
*** ijw has quit IRC23:06
corvusmordred: i think on bridge we need to copy gitea-cluster.yaml to hosts/group_vars/gitea.yaml23:06
*** ijw has joined #openstack-infra23:06
corvusgitea-cluster.yaml is used by the k8s cluster, since it doesn't fit into the normal inventory23:07
openstackgerritMatt Riedemann proposed openstack-infra/openstack-zuul-jobs master: Drop legacy-grenade-dsvm-neutron-multinode-live-migration  https://review.openstack.org/64020923:07
*** wolverineav has quit IRC23:07
corvusbut i tihnk i kept all the variable names the same, so it should be a subset of the original23:07
openstackgerritMonty Taylor proposed openstack-infra/system-config master: Set ansible_python_interpreter for zuul-preview  https://review.openstack.org/64022123:08
mordredcorvus, clarkb: I think that should fix zuul-preview23:08
*** wolverineav has joined #openstack-infra23:08
*** wolverineav has quit IRC23:08
corvusmordred: then i'll fix your thing :)23:08
clarkbmordred: +223:08
*** wolverineav has joined #openstack-infra23:08
mordredcorvus: yay! cause I can't find gitea-cluster ... likely cause I'm daft23:09
mordredoh - hosts/gitea-cluster.yaml23:09
corvusmordred: done.  the only variable i dropped was the xtradb backup password23:10
mordredcool23:10
*** slaweq has joined #openstack-infra23:11
*** rascasoft has quit IRC23:11
ianwinfra-root: if i could get a couple of eyes on https://review.openstack.org/#/c/640199/ for the graphite django update, once it rolls around graphite01.opendev.org should be reporting the stats it's currently rsyncing23:14
*** slaweq has quit IRC23:15
*** ijw has quit IRC23:16
*** ijw has joined #openstack-infra23:16
*** markvoelker has joined #openstack-infra23:18
*** wolverineav has quit IRC23:23
mordredianw: +A23:24
*** markvoelker has quit IRC23:24
*** markvoelker has joined #openstack-infra23:24
ianwthanks; probably a good initial target for letsencrypt certs too23:25
ianwi will leave upgrading everything to python3 to future work when it's run in a container ... that's a bridge too far ATM :)23:25
openstackgerritIvoline Ngong proposed openstack-infra/storyboard master: Updated_at content modified  https://review.openstack.org/63884023:27
*** ijw has quit IRC23:29
*** wolverineav has joined #openstack-infra23:29
*** ijw has joined #openstack-infra23:29
*** ijw has quit IRC23:31
*** ijw has joined #openstack-infra23:32
*** ekultails has quit IRC23:38
*** jamesmcarthur has quit IRC23:46
openstackgerritMerged openstack-infra/puppet-graphite master: graphite.wsgi : update for django 1.7  https://review.openstack.org/64019923:50
clarkbI need to write a firefox plugin that autocloses logs.o.o tabs after a day23:53
clarkbmordred: http://logs.openstack.org/25/604925/10/check/system-config-run-base-ansible-devel/4df913a/job-output.txt.gz#_2019-02-28_21_58_43_579799 do you think ansible would look at us silly if we suggested that a third aprty ci job?23:55
clarkbit appears ansible devel has broken testinfra, not sure if that is intentional or that anyone knows but seems like maybe that is important to not break?23:56
*** hwoarang has quit IRC23:57
clarkbI'm guessing we might have to bring the scope of that job down a bit for use as third aprty ci, but so far its caught all kinds of fun stuff that ansible probably would like to know about23:57
*** hwoarang has joined #openstack-infra23:59

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