*** jamesmcarthur has joined #openstack-infra | 00:02 | |
*** jamesmcarthur has quit IRC | 00:06 | |
*** rcernin has quit IRC | 00:07 | |
*** rcernin has joined #openstack-infra | 00:09 | |
*** yamamoto has joined #openstack-infra | 00:09 | |
*** rascasoft has quit IRC | 00:09 | |
*** tosky has quit IRC | 00:09 | |
*** armax has joined #openstack-infra | 00:15 | |
openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Nameservers are now managed with ansible https://review.openstack.org/629257 | 00:17 |
---|---|---|
clarkb | that should pass tests now | 00:17 |
*** slaweq has quit IRC | 00:18 | |
*** yamamoto has quit IRC | 00:19 | |
clarkb | and once that is in we should be able to safely delete the servers as nothing will be trying to talk to it anymore (no more anisble/puppet) | 00:19 |
*** rascasoft has joined #openstack-infra | 00:21 | |
clarkb | if anyone is wondering the large spike in yum failures e-r is reporting appear related to kolla being very broken | 00:26 |
clarkb | grafana repo isn't installing | 00:26 |
dmsimard | so golang has a bot called gerritbot which is totally different than our gerritbot: https://github.com/golang/go/wiki/GerritBot | 00:28 |
dmsimard | it syncs github pull requests to gerrit reviews | 00:29 |
dmsimard | a random pull request: https://github.com/golang/go/pull/29606 synced to gerrit: https://go-review.googlesource.com/c/go/+/156757 with the github PR link in the commit message | 00:30 |
clarkb | dmsimard: ya they also had git-review once upon a time | 00:30 |
dmsimard | once upon a time ? they're using the ones from infra now ? | 00:30 |
clarkb | no they renamed theirs to reduce confusion | 00:31 |
dmsimard | eh | 00:31 |
dmsimard | from a quick glance, it looks like the bot works well ? and it syncs in both directions | 00:33 |
*** owalsh_ has joined #openstack-infra | 00:33 | |
dmsimard | that's kind of neat | 00:34 |
fungi | depends on the use case | 00:34 |
fungi | we had a couple of reasons not to consider that approach | 00:34 |
*** rascasoft has quit IRC | 00:35 | |
fungi | first, legally, the openstack project couldn't accept patches from anyone who hadn't already agreed to their icla | 00:35 |
*** slaweq has joined #openstack-infra | 00:35 | |
dmsimard | Makes sense | 00:36 |
*** owalsh has quit IRC | 00:36 | |
clarkb | dmsimard: the sha1s don't match up | 00:37 |
clarkb | they must be squashing the commits down | 00:37 |
dmsimard | If you look at that pull request I sent, there is a bot that applies a "cla:yes" tag so I guess it's a thing :p | 00:38 |
dmsimard | clarkb: yes, actually I was wondering how they dealt with multiple commits in a PR. Good catch :) | 00:38 |
fungi | right, they'd still need contributors to separately agree to a cla outside github | 00:38 |
clarkb | and ya the commit message has a pointer back to the github sha1 too, not sure I agree with that design but can see why you might choose it | 00:39 |
fungi | but second, it looks like they can't sync review comments between gerrit changes and github prw | 00:39 |
fungi | prs | 00:39 |
fungi | so the benefit isn't clear to me if people who would rather only understand github still have to follow up in gerrit | 00:40 |
clarkb | fungi: I think this was a compromise for when there was a big kerfluffle in their community where one side said you can't open source without github and the core reviewers/maintainers said they can't review code effectively without gerrit | 00:41 |
fungi | looks like it echoes gerrit change comments one-way into github pr comments, but not back | 00:41 |
dmsimard | I see Gerrit comments in GitHub ? I guess this is one: https://github.com/golang/go/pull/29606#issuecomment-452168976 | 00:41 |
*** yamamoto has joined #openstack-infra | 00:41 | |
fungi | clarkb: yeah, i'd rather just pick a new term for free software if "open source" means you have to use github | 00:41 |
clarkb | fungi: we can't use open core that one is already taken now :P | 00:42 |
fungi | open mind software | 00:42 |
clarkb | (doesn't actually fit, but there is an open core conference or something now) | 00:42 |
fungi | dmsimard: yep, it's the other direction which is missing | 00:42 |
fungi | dmsimard: so the gerrit users can proclaim from on high into github, and the github-only users have no other choice but to respond with new commits | 00:43 |
fungi | (not the end of the world, i agree) | 00:43 |
clarkb | fungi: but also suggest you might be better of just using gerrit | 00:43 |
fungi | "Please don’t reply on this GitHub thread. Visit ..." | 00:43 |
fungi | (over and over and over...) | 00:44 |
fungi | also apparently it's gerrit bot copying github prs into their gerrit, but gopherbot copying gerrit change comments back to them | 00:45 |
clarkb | dansmith: some simple attempts to reproduce the weird behavior seem to show that the unified diff may be more reliable than side by side | 00:45 |
*** yamamoto has quit IRC | 00:45 | |
fungi | and it doesn't look like changes opened in their gerrit cause equivalent prs to be opened in github, so again github-only users aren't *really* going to be able to follow all their development activity anyway | 00:46 |
*** Swami has quit IRC | 00:46 | |
*** armax has quit IRC | 00:46 | |
clarkb | dansmith: I also have managed to sort out what triggers it | 00:47 |
clarkb | I've also now switched the render config option to slow to see if that helps at all (there have been reports both ways on that) | 00:47 |
*** slaweq has quit IRC | 00:47 | |
*** rascasoft has joined #openstack-infra | 00:51 | |
*** sthussey has quit IRC | 00:51 | |
*** armax has joined #openstack-infra | 00:56 | |
clarkb | dansmith: I don't have much data yet but ^ seems to jump less on large side by side diffs | 00:59 |
clarkb | I'll run with that for a bit and see if it helps | 00:59 |
dansmith | clarkb: it's been killing me on specs | 00:59 |
clarkb | dansmith: ya thats where I noticed it too. The side by side with one pane empty because its a giant new file | 01:03 |
*** rascasoft has quit IRC | 01:03 | |
clarkb | with lots of comments (specifically zuuls distributed scheduelr spec) | 01:03 |
mrhillsman | o/ | 01:08 |
mrhillsman | i was wondering if there is a general time for when things get reviewed or if i did this right - https://review.openstack.org/#/c/626084/ | 01:09 |
*** armax has quit IRC | 01:09 | |
mrhillsman | not sure if this type of change goes into a queue or just was missed with the holidays | 01:10 |
*** jesusaur has joined #openstack-infra | 01:12 | |
clarkb | mrhillsman: I think its mostly holidays and catching up on everything | 01:13 |
mrhillsman | ++ | 01:13 |
*** rascasoft has joined #openstack-infra | 01:14 | |
*** slaweq has joined #openstack-infra | 01:18 | |
*** owalsh_ is now known as owalsh | 01:23 | |
*** slaweq has quit IRC | 01:23 | |
*** rascasoft has quit IRC | 01:28 | |
*** rascasoft has joined #openstack-infra | 01:44 | |
dmsimard | fungi: the wiki mentions that it's because code reviews are done exclusively on gerrit | 01:46 |
dmsimard | It's another way to send a patch I guess. The kernel review process by email fascinates me :) | 01:48 |
*** hongbin_ has joined #openstack-infra | 01:48 | |
*** xek has quit IRC | 01:51 | |
dmsimard | fungi, clarkb: so would it be as problematic in gitea, then ? | 01:53 |
dmsimard | I realize that the spec mentions pull requests, wiki and issues would be disabled at first but it seems to leave a door opened | 01:54 |
clarkb | no I dont think it does | 02:09 |
clarkb | it says explicitly it doesnt :) | 02:09 |
clarkb | fwiw I personally dont find the PR workflow to be effective or one I'dwant to support. It is lossy and ambiguous as to how you should curate commits | 02:11 |
clarkb | and syncing between systems is even worse | 02:11 |
*** xek has joined #openstack-infra | 02:12 | |
*** xek has quit IRC | 02:12 | |
*** xek has joined #openstack-infra | 02:13 | |
clarkb | I think the wiki functionality is the one most people have in mind for potentially adding later | 02:14 |
*** rascasoft has quit IRC | 02:17 | |
dmsimard | clarkb: I re-read again, the door is opened for the wiki only, you're correct :p | 02:21 |
*** rascasoft has joined #openstack-infra | 02:31 | |
*** rascasoft has quit IRC | 02:36 | |
*** rascasoft has joined #openstack-infra | 02:39 | |
*** bobh has joined #openstack-infra | 02:41 | |
*** bobh has quit IRC | 02:47 | |
*** psachin has joined #openstack-infra | 02:48 | |
*** rascasoft has quit IRC | 02:48 | |
*** wolverineav has quit IRC | 02:48 | |
*** wolverineav has joined #openstack-infra | 02:48 | |
*** whoami-rajat has joined #openstack-infra | 02:52 | |
*** rascasoft has joined #openstack-infra | 02:53 | |
*** wolverineav has quit IRC | 02:54 | |
*** wolverineav has joined #openstack-infra | 02:55 | |
*** fuentess has quit IRC | 02:56 | |
*** wolverineav has quit IRC | 02:57 | |
*** wolverineav has joined #openstack-infra | 02:59 | |
*** rascasoft has quit IRC | 03:03 | |
*** wolverineav has quit IRC | 03:03 | |
*** gyee has quit IRC | 03:10 | |
*** markvoelker has joined #openstack-infra | 03:11 | |
*** markvoelker has quit IRC | 03:12 | |
*** rascasoft has joined #openstack-infra | 03:15 | |
*** apetrich has quit IRC | 03:16 | |
*** anteaya has quit IRC | 03:24 | |
*** rascasoft has quit IRC | 03:27 | |
*** ramishra has joined #openstack-infra | 03:29 | |
*** ykarel|away has joined #openstack-infra | 03:32 | |
openstackgerrit | Merged openstack/diskimage-builder master: package-installs: provide for skip from env var https://review.openstack.org/619119 | 03:34 |
*** rkukura has quit IRC | 03:36 | |
openstackgerrit | Merged openstack/diskimage-builder master: simple-init: allow for NetworkManager support https://review.openstack.org/619120 | 03:38 |
*** rascasoft has joined #openstack-infra | 03:39 | |
*** janki has joined #openstack-infra | 03:45 | |
*** spsurya has joined #openstack-infra | 03:46 | |
*** yamamoto has joined #openstack-infra | 03:52 | |
*** rascasoft has quit IRC | 03:53 | |
*** yamamoto has quit IRC | 03:54 | |
*** rlandy|rover|bbl is now known as rlandy|rover | 03:54 | |
*** yamamoto has joined #openstack-infra | 03:55 | |
*** diablo_rojo has quit IRC | 04:00 | |
*** rascasoft has joined #openstack-infra | 04:02 | |
*** yamamoto has quit IRC | 04:04 | |
*** yamamoto has joined #openstack-infra | 04:05 | |
*** yamamoto has quit IRC | 04:12 | |
*** rlandy|rover has quit IRC | 04:17 | |
*** bhavikdbavishi has joined #openstack-infra | 04:22 | |
*** hongbin_ has quit IRC | 04:24 | |
*** udesale has joined #openstack-infra | 04:27 | |
*** rascasoft has quit IRC | 04:28 | |
*** markvoelker has joined #openstack-infra | 04:37 | |
*** markvoelker has quit IRC | 04:42 | |
*** rascasoft has joined #openstack-infra | 04:42 | |
*** wolverineav has joined #openstack-infra | 04:45 | |
*** rascasoft has quit IRC | 04:52 | |
*** rf0lc0 has joined #openstack-infra | 04:53 | |
*** hwoarang has joined #openstack-infra | 04:56 | |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Add Fedora 29 nodes https://review.openstack.org/618672 | 04:56 |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Enable NetworkManager for Centos 7 https://review.openstack.org/619960 | 04:56 |
*** lifeless_ has joined #openstack-infra | 04:57 | |
*** jbadiapa has joined #openstack-infra | 04:57 | |
*** rascasoft has joined #openstack-infra | 04:58 | |
*** wolverineav has quit IRC | 05:00 | |
*** psachin has quit IRC | 05:02 | |
*** jesusaur has quit IRC | 05:02 | |
*** rfolco has quit IRC | 05:02 | |
*** sshnaidm has quit IRC | 05:02 | |
*** hwoarang_ has quit IRC | 05:02 | |
*** irclogbot_1 has quit IRC | 05:02 | |
*** rcarrillocruz has quit IRC | 05:02 | |
*** chason has quit IRC | 05:02 | |
*** lifeless has quit IRC | 05:02 | |
*** rascasoft has quit IRC | 05:02 | |
*** irclogbot_1 has joined #openstack-infra | 05:04 | |
*** auristor has quit IRC | 05:07 | |
*** jesusaur has joined #openstack-infra | 05:09 | |
*** sshnaidm has joined #openstack-infra | 05:09 | |
*** ykarel|away has quit IRC | 05:10 | |
*** zaro has joined #openstack-infra | 05:10 | |
*** rascasoft has joined #openstack-infra | 05:15 | |
*** wolverineav has joined #openstack-infra | 05:15 | |
*** jpena|off has quit IRC | 05:22 | |
*** jpena|off has joined #openstack-infra | 05:23 | |
*** rascasoft has quit IRC | 05:25 | |
*** ykarel|away has joined #openstack-infra | 05:26 | |
*** bhavikdbavishi has quit IRC | 05:26 | |
*** ykarel|away is now known as ykarel | 05:26 | |
*** bhavikdbavishi has joined #openstack-infra | 05:27 | |
*** armax has joined #openstack-infra | 05:27 | |
openstackgerrit | Merged openstack-infra/irc-meetings master: Consolidate openstack-meeting channels https://review.openstack.org/626182 | 05:29 |
*** udesale has quit IRC | 05:32 | |
*** roman_g has quit IRC | 05:32 | |
*** auristor has joined #openstack-infra | 05:36 | |
*** udesale has joined #openstack-infra | 05:40 | |
*** rascasoft has joined #openstack-infra | 05:48 | |
*** rascasoft has quit IRC | 05:53 | |
openstackgerrit | Merged openstack-infra/nodepool master: Add Fedora 29 testing https://review.openstack.org/618671 | 05:54 |
*** markvoelker has joined #openstack-infra | 06:04 | |
*** rkukura has joined #openstack-infra | 06:17 | |
*** e0ne has joined #openstack-infra | 06:22 | |
*** e0ne has quit IRC | 06:23 | |
*** ramishra has quit IRC | 06:47 | |
*** ramishra has joined #openstack-infra | 06:47 | |
*** wolverineav has quit IRC | 07:04 | |
*** wolverineav has joined #openstack-infra | 07:05 | |
*** pcaruana has joined #openstack-infra | 07:06 | |
*** wolverineav has quit IRC | 07:09 | |
*** quiquell|off is now known as quiquelll | 07:09 | |
*** quiquelll is now known as quiquell | 07:09 | |
*** udesale has quit IRC | 07:17 | |
*** e0ne has joined #openstack-infra | 07:18 | |
*** pgaxatte has joined #openstack-infra | 07:22 | |
*** e0ne has quit IRC | 07:22 | |
*** udesale has joined #openstack-infra | 07:22 | |
*** kjackal has joined #openstack-infra | 07:23 | |
*** rcernin has quit IRC | 07:26 | |
*** udesale has quit IRC | 07:27 | |
*** udesale has joined #openstack-infra | 07:29 | |
*** ykarel is now known as ykarel|lunch | 07:29 | |
*** rascasoft has joined #openstack-infra | 07:36 | |
*** bobh has joined #openstack-infra | 07:39 | |
AJaeger | ianw: I gave https://review.openstack.org/618672 another +W now to merge... | 07:40 |
AJaeger | ianw: can we merge 619960 already or does it need some release/restarts? | 07:40 |
*** bobh has quit IRC | 07:43 | |
*** e0ne has joined #openstack-infra | 07:47 | |
*** armax has quit IRC | 07:49 | |
openstackgerrit | Merged openstack-infra/project-config master: Add Fedora 29 nodes https://review.openstack.org/618672 | 07:51 |
amorin | hey all | 07:53 |
*** rpittau has joined #openstack-infra | 07:54 | |
*** lpetrut has joined #openstack-infra | 07:55 | |
amorin | clarkb: I saw your message about quota on BHS1 | 07:57 |
amorin | I will check | 07:57 |
*** quiquell is now known as quiquell|brb | 08:00 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/infra-specs master: Add anomaly detection in CI/CD jobs specification draft https://review.openstack.org/581214 | 08:04 |
*** gfidente has joined #openstack-infra | 08:06 | |
*** slaweq has joined #openstack-infra | 08:13 | |
*** bhavikdbavishi has quit IRC | 08:21 | |
*** ginopc has joined #openstack-infra | 08:22 | |
*** bhavikdbavishi has joined #openstack-infra | 08:22 | |
*** quiquell|brb is now known as quiquell | 08:28 | |
*** tosky has joined #openstack-infra | 08:37 | |
*** eumel8 has joined #openstack-infra | 08:37 | |
*** apetrich has joined #openstack-infra | 08:38 | |
amorin | clarkb: I upgraded the number of vcpu quota | 08:41 |
amorin | should be better | 08:41 |
amorin | I also resync the quota | 08:43 |
*** dkehn has quit IRC | 08:46 | |
*** e0ne has quit IRC | 08:47 | |
*** jpena|off is now known as jpena | 08:52 | |
*** quiquell has quit IRC | 08:53 | |
*** quiquell has joined #openstack-infra | 08:54 | |
*** quiquell has quit IRC | 08:54 | |
*** quiquell has joined #openstack-infra | 08:56 | |
*** yamamoto has joined #openstack-infra | 08:56 | |
*** yamamoto has quit IRC | 08:57 | |
*** jpich has joined #openstack-infra | 09:00 | |
*** ccamacho has joined #openstack-infra | 09:16 | |
*** kjackal has quit IRC | 09:22 | |
*** kjackal has joined #openstack-infra | 09:22 | |
openstackgerrit | Natal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr. https://review.openstack.org/629414 | 09:25 |
*** bobh has joined #openstack-infra | 09:25 | |
*** yamamoto has joined #openstack-infra | 09:27 | |
*** yamamoto_ has joined #openstack-infra | 09:27 | |
*** hrubi has joined #openstack-infra | 09:28 | |
*** yboaron has joined #openstack-infra | 09:29 | |
*** yamamoto has quit IRC | 09:31 | |
*** markvoelker has quit IRC | 09:33 | |
*** yamamoto_ has quit IRC | 09:33 | |
AJaeger | ssbarnea: regarding https://review.openstack.org/629205 - note that I'm not a core on devstack-gate | 09:36 |
ssbarnea | AJaeger: these days it seems hard to find reviewers on that repo :p | 09:37 |
*** kjackal has quit IRC | 09:38 | |
ssbarnea | i know is mostly in deprecation mode but is still used and i face issues on real jobs that originate from there, so we should fix them. any idea who could help? | 09:38 |
*** kjackal has joined #openstack-infra | 09:38 | |
*** derekh has joined #openstack-infra | 09:38 | |
ssbarnea | i asked on #openstack-qa but no answers yet | 09:39 |
*** ykarel|lunch is now known as ykarel | 09:39 | |
*** zhangfei has joined #openstack-infra | 09:43 | |
AJaeger | ssbarnea: sorry, no ideas who can help. Here and #openstack-qa seems best place indeed. | 09:43 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: WIP: Add spec for scale out scheduler https://review.openstack.org/621479 | 09:46 |
*** ramishra_ has joined #openstack-infra | 09:56 | |
*** ramishra has quit IRC | 09:57 | |
*** bhavikdbavishi has quit IRC | 10:00 | |
* iurygregory be back in a few, going to other building =) | 10:05 | |
*** dtantsur|afk is now known as dtantsur | 10:10 | |
*** iurygregory has quit IRC | 10:12 | |
*** panda|off is now known as panda | 10:18 | |
*** bobh has quit IRC | 10:19 | |
*** e0ne has joined #openstack-infra | 10:21 | |
*** jaosorior has quit IRC | 10:22 | |
*** zhangfei has quit IRC | 10:32 | |
openstackgerrit | Natal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr. https://review.openstack.org/629414 | 10:38 |
openstackgerrit | Natal Ngétal proposed openstack/diskimage-builder master: [Core] Change openstack-dev to openstack-discuss. https://review.openstack.org/622895 | 10:40 |
openstackgerrit | Natal Ngétal proposed openstack/diskimage-builder master: [Configuration] Add missing py37 and corrected default envlist. https://review.openstack.org/624670 | 10:41 |
*** ldnunes has joined #openstack-infra | 10:43 | |
*** electrofelix has joined #openstack-infra | 10:58 | |
*** jaosorior has joined #openstack-infra | 10:59 | |
openstackgerrit | Will Szumski proposed openstack-dev/pbr master: Fix white space handling in file names https://review.openstack.org/629161 | 11:03 |
*** pbourke has quit IRC | 11:10 | |
*** pbourke has joined #openstack-infra | 11:12 | |
openstackgerrit | Thierry Carrez proposed openstack-infra/system-config master: Remove meeting-5 and meeting-cp from meetbot https://review.openstack.org/629439 | 11:16 |
*** jamesmcarthur has joined #openstack-infra | 11:17 | |
*** jamesmcarthur has quit IRC | 11:21 | |
*** yamamoto has joined #openstack-infra | 11:22 | |
*** yamamoto has quit IRC | 11:23 | |
*** yamamoto has joined #openstack-infra | 11:23 | |
openstackgerrit | Merged openstack-infra/irc-meetings master: Update count_slot_usage for new recurrences https://review.openstack.org/625656 | 11:24 |
openstackgerrit | Merged openstack-infra/irc-meetings master: Add count_slot_usage argument for sensitivity https://review.openstack.org/625657 | 11:24 |
*** udesale has quit IRC | 11:24 | |
openstackgerrit | Will Szumski proposed openstack-dev/pbr master: Do not globally replace path prefix https://review.openstack.org/629006 | 11:32 |
*** chandankumar is now known as chkumar|out | 11:39 | |
*** rtjure has quit IRC | 11:40 | |
openstackgerrit | Will Szumski proposed openstack-dev/pbr master: Fix white space handling in file names https://review.openstack.org/629161 | 11:47 |
*** dpawlik has joined #openstack-infra | 11:48 | |
*** ykarel is now known as ykarel|afk | 11:58 | |
*** rtjure has joined #openstack-infra | 12:00 | |
*** jpena is now known as jpena|lunch | 12:08 | |
*** jangutter has joined #openstack-infra | 12:12 | |
*** wolverineav has joined #openstack-infra | 12:14 | |
jangutter | Weird question: I submitted a review, and openstack-infra updated the link to the review with a broken link on a blueprint. Did I do something stupid? | 12:14 |
jangutter | The review was https://review.openstack.org/571325 | 12:15 |
jangutter | It added: Gerrit topic: https://review.openstack.org/#q,topic:bp/vrouter-os-vif-conversion,n,z where it should have added https://review.openstack.org/#/q/topic:bp/vrouter-os-vif-conversion,n,z | 12:15 |
jangutter | It picked the correct topic, but it generated a URL where "#q," should be "#/q/" | 12:15 |
*** iurygregory has joined #openstack-infra | 12:18 | |
*** bobh has joined #openstack-infra | 12:18 | |
jangutter | I think it's this: https://github.com/openstack-infra/jeepyb/blob/master/jeepyb/cmd/update_blueprint.py#L92 | 12:19 |
*** wolverineav has quit IRC | 12:19 | |
*** rpittau is now known as rpittau|lunch | 12:20 | |
*** bobh has quit IRC | 12:24 | |
*** udesale has joined #openstack-infra | 12:28 | |
*** yamamoto_ has joined #openstack-infra | 12:29 | |
*** e0ne has quit IRC | 12:30 | |
*** yamamoto has quit IRC | 12:33 | |
*** AJaeger has quit IRC | 12:43 | |
*** ykarel|afk is now known as ykarel | 12:46 | |
*** yfried has joined #openstack-infra | 12:47 | |
yfried | Hi, is this the right place for pbr questions? | 12:47 |
*** rf0lc0 is now known as rfolco | 12:49 | |
*** AJaeger has joined #openstack-infra | 12:51 | |
fungi | yfried: #openstack-oslo is probably slightly more correct, but people here might know the answer too sure | 12:53 |
*** yamamoto_ has quit IRC | 12:54 | |
*** rlandy has joined #openstack-infra | 12:55 | |
*** yamamoto has joined #openstack-infra | 12:55 | |
*** rlandy is now known as rlandy|rover | 12:56 | |
*** yamamoto has quit IRC | 12:57 | |
fungi | ssbarnea: devstack-gate is an infra project, core reviewers on it are basically https://review.openstack.org/#/admin/groups/infra-core (plus sdague but he's not been around for a while) | 12:58 |
*** fuentess has joined #openstack-infra | 13:00 | |
yfried | fungi: tnx. I need to package all *.j2 files recursively. Do you know if pbr "data-files" can handle this? Do I need a MANIFEST.in file? | 13:00 |
fungi | jangutter: where on that review are you seeing the broken link? in the topic field of the detail box or somewhere else? | 13:03 |
jangutter | fungi: I'm seeing it on the blueprint whiteboard: https://blueprints.launchpad.net/nova/+spec/vrouter-os-vif-conversion (last line) | 13:04 |
*** jpena|lunch is now known as jpena | 13:04 | |
jangutter | fungi: I'm trying to find out where the '#/q/' url bit comes from and the best I could come up with is https://review.openstack.org/Documentation/config-gerrit.html#commentlink | 13:04 |
jangutter | fungi: pretty hard to google arbitrary weird junk. | 13:05 |
*** ykarel is now known as ykarel|afk | 13:06 | |
fungi | yfried: i do not know whether the data files configuration can take directories rather than individual files, though i expect that's up to setuptools not pbr itself | 13:06 |
openstackgerrit | Natal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr. https://review.openstack.org/629414 | 13:07 |
*** ykarel|afk is now known as ykarel | 13:07 | |
*** smarcet has joined #openstack-infra | 13:08 | |
yfried | fungi: in MANIFEST there's a "recursive-include" directive. I didn't see an equivalent in data-files | 13:08 |
*** trown|outtypewww is now known as trown | 13:09 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Default host_keys to empty list in static driver https://review.openstack.org/629478 | 13:11 |
*** yfried has quit IRC | 13:12 | |
fungi | jangutter: it looks like https://git.openstack.org/cgit/openstack-infra/jeepyb/tree/jeepyb/cmd/update_blueprint.py#n92 was never updated after the gerrit upgrade which invalidated the old url format | 13:14 |
jangutter | fungi: That line lasted 6 years! | 13:15 |
fungi | indeed | 13:16 |
jangutter | fungi: I've been trying to find where the url format is documented.... | 13:16 |
*** jcoufal has joined #openstack-infra | 13:16 | |
fungi | there was a non-backwards-compatible change in gerrit's url format around 2.11 or 2.12 | 13:16 |
fungi | unfortunately they never treated things like urls as a stable api | 13:17 |
*** dave-mccowan has joined #openstack-infra | 13:19 | |
jangutter | fungi: one would have thought a permalink would be a nice feature, sadly. | 13:19 |
*** rpittau|lunch is now known as rpittau | 13:21 | |
*** e0ne has joined #openstack-infra | 13:24 | |
*** yamamoto has joined #openstack-infra | 13:28 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Fixes from php7.2 migration https://review.openstack.org/629480 | 13:28 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Fixes from php7.2 migration https://review.openstack.org/629480 | 13:30 |
ssbarnea | fungi: thanks for pointing me to the gerrit group. probably i should print the membership to have it handy ;) | 13:38 |
fungi | ssbarnea: when in doubt, if you click the gear to the right of the project name when looking at a review in the gerrit webui (or go to projects->list at the top of the gerrit webui and search for the project), then switch to the access tab at the top you should hopefully see a group set for "Label Code-Review" with a -2 to +2 range; click that group name and you'll be on the membership list view for | 13:41 |
fungi | it | 13:42 |
fungi | that's how i usually go about finding out who to bug for reviews on something ;) | 13:42 |
*** dkehn has joined #openstack-infra | 13:49 | |
*** jamesmcarthur has joined #openstack-infra | 13:50 | |
*** ramishra has joined #openstack-infra | 13:50 | |
*** whoami-rajat has quit IRC | 13:52 | |
*** ramishra_ has quit IRC | 13:52 | |
*** yamamoto has quit IRC | 13:54 | |
mordred | AJaeger: https://review.openstack.org/#/c/629193/1/ -- very broken patch - but grenade-py3 is green - I think this means grenade-py3 isn't actually testing openstacksdk :) | 13:56 |
openstackgerrit | David Moreau Simard proposed openstack-infra/project-config master: Remove noop job for openstack/ara-web https://review.openstack.org/629489 | 13:59 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstack-zuul-jobs master: Updated laravel legacy jobs https://review.openstack.org/629495 | 14:08 |
fungi | mordred: seeing as how you're around, what's the best way to get the bindep role running after the ppa addition task in https://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/laravel-openstackid-unittests/run.yaml ? move it from the roles list to an include_role task in the appropriate part of the tasks list? | 14:11 |
mordred | fungi: ah - the ever popular problem of "what's the best way to deal with bindep+ppa" - which honestly doesnt' have a good answer | 14:21 |
*** kgiusti has joined #openstack-infra | 14:21 | |
mordred | fungi: my current take, which is more of giving up than a real answer is that things that need a PPA or other alternate repo to be installed shouldn't be in bindep.txt | 14:22 |
mordred | (thus the "install_js_tools.sh" in the zuul repo) | 14:22 |
mordred | fungi: but what I *REALLY | 14:22 |
frickler | mordred: seems we would need something like a grenade-tips job that includes some proper LIBS_FROM_GIT list? but certainly you have just proven that we can drop grenade-py3, as that doesn't stop broken patches at all | 14:23 |
mordred | fungi: but what I *REALLY* want is to add ppa support to bindep so that the whole thing could be self-contained | 14:23 |
*** boden has joined #openstack-infra | 14:24 | |
smarcet | fungi: mordred: morning guys please review https://review.openstack.org/#/c/629495/, thx | 14:25 |
mordred | frickler: yah. and grenade-py3 is still a legacy job, so I can't just pop a sensible override into the pipeline definition or aything | 14:25 |
*** jamesmcarthur has quit IRC | 14:26 | |
*** janki has quit IRC | 14:26 | |
*** janki has joined #openstack-infra | 14:27 | |
*** jamesmcarthur has joined #openstack-infra | 14:27 | |
*** yamamoto has joined #openstack-infra | 14:27 | |
mordred | fungi: oh - but yeah, to answer your actual question, move it to include_role: | 14:28 |
fungi | mordred: cool, so what smarcet did in 629495 i guess? | 14:29 |
mordred | ya | 14:29 |
fungi | smarcet: you're aware you can use "Depends-On: https://review.openstack.org/629495" in an openstackid change commit message to test whether that works, right? | 14:30 |
smarcet | fungi: yes sire | 14:30 |
smarcet | i will | 14:30 |
*** whoami-rajat has joined #openstack-infra | 14:31 | |
* fungi hasn't been called "sire" since his reactor days ;) | 14:31 | |
frickler | mordred: oh, indeed. so someone needs to take up andreaf's wip here https://review.openstack.org/548936 . tosky: do you still intend to continue with that? I could try to help a bit | 14:31 |
fungi | er, re-enactor | 14:31 |
*** yamamoto has quit IRC | 14:33 | |
openstackgerrit | Merged openstack-infra/nodepool master: Default host_keys to empty list in static driver https://review.openstack.org/629478 | 14:34 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Migration to PHP 7.x https://review.openstack.org/611936 | 14:35 |
*** eernst has quit IRC | 14:40 | |
openstackgerrit | Merged openstack-infra/zuul master: Clean up command sockets on stop https://review.openstack.org/628990 | 14:40 |
*** ykarel is now known as ykarel|away | 14:42 | |
openstackgerrit | Merged openstack-infra/zuul master: Ensure command_socket is last thing to close https://review.openstack.org/628995 | 14:43 |
*** janki has quit IRC | 14:47 | |
*** janki has joined #openstack-infra | 14:47 | |
*** janki has quit IRC | 14:51 | |
*** janki has joined #openstack-infra | 14:52 | |
*** janki has quit IRC | 14:53 | |
*** janki has joined #openstack-infra | 14:53 | |
*** janki has quit IRC | 14:54 | |
*** anteaya has joined #openstack-infra | 14:55 | |
*** janki has joined #openstack-infra | 14:55 | |
*** janki has quit IRC | 14:57 | |
*** janki has joined #openstack-infra | 14:57 | |
*** shardy has joined #openstack-infra | 14:58 | |
*** roman_g has joined #openstack-infra | 14:59 | |
*** janki has quit IRC | 14:59 | |
dtantsur | folks, anyone knows elastic-search here? how do I link a query to a story? | 15:00 |
openstackgerrit | Dmitry Tantsur proposed openstack-infra/elastic-recheck master: Create a query for story 2004744 https://review.openstack.org/629533 | 15:00 |
dtantsur | context ^^^ /cc TheJulia | 15:00 |
*** yboaron_ has joined #openstack-infra | 15:02 | |
*** yamamoto has joined #openstack-infra | 15:03 | |
*** arxcruz is now known as arxcruz|ruck | 15:03 | |
*** rlandy|rover is now known as rlandy | 15:04 | |
*** yboaron has quit IRC | 15:05 | |
*** ykarel|away is now known as ykarel | 15:06 | |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Create base-ozj base job https://review.openstack.org/587165 | 15:07 |
mordred | AJaeger: ^^ starting to work through the base-minimal-jobs stack | 15:07 |
*** yamamoto has quit IRC | 15:08 | |
TheJulia | dtantsur: thanks! | 15:08 |
*** chason_ has joined #openstack-infra | 15:09 | |
AJaeger | mordred: ok. | 15:10 |
*** ssbarnea has quit IRC | 15:10 | |
*** ssbarnea|rover has joined #openstack-infra | 15:10 | |
openstackgerrit | Merged openstack-infra/project-config master: Remove noop job for openstack/ara-web https://review.openstack.org/629489 | 15:11 |
odyssey4me | hi folks - on dec 6 I ran https://review.openstack.org/#/c/623253/ and it worked just fine (it's a cross-repo test), but now it's failing with http://logs.openstack.org/53/623253/1/check/openstack-ansible-deploy-aio_ceph-ubuntu-bionic/c5b5b9a/job-output.txt.gz#_2019-01-08_14_55_35_077278 (a very descriptive MODULE FAILURE) :/ | 15:15 |
odyssey4me | any ideas what may be causing that? | 15:15 |
odyssey4me | it's failing in this play: https://github.com/openstack/openstack-ansible/blob/master/zuul.d/playbooks/run.yml | 15:16 |
pabelanger | odyssey4me: http://logs.openstack.org/53/623253/1/check/openstack-ansible-deploy-aio_ceph-ubuntu-bionic/c5b5b9a/ara-report/result/b42c17fc-2cce-46ce-b036-7dfaface5611/ | 15:20 |
*** ykarel is now known as ykarel|away | 15:21 | |
odyssey4me | pabelanger lol, well - that'd do it | 15:21 |
pabelanger | /home/zuul/src/git.openstack.org/openstack/openstack-ansible-ceph_client was only created | 15:21 |
pabelanger | missing required-project? | 15:21 |
odyssey4me | pabelanger that may be - I'll add it to https://github.com/openstack/openstack-ansible/blob/master/zuul.d/jobs.yaml#L17 and see if that works | 15:22 |
odyssey4me | thanks! | 15:22 |
*** sara_nierodz has joined #openstack-infra | 15:23 | |
frickler | dtantsur: I think that this is a feature that hasn't been implemented yet | 15:23 |
dtantsur | sigh | 15:23 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Delete current base-minimal job https://review.openstack.org/587169 | 15:24 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Reset base-test playbooks https://review.openstack.org/587104 | 15:24 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add base-minimal / base-minimal-test playbooks https://review.openstack.org/587105 | 15:24 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Parent base-test to base-minimal-test https://review.openstack.org/587106 | 15:24 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Parent base to base-minimal https://review.openstack.org/587107 | 15:24 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Remove base-test now defined in openstack-zuul-jobs https://review.openstack.org/587113 | 15:24 |
*** weshay|ruck is now known as weshay | 15:24 | |
*** sara_nierodz is now known as snierodz_ | 15:24 | |
frickler | dtantsur: you could start by creating a story for that at https://storyboard.openstack.org/#!/project/713 ;) | 15:25 |
*** ykarel|away has quit IRC | 15:26 | |
openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Parent to base-ozj for testing https://review.openstack.org/587168 | 15:27 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Fixed issue with LV 5.6 upgrade https://review.openstack.org/629543 | 15:28 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Fixed issue with LV 5.6 upgrade https://review.openstack.org/629543 | 15:28 |
mordred | frickler, fungi: feel like +Aing https://review.openstack.org/#/c/587165 ? | 15:28 |
*** shardy has quit IRC | 15:29 | |
*** shardy has joined #openstack-infra | 15:29 | |
mordred | (it should be a very safe change as it doesn't do anything yet - main thing is that base-ozj should be identical to the current base-minimal) | 15:29 |
dtantsur | frickler: https://storyboard.openstack.org/#!/story/2004745 :) | 15:29 |
*** udesale has quit IRC | 15:34 | |
*** dpawlik has quit IRC | 15:35 | |
*** yamamoto has joined #openstack-infra | 15:36 | |
*** lpetrut has quit IRC | 15:37 | |
openstackgerrit | Merged openstack-infra/zuul master: Document missing executor stats https://review.openstack.org/626749 | 15:41 |
*** armax has joined #openstack-infra | 15:41 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Enforce max-job-timeout for jobs without configured timeout https://review.openstack.org/629552 | 15:42 |
*** yamamoto has quit IRC | 15:42 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Updated laravel legacy jobs https://review.openstack.org/629495 | 15:43 |
openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Add support for CentOS and RHEL for the install-nodejs role https://review.openstack.org/629554 | 15:45 |
*** quiquell is now known as quiquell|off | 15:45 | |
openstackgerrit | Natal Ngétal proposed openstack/diskimage-builder master: [Configuration] Switch to stestr. https://review.openstack.org/629414 | 15:47 |
openstackgerrit | Merged openstack-infra/project-config master: Create base-ozj base job https://review.openstack.org/587165 | 15:47 |
smcginnis | I'm experiencing the slow loading etherpad issue again. Anyone else seeing that? | 15:48 |
*** ykarel|away has joined #openstack-infra | 15:49 | |
openstackgerrit | Merged openstack-infra/zuul master: Fix skipped job counted as failed https://review.openstack.org/625910 | 15:49 |
*** wolverineav has joined #openstack-infra | 15:51 | |
*** ykarel|away is now known as ykarel | 15:52 | |
frickler | dtantsur: thanks. I guess in the meantime the best workaround would seem to be creating a bug in https://bugs.launchpad.net/openstack-gate and crosslinking that to your story and your query | 15:54 |
*** bhavikdbavishi has joined #openstack-infra | 15:54 | |
*** wolverineav has quit IRC | 15:55 | |
clarkb | we lost yfried but pbr should package all files that are managed by git. | 15:56 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Pull fetch-output back out of base https://review.openstack.org/629556 | 15:56 |
mordred | corvus, AJaeger: ^^ | 15:56 |
mordred | clarkb: putting them in the source repo and having them included in teh wheel as data-files are different | 15:57 |
mordred | s/repo/tarball/ | 15:57 |
mordred | if he wants data-files so that they get bundled in pkg-resources, then yeah, I think there would need to be a pbr patch to support recursing on directories or something | 15:58 |
AJaeger | frickler: btw. did you cleanup the 20 test nodes in ovh? Are they still needed? | 15:58 |
*** rcarrillocruz has joined #openstack-infra | 15:59 | |
mordred | corvus: so, I think the new patch to fetch-output wants to put docs and artifacts into logs if zuul.change is defined and otherwise put them into zuul.executor.work_root/{docs,artifacts} - yeah? | 16:00 |
clarkb | frickler: dtantsur: you should be able to paste kibana urls into stories as comments at the veryleast | 16:01 |
clarkb | but generally we've used elastic-recheck to drive that coordination of work | 16:01 |
corvus | mordred: i don't want to flip-flop on this. do we have the whole system written up somewhere? if not, can you do that? | 16:02 |
frickler | AJaeger: they were gone when I looked this morning, not sure who did the cleanup, but should be fine now anyway | 16:03 |
*** pcaruana has quit IRC | 16:03 | |
*** bhavikdbavishi has quit IRC | 16:04 | |
*** bhavikdbavishi has joined #openstack-infra | 16:04 | |
* frickler heads home now, bbl | 16:04 | |
mordred | corvus: http://lists.openstack.org/pipermail/openstack-infra/2017-October/005606.html | 16:06 |
mordred | corvus: I could put that into a spec though | 16:07 |
corvus | mordred: oh no, i don't need more words, i need less :) | 16:07 |
corvus | mordred: gimme a minute to digest the ml post | 16:07 |
mordred | corvus: scroll to "Artifacts and Docs" under "For OpenStack Today" | 16:08 |
*** yboaron_ has quit IRC | 16:09 | |
*** yamamoto has joined #openstack-infra | 16:10 | |
*** yboaron_ has joined #openstack-infra | 16:11 | |
corvus | mordred: yes. :) | 16:12 |
*** snierodz_ has quit IRC | 16:12 | |
mordred | corvus: actually - as I ponder that - I think the logic we're discussing wants to be a second role that moves things on the executor after fetch-output - beause uploading artifacts to the log server is an approach we're taking as openstack, but may not be how other deployers want to handle that | 16:13 |
corvus | mordred: (to your question at 16:00) | 16:13 |
corvus | mordred: that's fair | 16:13 |
mordred | corvus: so I think we want a second role which does the zuul.change based logic to put things into logs | 16:13 |
mordred | which in our case we'll put into the base job right after fetch-output | 16:13 |
corvus | mordred: yeah. i think *most* people will want it, but it will not hurt to have a second role and the system is nicer that way i think. | 16:14 |
tosky | frickler: if you have some time, be my guest! I'm a bit swamped with the M2 deadline for sahara, and I suppose it will continue a bit after M2 to clean up the fallout of two big changes landing in sahara | 16:14 |
mordred | corvus: ++ | 16:14 |
*** yamamoto has quit IRC | 16:14 | |
*** kjackal has quit IRC | 16:14 | |
mordred | corvus: hard question: what should that role be called? | 16:16 |
corvus | mordred: store-docs-and-artifacts-with-logs? log-output? merge-output? | 16:17 |
openstackgerrit | Merged openstack-infra/project-config master: Pull fetch-output back out of base https://review.openstack.org/629556 | 16:17 |
*** xarses_ has quit IRC | 16:17 | |
mordred | corvus: I like the word merge ... merge-output-to-logs ? | 16:18 |
*** gyee has joined #openstack-infra | 16:18 | |
corvus | wfm | 16:18 |
mordred | cool. I'll get that up in just a sec | 16:18 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Migration to PHP 7.x https://review.openstack.org/611936 | 16:21 |
*** shardy has quit IRC | 16:23 | |
*** armax has quit IRC | 16:25 | |
*** diablo_rojo has joined #openstack-infra | 16:26 | |
*** chkumar|out is now known as chandankumar | 16:27 | |
*** jamesmcarthur has quit IRC | 16:33 | |
*** jpena_ has joined #openstack-infra | 16:36 | |
*** nhicher has quit IRC | 16:39 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add validate-dco-license role https://review.openstack.org/629565 | 16:39 |
*** pabelanger has quit IRC | 16:39 | |
*** jpena has quit IRC | 16:39 | |
*** weshay has quit IRC | 16:40 | |
*** pabelanger has joined #openstack-infra | 16:40 | |
*** kjackal has joined #openstack-infra | 16:44 | |
*** ykarel is now known as ykarel|away | 16:44 | |
clarkb | logan-: as a datapoint I don't see anymoer of the tripleo subnode ssh failures since you restarted the neutron agents (we can keep watching that but so far looking good) | 16:45 |
clarkb | logan-: thank you for digging into that | 16:45 |
openstackgerrit | Merged openstack-infra/subunit2sql master: Fix inconsistent treatment of test status https://review.openstack.org/627196 | 16:45 |
*** jamesmcarthur has joined #openstack-infra | 16:45 | |
clarkb | amorin: http://grafana.openstack.org/d/BhcSH5Iiz/nodepool-ovh?orgId=1 shows we are using more of bhs1 now. Thank you | 16:46 |
*** rlandy is now known as rlandy|brb | 16:46 | |
*** eernst has joined #openstack-infra | 16:48 | |
*** ykarel|away has quit IRC | 16:49 | |
*** ykarel|away has joined #openstack-infra | 16:49 | |
*** lucasagomes has joined #openstack-infra | 16:50 | |
*** imacdonn has quit IRC | 16:51 | |
*** imacdonn has joined #openstack-infra | 16:51 | |
openstackgerrit | Merged openstack-infra/subunit2sql master: Replace DbMigrationError with DBMigrationError https://review.openstack.org/500858 | 16:59 |
*** e0ne has quit IRC | 17:03 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role to move docs and artifacts to log root https://review.openstack.org/629571 | 17:04 |
*** pgaxatte has quit IRC | 17:04 | |
*** rpittau has quit IRC | 17:08 | |
*** rlandy|brb is now known as rlandy | 17:09 | |
logan- | clarkb: awesome! no problem. catching up on backscroll I see your message yesterday afternoon -- will take a look at those hosts later this afternoon. | 17:09 |
*** dtantsur is now known as dtantsur|afk | 17:11 | |
openstackgerrit | sebastian marcet proposed openstack-infra/project-config master: Updated Tox Docs job to run in Bionic https://review.openstack.org/629579 | 17:12 |
AJaeger | frickler: great, thanks! | 17:12 |
*** eumel8 has quit IRC | 17:12 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid master: Migration to PHP 7.x https://review.openstack.org/611936 | 17:15 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role to move docs and artifacts to log root https://review.openstack.org/629571 | 17:16 |
*** armax has joined #openstack-infra | 17:17 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role to move docs and artifacts to log root https://review.openstack.org/629571 | 17:17 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add merge-output-to-logs to base-test https://review.openstack.org/629584 | 17:22 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Add fetch-output and merge-output-to-logs to base https://review.openstack.org/629585 | 17:22 |
*** Dobroslaw has quit IRC | 17:22 | |
*** yamamoto has joined #openstack-infra | 17:24 | |
*** irdr has quit IRC | 17:25 | |
*** yamamoto has quit IRC | 17:29 | |
*** smarcet has quit IRC | 17:29 | |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Upload artifacts in base-test job https://review.openstack.org/511860 | 17:31 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role to move docs and artifacts to log root https://review.openstack.org/629571 | 17:33 |
*** chandankumar is now known as chkumar|out | 17:42 | |
*** jpich has quit IRC | 17:43 | |
*** gfidente has quit IRC | 17:44 | |
*** wolverineav has joined #openstack-infra | 17:44 | |
*** aojea has quit IRC | 17:48 | |
clarkb | corvus: care to rereview https://review.openstack.org/#/c/629257/ I cleaned up some additional stuff that was original missed nad updated cacti | 17:48 |
clarkb | once that is in I can delete the old nameservers unless someone else would like to | 17:49 |
*** wolverineav has quit IRC | 17:49 | |
corvus | clarkb: ++ | 17:49 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-base-jobs master: Add fetch-output to base jobs https://review.openstack.org/628975 | 17:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-base-jobs master: Ignore errors on ssh key removal https://review.openstack.org/628976 | 17:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-base-jobs master: Add merge-output-to-logs to base job https://review.openstack.org/629604 | 17:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role to move docs and artifacts to log root https://review.openstack.org/629571 | 17:52 |
*** trown is now known as trown|lunch | 17:55 | |
*** derekh has quit IRC | 17:56 | |
*** yamamoto has joined #openstack-infra | 17:56 | |
*** ginopc has quit IRC | 17:59 | |
*** sshnaidm is now known as sshnaidm|afk | 18:01 | |
*** yamamoto has quit IRC | 18:01 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-base-jobs master: Add merge-output-to-logs to base job https://review.openstack.org/629604 | 18:07 |
openstackgerrit | Mark Goddard proposed openstack-infra/irc-meetings master: Add kayobe team meeting https://review.openstack.org/629612 | 18:10 |
*** jpena_ is now known as jpena|off | 18:12 | |
openstackgerrit | Merged openstack-infra/nodepool master: Extract out common config parsing for ConfigPool https://review.openstack.org/621642 | 18:14 |
*** bhavikdbavishi1 has joined #openstack-infra | 18:15 | |
*** armax has quit IRC | 18:16 | |
*** bhavikdbavishi has quit IRC | 18:16 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 18:16 | |
*** armax has joined #openstack-infra | 18:16 | |
*** dpawlik has joined #openstack-infra | 18:16 | |
*** bhavikdbavishi has quit IRC | 18:20 | |
*** wolverineav has joined #openstack-infra | 18:27 | |
*** wolverineav has quit IRC | 18:30 | |
*** wolverineav has joined #openstack-infra | 18:31 | |
*** lpetrut has joined #openstack-infra | 18:33 | |
*** ykarel|away has quit IRC | 18:34 | |
*** e0ne has joined #openstack-infra | 18:35 | |
*** jamesmcarthur has quit IRC | 18:36 | |
*** lpetrut has quit IRC | 18:37 | |
*** irdr has joined #openstack-infra | 18:47 | |
*** electrofelix has quit IRC | 18:55 | |
*** yboaron_ has quit IRC | 18:57 | |
*** e0ne has quit IRC | 19:02 | |
*** jamesmcarthur has joined #openstack-infra | 19:04 | |
*** e0ne has joined #openstack-infra | 19:05 | |
*** trown|lunch is now known as trown | 19:08 | |
*** harlowja has joined #openstack-infra | 19:10 | |
*** yamamoto has joined #openstack-infra | 19:16 | |
amorin | clarkb: no pb | 19:17 |
*** armax has quit IRC | 19:17 | |
*** panda has quit IRC | 19:19 | |
*** yamamoto has quit IRC | 19:21 | |
*** panda has joined #openstack-infra | 19:21 | |
*** rkukura has quit IRC | 19:25 | |
fungi | okay, so i'm just now getting to trying to launch a new storyboard-dev01.opendev.org server and it's not applying any puppet that i can tell. the hostname matches the pattern at https://git.openstack.org/cgit/openstack-infra/system-config/tree/manifests/site.pp#n592 so what else am i missing? does it need to be added to our static manifest first? | 19:28 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Enforce max-job-timeout for jobs without configured timeout https://review.openstack.org/629552 | 19:29 |
*** lathiat has quit IRC | 19:29 | |
clarkb | fungi: the way it works now is that all it does is apply our base ansible server stuff | 19:31 |
clarkb | fungi: then you add it to the inventory and kick.sh or wait for the next run to puppet it | 19:32 |
fungi | aha, thanks. so working as expected? | 19:32 |
openstackgerrit | Merged openstack-infra/zuul master: Add cgroup support to ram sensor https://review.openstack.org/549506 | 19:32 |
clarkb | yes. Also keep in mind you may want to reboot after puppeting (I think this is the biggest gotcha, that was automagic before) | 19:32 |
fungi | probably warrants expanding what we've got in https://git.openstack.org/cgit/openstack-infra/system-config/tree/launch/README | 19:32 |
clarkb | ++ | 19:32 |
fungi | i'll see what i can work out | 19:32 |
pabelanger | clarkb: corvus: ianw: So, I've been hacking away on https://specs.openstack.org/openstack-infra/infra-specs/specs/zuulv3-3rd-party-ci.html over the last few weeks, I'm basically at the point now in my local environment to start setting up pipelines in zuul and run a job in check. One thing I'd actually like to experiment with, is seeing if we can load the pipelines for 3pci directly from git.o.o. I | 19:34 |
pabelanger | was thinking we could use project-config-example or sandbox to test, and if works figure out the right location (or we can decided now). The main thing I want to confirm, is if we merge a pipeline stanza into untrusted job, is it won't affect zuul.o.o. I guess we could also update main.yaml to ignore pipelines from a specific project too | 19:34 |
clarkb | pabelanger: we might also want to double check that the check pipeline is common enough among what 3pci are doing | 19:36 |
clarkb | pabelanger: many of them due custom triggers so that you can recheck only their jobs | 19:36 |
clarkb | (and that won't work in a centrally hosted config repo) | 19:36 |
pabelanger | clarkb: yah, that is the trade off I think, custom triggers. | 19:36 |
corvus | clarkb: well, we can host a config that we recommend, and since we don't recommend custom triggers, job done. | 19:37 |
clarkb | corvus: ya but if no one uses it... | 19:37 |
clarkb | then maybe we are better off making it easier to load a config locally | 19:37 |
corvus | well, it is easy :) | 19:37 |
corvus | just fork the repo and you're done | 19:38 |
corvus | either way, i think pabelanger's idea has merit | 19:38 |
corvus | (you can use the repo directly and get our recommended config, you can fork the repo and modify it to suit if you want to go off-script) | 19:38 |
clarkb | ya | 19:38 |
clarkb | and I want to say that zuul will refuse to load pipelines out of untrusted repos but its been a while since I reviewed the config loading code | 19:39 |
pabelanger | yah, I cannot remember myself | 19:39 |
corvus | i believe it's an error, so we'd want a repo with pipelines excluded from the config objects for that repo | 19:40 |
clarkb | ssbarnea|rover: I've reviewed some of your d-g changes. Left some comments. Thinking beyond those changes is tripleo planning to stop using devstack-gate? I don't think the infra team intends to support it long term (and eg update the ansible version in use there) | 19:41 |
corvus | pabelanger: (if you just propose a change to sandbox with a pipeline, i expect zuul to return an error) | 19:41 |
clarkb | corvus: ah | 19:41 |
pabelanger | corvus: okay, let me try first, and then prepare a patch to main.yaml | 19:41 |
clarkb | ssbarnea|rover: if we can help with sketching out what a conversion to zuulv3 native jobs looks like I think we'd be happy to do that. | 19:42 |
ssbarnea|rover | clarkb: in fact is not tripleo that is using the devstack-gate, i found it on some other projects (need to check which one) | 19:43 |
clarkb | ssbarnea|rover: ok whichever projects they are I think the ideal would be to work with them to update to zuulv3 native jobs rather than continue to fix or improve devstack-gate | 19:44 |
clarkb | (in the short term we may need to do both) | 19:44 |
*** rkukura has joined #openstack-infra | 19:44 | |
fungi | grenade is still the elephant in the room | 19:44 |
ssbarnea|rover | clarkb: lets try to do a grep to find out who are the users | 19:44 |
clarkb | fungi: ya I think frickler intends to target that one specifically | 19:45 |
fungi | tosky had picked up the effort to get grenade jobs onto zuul v3 native configuration, but didn't have time to finish | 19:45 |
corvus | https://review.openstack.org/#/c/548936/ looks to be the effort in question | 19:49 |
openstackgerrit | Sorin Sbarnea proposed openstack-infra/zuul master: docs: Added missing -i on docker-compose up command https://review.openstack.org/629652 | 19:52 |
fungi | yep | 19:55 |
fungi | i asked him about it back before the holidays and he said he was hoping to find time to resume work on it, but i guess he hasn't been able to yet | 19:56 |
fungi | if memory serves, he wanted it for sahara so that he could more easily extend the job for their devstack plugin or something | 19:56 |
clarkb | a quick look at that shows the framework is in place though which is good | 19:57 |
clarkb | its mostly a matter of adding the run grenade steps to https://review.openstack.org/#/c/548936/24/playbooks/grenade.yaml ? | 19:57 |
openstackgerrit | Merged openstack-infra/system-config master: Nameservers are now managed with ansible https://review.openstack.org/629257 | 19:57 |
clarkb | oh hrm no its breaking grenade apart in the job | 19:57 |
clarkb | it might be easier to cmd: ./grenade.sh | 20:02 |
clarkb | but I don't have enough fo that context switched in to know for sure | 20:02 |
fungi | cmurphy: clarkb: the futureparser list also has incorrect globs for storyboard and storyboard-dev servers, i'm noticing, but since i'm getting ready to replace the servers maybe that's okay? | 20:04 |
clarkb | fungi: do the globs match only the new or the old servers? | 20:06 |
fungi | clarkb: only the new servers | 20:06 |
clarkb | if they only match the new then I think its fine. We should be able to quickly fix any puppet issues you run into on the new servers | 20:06 |
*** efried has joined #openstack-infra | 20:06 | |
fungi | same error which was made with the mailman hostname globs | 20:06 |
fungi | [0-9]* means a digit followed by anything, not zero or more digits | 20:06 |
clarkb | ya | 20:07 |
clarkb | the main issue I've run into is apache vhosts not templating properly due to variable scoping errors. TO fix those you switch to the scope lookupvar function in the erb templates | 20:08 |
*** e0ne has quit IRC | 20:08 | |
*** dpawlik has quit IRC | 20:08 | |
fungi | yeah, i'll keep an eye out | 20:09 |
fungi | out of curiosity, is there a reason we need the private_v4 addresses in our static inventory? | 20:10 |
clarkb | fungi: I am not aware of one | 20:13 |
fungi | mordred: ^ you may know why we included it? | 20:13 |
fungi | seems like tracking a useless piece of data, but odds are i'm just not thinking of the reason we needed it | 20:14 |
openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Prevent gitea pods from being co-scheduled https://review.openstack.org/629655 | 20:14 |
clarkb | dansmith: do you know the context of https://review.openstack.org/#/c/487664/4 I think mriedem is out this week and want to double check that the jobs that need this debugging run legacy devstack-gate instead of the new devstack stuff | 20:14 |
dansmith | clarkb: I don't | 20:14 |
mordred | fungi: I think it's just that we had been tracking it before - I do not believe we've ever used that for anything | 20:15 |
mordred | fungi: I would be perfectly fine excluding that data | 20:16 |
*** ldnunes has quit IRC | 20:16 | |
fungi | thanks. i guess we should consider that if nobody comes up with a good reason not to | 20:16 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config master: Add storyboard-dev01.opendev.org to inventory https://review.openstack.org/629656 | 20:19 |
clarkb | going to grab lunch now. I'll double check puppet stopped running on the old nameservers, delete them and clean up dns records for them this afternoon. Then I'll try to catch up on any fedora29 stuff that needs to ahppen and then hopefully have time to do administrivia | 20:20 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/zone-opendev.org master: Add storyboard-dev01.opendev.org https://review.openstack.org/629657 | 20:24 |
fungi | i think the error i'm getting from rackdns when trying to set the reverse ptr has to do with the address records not resolving | 20:25 |
fungi | oh, nope, it has to do with me being stupid | 20:26 |
fungi | i was pasting the commands to add the address records via rackdns :/ | 20:26 |
tosky | fungi, clarkb : frickler said earlier on this channel that he may have timeto work on it | 20:27 |
clarkb | fungi new domain :) | 20:27 |
fungi | tosky: yep, fingers crossed! | 20:27 |
tosky | I'm a bit swamped with some structural sahara changes | 20:27 |
fungi | happens to all of us | 20:27 |
fungi | clarkb: mordred: storyboard-dev01.opendev.org server is booted and has reverse dns, once 629656 and 629657 get reviewed i should be able to proceed with testing whether it gets puppeted | 20:28 |
*** smarcet has joined #openstack-infra | 20:35 | |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-gerrit master: Add support for specifying which refs to replicate https://review.openstack.org/629658 | 20:35 |
*** slaweq has quit IRC | 20:37 | |
*** efried has quit IRC | 20:40 | |
openstackgerrit | Monty Taylor proposed openstack-infra/system-config master: Only replicate branches and tags to gitea https://review.openstack.org/629660 | 20:41 |
*** kjackal has quit IRC | 20:41 | |
clarkb | fungi: +2 on both. For the first one you might want ot just approve with one vote? its an update for a dev server. The other affects productio ndns so is a little more important but also not used for much yet :) | 20:41 |
openstackgerrit | Merged openstack-infra/zuul master: docs: Added missing -i on docker-compose up command https://review.openstack.org/629652 | 20:43 |
*** markvoelker has joined #openstack-infra | 20:45 | |
fungi | thanks! | 20:48 |
*** slaweq has joined #openstack-infra | 20:53 | |
*** yboaron_ has joined #openstack-infra | 21:02 | |
*** yamamoto has joined #openstack-infra | 21:04 | |
*** yamamoto has quit IRC | 21:09 | |
clarkb | I've stopped nsd on ns1.openstack.org and ns2.openstack.org | 21:12 |
*** ociuhandu has joined #openstack-infra | 21:13 | |
clarkb | let me know if you notice any dns issues. I'll remove A and AAAA records for them next | 21:14 |
fungi | awesome | 21:15 |
clarkb | following up on fedora-29 it seems that the step we are at is waiting for nodepool to build images and boot them | 21:19 |
clarkb | the change to make this happen, https://review.openstack.org/#/c/618672/3/nodepool/nodepool.yaml is merged and applied but nodepool isn't doing it yet | 21:20 |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-gerrit master: Add support for specifying which refs to replicate https://review.openstack.org/629658 | 21:20 |
clarkb | I'm initially stumped as to why that would be | 21:22 |
clarkb | do we need to restart builders to pick up new config? that may actually be it | 21:23 |
clarkb | I'm going to restart the builders on nb01 and nb02 to check | 21:25 |
mordred | clarkb: I thought nodepool would notice new config | 21:27 |
mordred | Shrews: ^^ ? | 21:27 |
Shrews | clarkb: i don't *think* a restart is necessary | 21:27 |
*** jamesmcarthur has quit IRC | 21:27 | |
Shrews | but i cannot think of any reason why they aren't picked up yet. been a while since i've thought about any builder stuff | 21:27 |
*** jamesmcarthur has joined #openstack-infra | 21:29 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for afs servers https://review.openstack.org/629662 | 21:29 |
openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Turn on future parser for kerberos servers https://review.openstack.org/629663 | 21:29 |
clarkb | mordred: Shrews well both builders were out to lunch. nb01 since the 4th and nb02 since 2018 somtime | 21:30 |
clarkb | both were hungup on dib builds that were trying t oclone git repos | 21:30 |
clarkb | I've cleaned that up and restarted the builders and now image builds are happening again | 21:30 |
clarkb | my guess is fedora-29 builds are queued up behind the backlog? | 21:31 |
clarkb | Shrews: we might want to add a timeout on the dib fork? | 21:31 |
clarkb | like say 8 hours | 21:31 |
* clarkb context switches to cleaning up dns | 21:31 | |
clarkb | but ya I think we may have queued things up but stuff got stuck behind the backlog. We should know in a few hours | 21:34 |
fungi | so it probably did pick up the config update automatically, but was basically stuck so didn't act on it? | 21:34 |
clarkb | that is my theory | 21:36 |
mordred | sounds right | 21:36 |
clarkb | it should be unstuck now and once it clears the backlog we should see fedora-29 | 21:36 |
clarkb | infra-root I see an ns3.openstack.org A record (not sure about AAAA yet, can I remove that too?) | 21:36 |
clarkb | I'm removing A and AAAA records for adns1.openstack.org ns1.openstack.org and ns2.openstack.org now | 21:37 |
fungi | mordred: is the ref replication filter in service of not replicating change refs to gitea? if so, curious why it's a problem | 21:37 |
openstackgerrit | Merged openstack-infra/system-config master: Add storyboard-dev01.opendev.org to inventory https://review.openstack.org/629656 | 21:39 |
fungi | clarkb: those ip addresses for ns3 respond to ping, and the ssh banner claims it's an ubuntu server | 21:39 |
*** kgiusti has left #openstack-infra | 21:39 | |
fungi | may be timing out trying to perform reverse dns lookups on my client though, ssh is hanging for me so far | 21:39 |
*** ociuhandu has quit IRC | 21:39 | |
mordred | fungi: yes, in fact it is, and is consumed here: https://review.openstack.org/629660 | 21:40 |
mordred | there's two things - which I think are likely both solvable in time ... | 21:40 |
openstackgerrit | Merged openstack-infra/nodepool master: Extract common config parsing for ProviderConfig https://review.openstack.org/625094 | 21:41 |
fungi | i see, the commit message is fairly enlightening | 21:41 |
mordred | \o/ commit messages | 21:41 |
clarkb | corvus: ^ do you know what ns3.openstack.org is? | 21:42 |
openstackgerrit | Colleen Murphy proposed openstack-infra/system-config master: Update review-dev.o.o to puppet 4 https://review.openstack.org/629667 | 21:42 |
clarkb | at this point I think the last step for adns1.openstack.org ns1.openstack.org and ns2.openstack.org is to delete them | 21:42 |
fungi | the ip addresses for ns3 are in ord, not dfw | 21:45 |
fungi | i wonder if that was an attempt at going cross-region with those nameservers but then later deleted and the dns entries for it never cleaned up? | 21:47 |
tosky | I may have asked it already: is it still possible to use the finger client to access the logs of running zuul jobs? I remember that I could do it when the old zuul interface was used | 21:48 |
clarkb | fungi: ya I don't see it in our ord server listing | 21:48 |
fungi | you can, yes | 21:48 |
clarkb | adns1 is there though | 21:48 |
mordred | tosky: finger d59ac94eb0374f3ba8b0cc8be2341c61@zuul.openstack.org | 21:50 |
fungi | tosky: it's `finger xxxxxxx@zuul.openstack.org` where the xxxxxxx is the build id | 21:50 |
mordred | tosky: is the pattern | 21:50 |
fungi | and you can grab the build id from the web console stream url | 21:51 |
tosky | oh, right | 21:51 |
tosky | thanks | 21:51 |
tosky | much faster than the web interface | 21:52 |
fungi | So for http://zuul.openstack.org/stream/81ab3f251c32426380d5dfa83f17cc33?logfile=console.log you can `finger 81ab3f251c32426380d5dfa83f17cc33@zuul.openstack.org` | 21:52 |
clarkb | do we think we need any snapshots of these servers? I'm guessing no since the data they hosted is/was all in zone files and hiera? | 21:53 |
fungi | clarkb: there should be nothing of importance on them now | 21:53 |
fungi | unless someone put something they cared about in their homedir | 21:53 |
mordred | fungi: it's where I keep my warez | 21:54 |
fungi | great idea | 21:54 |
clarkb | I will be deleting f8c9e5d6-818f-4168-a53e-414c7e3ccb34 adns1.openstack.org in ORD, af56cafc-6a3d-4ffb-b443-932ece962673 ns1.openstack.org in DFW, and c4f203c0-5315-45ac-ab11-dbce5bd33d67 ns2.openstack.org in ca-ymq-1 | 21:55 |
clarkb | infra-root ^ last call | 21:57 |
*** fuentess has quit IRC | 21:58 | |
clarkb | #status log deleted f8c9e5d6-818f-4168-a53e-414c7e3ccb34 adns1.openstack.org in ORD, af56cafc-6a3d-4ffb-b443-932ece962673 ns1.openstack.org in DFW, and c4f203c0-5315-45ac-ab11-dbce5bd33d67 ns2.openstack.org in ca-ymq-1. DNS now hosted by opendev nameservers | 22:07 |
openstackstatus | clarkb: finished logging | 22:07 |
clarkb | corvus: ^ the last remaining thing is whether or not A and AAAA records for ns3.openstack.org can be removed | 22:07 |
*** ccamacho has quit IRC | 22:08 | |
clarkb | | fedora-29-0000000001 | fedora-29 | nb02 | qcow2,raw,vhd | building | 00:00:00:24 | | 22:10 |
clarkb | that must've been it. stuck in the queue behind stuck dib builds | 22:10 |
*** trown is now known as trown|outtypewww | 22:10 | |
corvus | clarkb: i think so | 22:10 |
*** smarcet has left #openstack-infra | 22:12 | |
clarkb | I'll get those cleaned up too then | 22:13 |
clarkb | fungi: corvus reverse dns lookup on the ipv4 addr says its artfixdaily.com | 22:20 |
clarkb | so ya hsould be safe to remove :) | 22:21 |
clarkb | #status log removed ns3.openstack.org A 166.78.116.117 and ns3.openstack.org AAAA 2001:4801:7825:103:be76:4eff:fe10:4f7a as these IPs don't seem to belong to us anymore. | 22:23 |
*** lujinluo has joined #openstack-infra | 22:23 | |
openstackstatus | clarkb: finished logging | 22:23 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Fixed cloud assets bugs * missing lib * missing real path https://review.openstack.org/629672 | 22:23 |
fungi | clarkb: yeah, i saw that ptr record too, but don't trust that it wasn't something stale prior to our use of the address either | 22:25 |
*** ccamacho has joined #openstack-infra | 22:27 | |
clarkb | fungi: well it resolve sforward and their website is up at that addr | 22:28 |
fungi | i've seen rackspace fail to scrub ptr records in the past, but yeah | 22:28 |
*** ccamacho has quit IRC | 22:28 | |
fungi | seems to not be ours at any rate | 22:28 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Fixed cloud assets bugs * missing lib * missing real path https://review.openstack.org/629672 | 22:35 |
*** boden has quit IRC | 22:38 | |
fungi | clarkb: do you happen to know whether we still need the group variables in the global site manifest, or whether those are vestigial now? | 22:46 |
clarkb | fungi: we use those for hiera lookups | 22:47 |
fungi | okay, so mapping in inventory/groups.yaml alone is still not sufficient | 22:47 |
clarkb | correct, that mapping is for the ansible side of thing s(tells ansible which hiera files to copy for example) | 22:47 |
clarkb | then on the puppet side the $group var tells puppet how to lookup those values | 22:47 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config master: Create a storyboard-dev hiera lookup group https://review.openstack.org/629674 | 22:49 |
fungi | clarkb: ^ so like that? | 22:49 |
fungi | i've copied host_vars/storyboard-dev.openstack.org.yaml to group_vars/storyboard-dev.yaml on bridge.o.o as well | 22:51 |
clarkb | yes | 22:51 |
fungi | and i guess the old host_vars file can go away once that merges | 22:51 |
*** smarcet has joined #openstack-infra | 22:51 | |
*** rcernin has joined #openstack-infra | 22:52 | |
*** smarcet has quit IRC | 22:52 | |
*** yamamoto has joined #openstack-infra | 22:52 | |
clarkb | yup. At one point I cleaned up a fair bit of redundant files like that | 22:52 |
clarkb | I imagine there is more we could do though | 22:52 |
*** diablo_rojo has quit IRC | 22:53 | |
*** yamamoto has quit IRC | 22:56 | |
*** markvoelker has quit IRC | 22:57 | |
*** markvoelker has joined #openstack-infra | 22:57 | |
*** markvoelker has quit IRC | 23:02 | |
*** jcoufal has quit IRC | 23:03 | |
*** whoami-rajat has quit IRC | 23:04 | |
clarkb | | fedora-29-0000000001 | fedora-29 | nb02 | qcow2,raw,vhd | ready | 00:00:00:06 | | 23:04 |
clarkb | and now we wait for the image to boot | 23:05 |
ianw | clarkb: yeah, that was weird, thanks just checking on that | 23:05 |
*** jamesmcarthur has quit IRC | 23:07 | |
fungi | this seems like not the only time in recent history all our nodepool builders were stuck | 23:08 |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Add CentOS NetworkManager testing node https://review.openstack.org/629680 | 23:08 |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Revert "Add CentOS NetworkManager testing node" https://review.openstack.org/629681 | 23:08 |
fungi | didn't the same thing happen a month or two back? | 23:08 |
ianw | fungi: hrm, don't remember that. i used to have monitoring scripts, but when we split into multiple builders it became harder | 23:09 |
clarkb | ya they were broken when we got back from summit in berlin | 23:09 |
fungi | and here i was about to chalk that memory up to a feverish, eggnog-induced dream | 23:10 |
*** jamesmcarthur has joined #openstack-infra | 23:10 | |
*** jamesmcarthur has quit IRC | 23:11 | |
clarkb | I'm trying to remember those circumstances, they had both broken on the same image build iirc | 23:11 |
clarkb | and I want to say theory was something particular to that build at that time was the cause? | 23:11 |
*** diablo_rojo has joined #openstack-infra | 23:12 | |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Run k8s-on-openstack to manage k8s control plane https://review.openstack.org/626965 | 23:16 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add resources for deploying rook and xtradb to kuberenets https://review.openstack.org/626054 | 23:16 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Add gitea k8s resource definitions and playbook https://review.openstack.org/626759 | 23:16 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Prevent gitea pods from being co-scheduled https://review.openstack.org/629655 | 23:16 |
openstackgerrit | James E. Blair proposed openstack-infra/system-config master: Use project-config when syncing gitea repos https://review.openstack.org/629683 | 23:16 |
corvus | mordred: ^ er, most of that was rebasing i think. | 23:17 |
corvus | i'm running the contents of the last change now to create gitea repos | 23:17 |
*** jamesmcarthur has joined #openstack-infra | 23:18 | |
corvus | http://38.108.68.64/ is today's gitea url | 23:19 |
clarkb | now I want tea | 23:20 |
corvus | me too | 23:20 |
corvus | i think i'll make some | 23:20 |
corvus | (it's up to openstack/deb-python-heatclient) | 23:20 |
fungi | without heat, the tea takes too long to steep | 23:22 |
*** rlandy has quit IRC | 23:25 | |
openstackgerrit | Merged openstack-infra/system-config master: Install openshift module on bridge https://review.openstack.org/626748 | 23:27 |
*** jamesmcarthur has quit IRC | 23:30 | |
*** yamamoto has joined #openstack-infra | 23:30 | |
ianw | [root@fedora-29-vexxhost-vexxhost-sjc1-0001606322 ~]# nmcli c | 23:31 |
ianw | NAME UUID TYPE DEVICE | 23:31 |
ianw | System ens3 21d47e65-8523-1a06-af22-6f121086f085 ethernet ens3 | 23:31 |
ianw | that simple command took quite a lot of building blocks to get to! | 23:31 |
*** jamesmcarthur has joined #openstack-infra | 23:32 | |
clarkb | ianw: re centos7 network manager, we could also potentially just make the switch with 8 and leave 7 as is? | 23:32 |
*** lathiat has joined #openstack-infra | 23:34 | |
*** jamesmcarthur has quit IRC | 23:35 | |
ianw | my thinking there was that it's supported on 7, so it's not a weird thing, and so reducing or differences to 8 helps, and also making it so we just support one path in infra helps us too | 23:35 |
clarkb | ya having a common toolset is nice | 23:36 |
*** jamesmcarthur has joined #openstack-infra | 23:36 | |
ianw | if it was something as big as systemd/sysv type change, i'd think running in parallel ... but in this case i think basically nothing should notice | 23:37 |
*** jamesmcarthur has quit IRC | 23:37 | |
ianw | (insert fingers crossed emoji) | 23:37 |
*** rkukura has quit IRC | 23:37 | |
ianw | 🤞 | 23:37 |
clarkb | my font unfortunately did not render that | 23:38 |
anteaya | I also lack the fingers crossed emoji | 23:39 |
openstackgerrit | Merged openstack-infra/system-config master: Create a storyboard-dev hiera lookup group https://review.openstack.org/629674 | 23:42 |
*** rkukura has joined #openstack-infra | 23:46 | |
clarkb | ianw: are we ready to bump fedora-latest to 29 now that image is built and bootable? | 23:50 |
ianw | clarkb: yeah, that was the intention -> https://review.openstack.org/#/c/618673/ | 23:51 |
ianw | i guess my prior mail didn't really cover this ... i wonder what's using it | 23:52 |
ianw | mostly us, ansible-* projects, couple of others | 23:54 |
openstackgerrit | John Studarus proposed openstack-infra/nodepool master: Work in Progress https://review.openstack.org/629688 | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!