*** rkukura has joined #openstack-infra | 00:02 | |
*** slaweq has joined #openstack-infra | 00:11 | |
*** slaweq has quit IRC | 00:15 | |
*** rcernin has quit IRC | 00:18 | |
*** rcernin has joined #openstack-infra | 00:19 | |
*** markvoelker has quit IRC | 00:20 | |
*** ddurst has quit IRC | 00:30 | |
*** dklyle has quit IRC | 00:32 | |
*** efried_2dmtg has quit IRC | 00:35 | |
*** ddurst has joined #openstack-infra | 00:35 | |
*** rh-jelabarre has joined #openstack-infra | 00:38 | |
*** dklyle has joined #openstack-infra | 00:48 | |
*** bgmccollum has quit IRC | 00:48 | |
*** rlandy has quit IRC | 00:48 | |
*** wolverineav has quit IRC | 00:50 | |
*** wolverineav has joined #openstack-infra | 01:04 | |
*** whoami-rajat has joined #openstack-infra | 01:06 | |
*** gyee has quit IRC | 01:10 | |
*** dklyle has quit IRC | 01:10 | |
*** bobh has joined #openstack-infra | 01:16 | |
*** bgmccollum has joined #openstack-infra | 01:17 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add openstack/ansible-role-borgbackup https://review.openstack.org/633095 | 01:19 |
---|---|---|
*** bobh has quit IRC | 01:24 | |
*** bobh has joined #openstack-infra | 01:26 | |
*** rkukura has quit IRC | 01:35 | |
*** ijw has quit IRC | 01:37 | |
*** rkukura has joined #openstack-infra | 01:43 | |
*** bobh has quit IRC | 01:50 | |
*** agopi has joined #openstack-infra | 01:51 | |
*** ijw has joined #openstack-infra | 01:53 | |
*** rkukura has quit IRC | 01:55 | |
*** wolverineav has quit IRC | 01:56 | |
*** ijw has quit IRC | 01:57 | |
*** armax has quit IRC | 02:03 | |
*** wolverineav has joined #openstack-infra | 02:05 | |
*** wolverineav has quit IRC | 02:11 | |
*** slaweq has joined #openstack-infra | 02:11 | |
openstackgerrit | Merged openstack-infra/grafyaml master: Removed older version of python3.4 https://review.openstack.org/628751 | 02:15 |
*** slaweq has quit IRC | 02:15 | |
*** markvoelker has joined #openstack-infra | 02:25 | |
*** hongbin has joined #openstack-infra | 02:25 | |
*** wolverineav has joined #openstack-infra | 02:47 | |
*** bhavikdbavishi has joined #openstack-infra | 02:47 | |
*** eernst has joined #openstack-infra | 02:50 | |
*** Qianbiao has joined #openstack-infra | 02:50 | |
*** eernst has quit IRC | 02:50 | |
Qianbiao | good morning | 02:50 |
Qianbiao | I am building a CI and testing it on openstack-dev/ci-sandbox now | 02:51 |
Qianbiao | gerrit review succecced but gerrit approve failed. any idea? | 02:51 |
*** bhavikdbavishi has quit IRC | 02:52 | |
ianw | Qianbiao: do you mean having your 3rd party ci vote on changes? | 02:54 |
Qianbiao | yes | 02:54 |
Qianbiao | https://review.openstack.org/#/c/630893/ | 02:54 |
Qianbiao | i am testing this patch now | 02:54 |
*** markvoelker has quit IRC | 02:55 | |
Qianbiao | ianw | 02:56 |
Qianbiao | ianw gerrit review works now, but gerrit approve failed | 02:57 |
Qianbiao | Not sure why. It says that ci-sandbox project is public, not privileges required. | 02:57 |
*** chandankumar has quit IRC | 02:57 | |
Qianbiao | anyone should have permission to gerrit approve | 02:58 |
*** chandankumar has joined #openstack-infra | 02:59 | |
*** bhavikdbavishi has joined #openstack-infra | 02:59 | |
ianw | yes, it should. it looks like Huawei Ironic CI left a +1 vote on PS6 | 02:59 |
*** agopi has quit IRC | 03:01 | |
Qianbiao | ianw it does not show in workflow label | 03:03 |
*** psachin has joined #openstack-infra | 03:03 | |
ianw | Qianbiao: it hasn't put a +W ? i can see just now it left a +1 vote that shows up | 03:04 |
Qianbiao | ianw not sure why +W means. | 03:05 |
Qianbiao | i use ssh -i /var/lib/jenkins/.ssh/id_rsa -p 29418 huawei-ironic-ci@review.openstack.org gerrit review 630893,7 --message 'Build Successful' --verified 1 --code-review 0 | 03:05 |
Qianbiao | it works. | 03:05 |
Qianbiao | ssh -i /var/lib/jenkins/.ssh/id_rsa -p 29418 huawei-ironic-ci@review.openstack.org gerrit approve 630893,7 --message 'Build Successful' --verified 1 --code-review 0 | 03:06 |
Qianbiao | it failed. | 03:06 |
*** udesale has joined #openstack-infra | 03:06 | |
ianw | Qianbiao: "approve" isn't a thing; we have the "workflow" label, which with sufficient verified +2's triggers zuul to merge it on our behalf | 03:10 |
Qianbiao | i see. do u mean if i change --verified 1 to --verified 2, | 03:12 |
Qianbiao | it then will appear in workflow label? | 03:12 |
*** bobh has joined #openstack-infra | 03:12 | |
*** armax has joined #openstack-infra | 03:12 | |
ianw | Qianbiao: no, you'd have to actually specify the workflow label ... but 3rd party ci's will not be adding workflow votes | 03:13 |
ianw | 3rd party ci would be in a group that can only add +/- 1 verified votes; the final workflow votes (commonly referred to as +W) are done by human core reviewers | 03:14 |
*** apetrich has quit IRC | 03:15 | |
*** whoami-rajat has quit IRC | 03:15 | |
Qianbiao | ianw could u help me checking https://review.openstack.org/#/c/630893/ | 03:16 |
Qianbiao | Is huawei-ironic-ci vote right now? | 03:16 |
*** hongbin_ has joined #openstack-infra | 03:19 | |
*** rh-jelabarre has quit IRC | 03:20 | |
*** hongbin has quit IRC | 03:21 | |
ianw | Qianbiao: sort of, it is correctly adding a +1 verified vote. however the comment format is slightly wrong, so it will not match the regexes etc that put it in the box with the other ci reports | 03:23 |
ianw | \* Sandbox-CI http://116.66.187.79/huawei-fc-dsvm-tempest/720/ : SUCCESS | 03:23 |
ianw | you've got a leading slash | 03:23 |
Qianbiao | i see | 03:23 |
Qianbiao | ianw what about now | 03:25 |
ianw | Qianbiao: yep, that all looks correct to me | 03:26 |
Qianbiao | ok thanks ianw | 03:26 |
Qianbiao | It's very kind of you. | 03:26 |
ianw | no worries :) | 03:27 |
*** armax has quit IRC | 03:35 | |
*** agopi has joined #openstack-infra | 03:36 | |
*** dklyle has joined #openstack-infra | 03:40 | |
*** ramishra has joined #openstack-infra | 03:41 | |
*** agopi has quit IRC | 03:43 | |
*** whoami-rajat has joined #openstack-infra | 03:45 | |
*** agopi has joined #openstack-infra | 03:46 | |
*** bobh has quit IRC | 03:51 | |
*** markvoelker has joined #openstack-infra | 03:52 | |
*** ykarel|away has joined #openstack-infra | 03:52 | |
*** dklyle has quit IRC | 04:02 | |
*** bhavikdbavishi has quit IRC | 04:03 | |
*** slaweq has joined #openstack-infra | 04:11 | |
*** bhavikdbavishi has joined #openstack-infra | 04:12 | |
*** slaweq has quit IRC | 04:16 | |
*** ykarel|away is now known as ykarel | 04:25 | |
*** markvoelker has quit IRC | 04:25 | |
*** Bhujay has joined #openstack-infra | 04:36 | |
*** wolverineav has quit IRC | 04:43 | |
*** hongbin_ has quit IRC | 04:43 | |
*** wolverineav has joined #openstack-infra | 04:46 | |
*** wolverineav has quit IRC | 04:50 | |
*** wolverineav has joined #openstack-infra | 04:54 | |
*** udesale has quit IRC | 05:01 | |
*** udesale has joined #openstack-infra | 05:04 | |
*** slaweq has joined #openstack-infra | 05:11 | |
*** slaweq has quit IRC | 05:16 | |
*** markvoelker has joined #openstack-infra | 05:22 | |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: pip-and-virtualenv : only remove system files on centos https://review.openstack.org/633141 | 05:29 |
*** spsurya has joined #openstack-infra | 05:40 | |
*** udesale has quit IRC | 05:45 | |
*** udesale has joined #openstack-infra | 05:46 | |
*** armax has joined #openstack-infra | 05:48 | |
*** rkukura has joined #openstack-infra | 05:54 | |
*** wolverineav has quit IRC | 05:54 | |
*** markvoelker has quit IRC | 05:54 | |
*** wolverineav has joined #openstack-infra | 05:55 | |
*** wolverineav has quit IRC | 05:59 | |
*** wolverineav has joined #openstack-infra | 06:01 | |
*** ykarel has quit IRC | 06:02 | |
*** wolverineav has quit IRC | 06:06 | |
*** slaweq has joined #openstack-infra | 06:11 | |
*** slaweq has quit IRC | 06:15 | |
*** yamamoto has quit IRC | 06:17 | |
*** yamamoto has joined #openstack-infra | 06:20 | |
*** kjackal has joined #openstack-infra | 06:24 | |
*** ykarel has joined #openstack-infra | 06:26 | |
*** kopecmartin|off is now known as kopecmartin|devc | 06:27 | |
*** wolverineav has joined #openstack-infra | 06:35 | |
*** quiquell|off is now known as quiquell | 06:44 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/storyboard master: Imported Translations from Zanata https://review.openstack.org/625393 | 06:51 |
*** yamamoto has quit IRC | 07:08 | |
*** ociuhandu has joined #openstack-infra | 07:20 | |
*** ociuhandu_ has quit IRC | 07:20 | |
*** yamamoto has joined #openstack-infra | 07:21 | |
*** yboaron_ has joined #openstack-infra | 07:24 | |
*** roman_g has joined #openstack-infra | 07:29 | |
*** e0ne has joined #openstack-infra | 07:38 | |
*** pgaxatte has joined #openstack-infra | 07:40 | |
*** weshay has quit IRC | 07:47 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: config: add playbooks to job.toDict() https://review.openstack.org/621343 | 07:48 |
*** bhavikdbavishi has quit IRC | 07:50 | |
*** wolverineav has quit IRC | 07:51 | |
*** markvoelker has joined #openstack-infra | 07:52 | |
*** soumplis has joined #openstack-infra | 07:52 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: config: add tenant.toDict() method and REST endpoint https://review.openstack.org/621344 | 07:55 |
*** slaweq has joined #openstack-infra | 08:00 | |
*** quiquell is now known as quiquell|brb | 08:01 | |
*** panda|off is now known as panda | 08:04 | |
*** roman_g has quit IRC | 08:05 | |
*** rpittau has joined #openstack-infra | 08:10 | |
*** ccamacho has joined #openstack-infra | 08:13 | |
*** ykarel is now known as ykarel|lunch | 08:14 | |
*** tkajinam has quit IRC | 08:15 | |
*** quiquell|brb is now known as quiquell | 08:19 | |
*** markvoelker has quit IRC | 08:25 | |
*** shardy has joined #openstack-infra | 08:25 | |
*** bhavikdbavishi has joined #openstack-infra | 08:26 | |
*** ginopc has joined #openstack-infra | 08:45 | |
*** kashyap has joined #openstack-infra | 08:48 | |
*** tosky has joined #openstack-infra | 08:49 | |
*** dklyle has joined #openstack-infra | 08:50 | |
*** shardy has quit IRC | 08:50 | |
*** jpena|off is now known as jpena | 08:51 | |
*** shardy has joined #openstack-infra | 08:51 | |
*** Qianbiao has quit IRC | 08:54 | |
*** jpich has joined #openstack-infra | 08:56 | |
*** apetrich has joined #openstack-infra | 09:01 | |
*** strigazi has quit IRC | 09:15 | |
*** strigazi has joined #openstack-infra | 09:16 | |
*** markvoelker has joined #openstack-infra | 09:22 | |
*** rtjure has quit IRC | 09:24 | |
*** rtjure has joined #openstack-infra | 09:26 | |
*** udesale has quit IRC | 09:29 | |
*** udesale has joined #openstack-infra | 09:30 | |
*** derekh has joined #openstack-infra | 09:31 | |
*** roman_g has joined #openstack-infra | 09:35 | |
*** ssbarnea|bkp2 has joined #openstack-infra | 09:38 | |
*** ssbarnea|rover has quit IRC | 09:39 | |
*** ykarel|lunch is now known as ykarel | 09:40 | |
*** rpittau has quit IRC | 09:51 | |
*** rpittau has joined #openstack-infra | 09:53 | |
*** markvoelker has quit IRC | 09:55 | |
dulek | andreaf: Only second +2 is needed on https://review.openstack.org/#/c/626885/ - this will reduce jobs flakiness a lot in Kuryr-Kubernetes gates. | 09:55 |
*** ssbarnea|bkp2 has quit IRC | 09:59 | |
*** ssbarnea|rover has joined #openstack-infra | 10:00 | |
*** ssbarnea|rover has quit IRC | 10:01 | |
*** ssbarnea has joined #openstack-infra | 10:01 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack-infra/zuul-jobs master: Allow different filenames for Dockerfiles https://review.openstack.org/632979 | 10:02 |
*** luizbag has joined #openstack-infra | 10:03 | |
*** roman_g has quit IRC | 10:03 | |
*** ssbarnea|bkp2 has joined #openstack-infra | 10:04 | |
*** ssbarnea has quit IRC | 10:07 | |
tobias-urdin | infra-root: requesting a second core review on https://review.openstack.org/#/c/632699/ then we can retest release again | 10:12 |
tobias-urdin | 19:40 < fungi> sudo zuul enqueue-ref --tenant=openstack --trigger=gerrit --pipeline=release --project=openstack/puppet-aodh --ref=refs/tags/14.2.0 --newrev=617ffad84b633618490ca1023f8a31d9694b31a9 | 10:12 |
*** ramishra has quit IRC | 10:15 | |
*** shardy has quit IRC | 10:19 | |
*** shardy has joined #openstack-infra | 10:20 | |
*** yamamoto has quit IRC | 10:23 | |
openstackgerrit | Matthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes https://review.openstack.org/631049 | 10:38 |
*** ramishra has joined #openstack-infra | 10:39 | |
*** markvoelker has joined #openstack-infra | 10:52 | |
*** shardy has quit IRC | 10:55 | |
*** jaosorior has quit IRC | 11:07 | |
*** ociuhandu has quit IRC | 11:07 | |
*** yamamoto has joined #openstack-infra | 11:09 | |
*** shardy has joined #openstack-infra | 11:10 | |
*** yamamoto has quit IRC | 11:14 | |
*** shardy has quit IRC | 11:15 | |
*** shardy has joined #openstack-infra | 11:17 | |
*** shardy has quit IRC | 11:22 | |
*** markvoelker has quit IRC | 11:25 | |
*** bhavikdbavishi has quit IRC | 11:33 | |
*** shardy has joined #openstack-infra | 11:35 | |
*** shardy has quit IRC | 11:39 | |
*** shardy has joined #openstack-infra | 11:53 | |
*** e0ne has quit IRC | 11:59 | |
*** yamamoto has joined #openstack-infra | 12:05 | |
*** kjackal has quit IRC | 12:10 | |
*** smarcet has joined #openstack-infra | 12:10 | |
*** yamamoto has quit IRC | 12:12 | |
*** wolverineav has joined #openstack-infra | 12:28 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Improve logging around ansible timeouts https://review.openstack.org/633191 | 12:31 |
*** wolverineav has quit IRC | 12:32 | |
*** smarcet has quit IRC | 12:33 | |
*** zaneb has quit IRC | 12:34 | |
*** zbitter has joined #openstack-infra | 12:34 | |
*** kjackal has joined #openstack-infra | 12:35 | |
*** smarcet has joined #openstack-infra | 12:36 | |
*** e0ne has joined #openstack-infra | 12:41 | |
*** quiquell is now known as quiquell|food | 12:41 | |
*** gcutrini has joined #openstack-infra | 12:42 | |
*** jpena is now known as jpena|lunch | 12:44 | |
*** smarcet has quit IRC | 12:45 | |
*** kashyap has left #openstack-infra | 12:50 | |
*** whoami-rajat has quit IRC | 12:55 | |
*** bhavikdbavishi has joined #openstack-infra | 13:01 | |
*** kjackal has quit IRC | 13:02 | |
*** yamamoto has joined #openstack-infra | 13:03 | |
*** gcutrini has quit IRC | 13:07 | |
stephenfin | clarkb, fungi: How does one cut a new release of git-review? I suspect one is long overdue | 13:08 |
*** rh-jelabarre has joined #openstack-infra | 13:11 | |
*** yamamoto has quit IRC | 13:11 | |
evrardjp | corvus: still in the docker publishing topic: https://review.openstack.org/#/c/632979/ :D | 13:12 |
*** mriedem has joined #openstack-infra | 13:12 | |
evrardjp | stephenfin: good question, and should we move this to being released by releases? | 13:13 |
stephenfin | evrardjp: I'm not 100% sure what that would involve but it makes sense from a high level alright, yes | 13:13 |
evrardjp | yes, I am just asking the questions here :) | 13:14 |
evrardjp | I don't know either, but it's maybe worth having a look if we are forgetting these | 13:14 |
*** ccamacho has quit IRC | 13:14 | |
*** armstrong has joined #openstack-infra | 13:17 | |
*** smarcet has joined #openstack-infra | 13:21 | |
*** smarcet has quit IRC | 13:26 | |
*** quiquell|food is now known as quiquell | 13:27 | |
*** ccamacho has joined #openstack-infra | 13:29 | |
*** rcernin has quit IRC | 13:30 | |
*** rlandy has joined #openstack-infra | 13:31 | |
*** eharney has joined #openstack-infra | 13:44 | |
*** jpena|lunch is now known as jpena | 13:49 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Make setup playbook timeout configurable https://review.openstack.org/633206 | 13:54 |
AJaeger | pabelanger: could you look at https://review.openstack.org/632699 , please? | 13:55 |
*** bhavikdbavishi has quit IRC | 13:56 | |
dmsimard | Gerrit 2.16.4 is out: https://groups.google.com/forum/#!topic/repo-discuss/JSJxPsd5fsc | 13:57 |
*** trown|outtypewww is now known as trown | 13:57 | |
*** kgiusti has joined #openstack-infra | 14:00 | |
*** weshay has joined #openstack-infra | 14:06 | |
*** ramishra has quit IRC | 14:07 | |
*** yamamoto has joined #openstack-infra | 14:08 | |
Shrews | infra-root, i plan to upgrade our nodepool builders (and possibly launchers) today, starting around 3 hours from now. | 14:11 |
mordred | dmsimard: nice. we need to get ourselves upgraded to 2.15 so that we can upgrade to 2.16 | 14:12 |
mordred | Shrews: ++ | 14:12 |
dmsimard | dat polymer UI though | 14:13 |
Shrews | infra-root, that should give us an new dib build timeout and port cleanup | 14:13 |
mordred | Shrews: ah - we re-landed the port cleanup patch? | 14:15 |
mordred | Shrews: fingers crossed! | 14:15 |
Shrews | mordred: yeah | 14:15 |
*** kgiusti has left #openstack-infra | 14:20 | |
openstackgerrit | Merged openstack-infra/project-config master: Update base-test jobs to publish artifacts too https://review.openstack.org/631045 | 14:25 |
*** kgiusti has joined #openstack-infra | 14:26 | |
*** jcoufal has joined #openstack-infra | 14:28 | |
*** Bhujay has quit IRC | 14:29 | |
*** ekultails has joined #openstack-infra | 14:43 | |
*** whoami-rajat has joined #openstack-infra | 14:46 | |
*** smarcet has joined #openstack-infra | 14:46 | |
*** dave-mccowan has joined #openstack-infra | 14:46 | |
*** e0ne has quit IRC | 14:50 | |
*** dave-mccowan has quit IRC | 14:51 | |
*** psachin has quit IRC | 14:53 | |
*** spsurya has quit IRC | 15:07 | |
*** yamamoto has quit IRC | 15:12 | |
*** dklyle has quit IRC | 15:22 | |
*** ykarel is now known as ykarel|away | 15:22 | |
tobiash | yeah, that port-cleanup should work now ;) | 15:23 |
*** ykarel|away has quit IRC | 15:29 | |
*** spsurya has joined #openstack-infra | 15:30 | |
*** yamamoto has joined #openstack-infra | 15:35 | |
*** betherly has joined #openstack-infra | 15:35 | |
*** betherly has quit IRC | 15:36 | |
*** betherly has joined #openstack-infra | 15:36 | |
*** kmarc_ is now known as kmarc | 15:42 | |
*** bnemec has quit IRC | 15:43 | |
*** ramishra has joined #openstack-infra | 15:44 | |
*** cmurphy is now known as cmorpheus | 15:51 | |
*** ykarel|away has joined #openstack-infra | 15:52 | |
*** e0ne has joined #openstack-infra | 15:55 | |
*** quiquell is now known as quiquell|off | 15:56 | |
*** udesale has quit IRC | 15:56 | |
*** pgaxatte has quit IRC | 15:57 | |
*** efried has joined #openstack-infra | 16:02 | |
*** dklyle has joined #openstack-infra | 16:03 | |
*** wolverineav has joined #openstack-infra | 16:04 | |
*** efried is now known as efried_mtg | 16:05 | |
*** ramishra has quit IRC | 16:08 | |
*** wolverineav has quit IRC | 16:08 | |
*** yamamoto has quit IRC | 16:08 | |
*** yamamoto has joined #openstack-infra | 16:13 | |
*** rkukura has quit IRC | 16:13 | |
*** bhavikdbavishi has joined #openstack-infra | 16:14 | |
*** yamamoto has quit IRC | 16:17 | |
corvus | evrardjp: nice! i made a suggestion about the var name, otherwise lgtm | 16:18 |
*** bhavikdbavishi has quit IRC | 16:18 | |
*** adriancz has quit IRC | 16:21 | |
*** kjackal has joined #openstack-infra | 16:23 | |
*** panda is now known as panda|off | 16:23 | |
clarkb | stephenfin: evrardjp there should be a release group for git-review that can push tags. We've avoided using the openatack release tooling because we often are a chicken and egg for them with our tools | 16:32 |
clarkb | if infra breaks releases we need to get fix released independently | 16:32 |
*** wolverineav has joined #openstack-infra | 16:34 | |
stephenfin | clarkb: Gotcha. In that case, how does someone outside this group request a release? Just ask nicely? | 16:36 |
clarkb | yup, and suggestion for version based on included changes is helpful. | 16:37 |
Shrews | clarkb: corvus: seems the new nodepool code has propogated to the builders, but i still see a version of openstacksdk in 'pbr freeze' that is less than the new requirements.txt version. | 16:37 |
clarkb | Shrews: is that for python3? | 16:38 |
stephenfin | clarkb: I'd suggest 2.0, given commit 0376883 removes a feature (the auto branch naming based on 'Closes-Bug' commit message footers) | 16:38 |
clarkb | I do python3 /usr/local/bin/pbr freeze to be extra sure | 16:38 |
Shrews | clarkb: oh, that's probably it. lemme check | 16:38 |
corvus | or pip3 freeze might be good enough | 16:39 |
*** e0ne has quit IRC | 16:39 | |
Shrews | yup, both of those report the correct thing | 16:39 |
Shrews | can't wait until py2 is no longer a thing | 16:40 |
Shrews | or we rewrite in c++ :) | 16:40 |
clarkb | stephenfin: hrm I thought we were going to avoid big changes like that but will have to refresh memory on that one. iirc mordred was in favor | 16:41 |
Shrews | infra-root, ok, gonna restart the builder on nb03 first since it seems idle | 16:41 |
stephenfin | clarkb: Indeed. I think the main argument was this was functionality that went above and beyond what git-review was supposed to do (be a simple tool to work with Gerrit) and shouldn't have ever existed | 16:42 |
stephenfin | It also only supported LP. The alternative was extending it further to support StoryBoard | 16:42 |
clarkb | stephenfin: and we'll probably cut releasemonday? I expect today (friday) is bad day for a release | 16:43 |
clarkb | stephenfin: rgr | 16:43 |
stephenfin | Aye, good call | 16:43 |
corvus | yeah, let's observe the friday moritorium on git-review releases | 16:43 |
corvus | clarkb, stephenfin: given clarkb's surprise -- do we want to maybe circulate a changelog to the infra list and ask folks if we're okay with a release, or if anyone wants to panic about things like closes-bug? | 16:44 |
stephenfin | corvus: Sure, I can do that | 16:44 |
corvus | (the corollary to most of us not expecting big changes is that if any happen, most of us won't have been looking :) | 16:45 |
clarkb | corvus: probably a good idea. I recall the change to generally be well suplorted in review, but viewing it in context of release versioning may change opinions | 16:45 |
*** bhavikdbavishi has joined #openstack-infra | 16:46 | |
corvus | personally, i'm okay with it and buy the argument it's difficult to support well and is therefore a reasonable "big change" to make. | 16:46 |
*** jcoufal has quit IRC | 16:48 | |
Shrews | clarkb: any issue if i manually kick off a build of debian-stretch-arm64 ? | 16:49 |
Shrews | just want to watch nb03 do something | 16:49 |
*** redrobot has quit IRC | 16:51 | |
clarkb | shouldnt be | 16:51 |
*** betherly has quit IRC | 16:53 | |
*** kjackal has quit IRC | 16:54 | |
Shrews | hrm, something is wrong. fun | 16:54 |
*** rpittau has quit IRC | 16:54 | |
stephenfin | done | 16:55 |
Shrews | and all i get in the build log is "Exit code: 1" | 16:55 |
*** ginopc has quit IRC | 16:55 | |
*** jcoufal has joined #openstack-infra | 16:56 | |
*** kjackal has joined #openstack-infra | 16:59 | |
*** dklyle has quit IRC | 17:00 | |
*** diablo_rojo has joined #openstack-infra | 17:01 | |
Shrews | diskimage_builder.element_dependencies.MissingElementException: Element 'openstack-repos' not found | 17:03 |
Shrews | ?? | 17:03 |
*** derekh has quit IRC | 17:05 | |
pabelanger | Shrews: that will be in project-config, path should be added for nodepool-builder in /etc/defaults/nodepool IIRC | 17:05 |
openstackgerrit | Merged openstack-infra/zuul master: Explicitly callout ZooKeeper as ext dependency https://review.openstack.org/632732 | 17:06 |
pabelanger | Shrews: sorry, in /etc/nodepool/elements | 17:07 |
Shrews | pabelanger: thx | 17:07 |
*** dklyle has joined #openstack-infra | 17:07 | |
pabelanger | http://git.openstack.org/cgit/openstack-infra/puppet-nodepool/tree/manifests/init.pp#n163 is puppet code | 17:07 |
*** mriedem is now known as mriedem_afk | 17:10 | |
openstackgerrit | Jan Kundrát proposed openstack-infra/zuul master: Incoming connections over 19885/TCP are needed on nodes https://review.openstack.org/633242 | 17:11 |
*** gyee has joined #openstack-infra | 17:11 | |
Shrews | hrm, for some reason, disk-image-create isn't even writing out a log file | 17:12 |
*** mrhillsman is now known as mrhillsman_lunch | 17:13 | |
pabelanger | path issue? | 17:14 |
Shrews | pabelanger: i don't think so? | 17:18 |
andreaf | dulek: uhm, I have very little context to approve that, but if the kuryr gate shows mem utilisation is ok I guess it should be fine | 17:19 |
*** dklyle has quit IRC | 17:19 | |
andreaf | dulek: since frickler had a comment on it, it would be good to get his ok | 17:19 |
Shrews | pabelanger: running the command by hand and it works | 17:20 |
*** kjackal has quit IRC | 17:20 | |
*** kjackal has joined #openstack-infra | 17:23 | |
pabelanger | Shrews: can you see the diskimage-create being called in popen? does stdout=subprocess.PIPE some how imply a shell? | 17:25 |
pabelanger | I'm not to familiar with subprocess sadly | 17:25 |
openstackgerrit | Merged openstack-infra/zuul master: Make setup playbook timeout configurable https://review.openstack.org/633206 | 17:26 |
openstackgerrit | Merged openstack-infra/zuul master: Improve logging around ansible timeouts https://review.openstack.org/633191 | 17:26 |
*** jcoufal has quit IRC | 17:26 | |
Shrews | pabelanger: maybe that has something to do with it. i can't verify that the process was even started. will look into that | 17:30 |
Shrews | we probably should revert that change while i look into this | 17:32 |
corvus | Shrews: i'll be interested to see what we missed in testing :) | 17:35 |
openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: Revert "Add a timeout for the image build" https://review.openstack.org/633252 | 17:36 |
*** jcoufal has joined #openstack-infra | 17:36 | |
Shrews | corvus: yeah | 17:37 |
Shrews | it's not a *real* feature unless it's been reverted at least once | 17:37 |
*** diablo_rojo has quit IRC | 17:38 | |
Shrews | let's get that revert in asap so i can restart nb03 | 17:39 |
Shrews | infra-root ^^^ | 17:39 |
corvus | +3 | 17:39 |
Shrews | danke | 17:39 |
Shrews | given this issue, i don't want to restart launchers today | 17:41 |
*** jpena is now known as jpena|off | 17:44 | |
* Shrews takes the merge wait time opportunity to find lunch | 17:45 | |
corvus | Shrews will not go to space today | 17:46 |
openstackgerrit | Merged openstack-infra/zuul master: executor: properly format error exception https://review.openstack.org/630928 | 17:49 |
*** jpich has quit IRC | 17:53 | |
*** Dobroslaw has quit IRC | 17:56 | |
*** bhavikdbavishi has quit IRC | 17:57 | |
*** dklyle has joined #openstack-infra | 18:00 | |
*** ykarel|away has quit IRC | 18:02 | |
*** ccamacho has quit IRC | 18:05 | |
*** shardy has quit IRC | 18:06 | |
*** wolverineav has quit IRC | 18:13 | |
*** mrhillsman_lunch is now known as mrhillsman | 18:15 | |
openstackgerrit | Merged openstack-infra/zuul master: Fix noop job toDict() https://review.openstack.org/630409 | 18:17 |
*** trown is now known as trown|lunch | 18:30 | |
*** wolverineav has joined #openstack-infra | 18:33 | |
*** whoami-rajat has quit IRC | 18:34 | |
*** wolverineav has quit IRC | 18:35 | |
*** wolverineav has joined #openstack-infra | 18:35 | |
*** ijw has joined #openstack-infra | 18:40 | |
*** ijw has joined #openstack-infra | 18:41 | |
*** ijw has quit IRC | 18:41 | |
*** mriedem_afk is now known as mriedem | 18:41 | |
*** ijw has joined #openstack-infra | 18:42 | |
*** ijw has quit IRC | 18:46 | |
*** ijw has joined #openstack-infra | 18:56 | |
*** yamamoto has joined #openstack-infra | 18:57 | |
*** wolverineav has quit IRC | 19:00 | |
*** yamamoto has quit IRC | 19:02 | |
*** wolverineav has joined #openstack-infra | 19:02 | |
*** bnemec has joined #openstack-infra | 19:02 | |
openstackgerrit | Paul Vinciguerra proposed openstack-infra/zuul master: configloader.py: Not all jobs have updated attribute. https://review.openstack.org/633259 | 19:05 |
*** luizbag has quit IRC | 19:08 | |
*** bnemec is now known as beekneemech | 19:11 | |
*** dtroyer has quit IRC | 19:12 | |
*** dtroyer has joined #openstack-infra | 19:13 | |
*** e0ne has joined #openstack-infra | 19:25 | |
*** wolverineav has quit IRC | 19:33 | |
*** wolverineav has joined #openstack-infra | 19:34 | |
*** rossella_s has quit IRC | 19:36 | |
*** electrofelix has quit IRC | 19:37 | |
*** wolverineav has quit IRC | 19:38 | |
*** wolverineav has joined #openstack-infra | 19:39 | |
*** rossella_s has joined #openstack-infra | 19:41 | |
*** beekneemech has quit IRC | 19:42 | |
*** smarcet has quit IRC | 19:43 | |
*** smarcet has joined #openstack-infra | 19:43 | |
*** betherly has joined #openstack-infra | 19:45 | |
*** smarcet has quit IRC | 19:45 | |
*** wolverineav has quit IRC | 19:45 | |
*** smarcet has joined #openstack-infra | 19:46 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: support foreign required-projects https://review.openstack.org/613143 | 19:47 |
*** armstrong has quit IRC | 19:47 | |
*** bobh has joined #openstack-infra | 19:48 | |
openstackgerrit | Paul Vinciguerra proposed openstack-infra/zuul master: configloader.py: Not all jobs have an .updated attribute. https://review.openstack.org/633259 | 19:49 |
*** dklyle has quit IRC | 19:51 | |
*** raissa has joined #openstack-infra | 19:54 | |
*** bobh has quit IRC | 19:56 | |
*** raissa has quit IRC | 19:57 | |
*** smarcet has quit IRC | 20:01 | |
*** rkukura has joined #openstack-infra | 20:01 | |
*** trown|lunch is now known as trown | 20:01 | |
*** smarcet has joined #openstack-infra | 20:02 | |
*** smarcet has quit IRC | 20:03 | |
*** jcoufal has quit IRC | 20:05 | |
*** dklyle has joined #openstack-infra | 20:13 | |
openstackgerrit | Tom Barron proposed openstack-infra/project-config master: migrate manila-image-elements to in-tree job https://review.openstack.org/632412 | 20:15 |
*** smarcet has joined #openstack-infra | 20:15 | |
openstackgerrit | Tom Barron proposed openstack-infra/openstack-zuul-jobs master: Remove legacy manila artifacts job https://review.openstack.org/632362 | 20:16 |
*** e0ne has quit IRC | 20:18 | |
*** wolverineav has joined #openstack-infra | 20:18 | |
*** bobh has joined #openstack-infra | 20:18 | |
*** wolverineav has quit IRC | 20:23 | |
*** smarcet has quit IRC | 20:25 | |
*** zul has quit IRC | 20:26 | |
*** smarcet has joined #openstack-infra | 20:27 | |
*** bdodd has quit IRC | 20:29 | |
*** bdodd has joined #openstack-infra | 20:30 | |
*** kei-ichi has quit IRC | 20:31 | |
*** efried_mtg has quit IRC | 20:31 | |
*** smarcet has quit IRC | 20:33 | |
*** spsurya has quit IRC | 20:37 | |
johnsom | Gerrit seems a bit slow today, both UI and git reviews. Not sure if I am alone in that or not. | 20:39 |
*** smarcet has joined #openstack-infra | 20:46 | |
*** slaweq has quit IRC | 20:49 | |
corvus | Shrews: tell me about this error | 20:50 |
*** wolverineav has joined #openstack-infra | 20:50 | |
corvus | 2019-01-25 19:36:19,031 INFO nodepool.builder.BuildWorker.0: Logging to /var/log/nodepool/builds/debian-stretch-arm64-0000000107.log | 20:51 |
corvus | 2019-01-25 19:36:20,441 INFO nodepool.builder.BuildWorker.0: DIB failed creating debian-stretch-arm64 (1) (timeout=False) | 20:51 |
corvus | Shrews: is that the error in question? | 20:51 |
corvus | (cause that seems to have run dib -- there's a legit error in that log file) | 20:51 |
clarkb | johnsom: a quick glance at our java monitoring tools does t show any immediate red flags | 20:52 |
clarkb | johnsom: any specifics on what is slow? | 20:52 |
*** wolverineav has quit IRC | 20:55 | |
*** dklyle has quit IRC | 20:55 | |
johnsom | Just moving patch to patch via the web site, and then doing a git review seems to take longer than normal to finish | 20:55 |
johnsom | Maybe it's just some routing issue in my corner of the world. | 20:55 |
Shrews | corvus: that was the run i ran by hand | 20:55 |
Shrews | corvus: i just noticed a version of dib in /usr/local/lib/python2.7 and one in /usr/local/lib/python3.5. the one in /usr/local/bin seems to be python2 based | 20:56 |
Shrews | wondering if that has something to do with this | 20:57 |
Shrews | the py3 version is older, actually (2.16.0 vs 2.19.0 in py2) | 20:59 |
Shrews | how odd | 20:59 |
*** dklyle has joined #openstack-infra | 20:59 | |
Shrews | yet the py3 version does not appear in PATH | 20:59 |
*** dklyle has quit IRC | 21:13 | |
*** wolverineav has joined #openstack-infra | 21:14 | |
*** slaweq has joined #openstack-infra | 21:16 | |
*** slaweq has quit IRC | 21:21 | |
*** wolverineav has quit IRC | 21:29 | |
*** wolverineav has joined #openstack-infra | 21:30 | |
*** wolverineav has quit IRC | 21:30 | |
*** rlandy has quit IRC | 21:37 | |
Shrews | corvus: i'm coming up on EOD. recommendations on how to handle nb03? land the revert and restart nodepool-builder? | 21:38 |
*** trown is now known as trown|outtypewww | 21:40 | |
*** ekultails has quit IRC | 21:41 | |
*** wolverineav has joined #openstack-infra | 21:52 | |
*** eernst has joined #openstack-infra | 22:01 | |
*** ijw_ has joined #openstack-infra | 22:02 | |
corvus | Shrews: /win 25 | 22:04 |
corvus | gah | 22:04 |
corvus | Shrews: so what does the actual failure look like? | 22:05 |
*** ijw has quit IRC | 22:06 | |
corvus | oh i see | 22:06 |
Shrews | corvus: surprisingly lacking. what you pasted earlier is all there is | 22:06 |
corvus | i started the builder and see that it's now attempting and failing to build quickly | 22:06 |
Shrews | yep | 22:06 |
Shrews | the lack of info here is VERY frustrating | 22:06 |
corvus | Shrews: oh, maybe we shouldn't remove the stdout pipe | 22:09 |
Shrews | corvus: i thought of that, but it did work locally for me | 22:10 |
Shrews | corvus: the stdout pipe was the whole reason we needed to use the readline() call though, which blocked | 22:11 |
corvus | Shrews: we could use select/read instead | 22:11 |
corvus | Shrews: if i run "nodepool-builder -d" it appears to work | 22:11 |
Shrews | yes, that's how i ran it too | 22:12 |
corvus | which makes me more inclined to think that this is dib refusing to run if there's no stdout | 22:12 |
Shrews | hmm | 22:12 |
corvus | that would explain why testing didn't catch it (we don't really run dib) | 22:12 |
corvus | though... we do really run dib in the devstack job, rightL | 22:12 |
corvus | ? | 22:12 |
corvus | erm... is there still a devstack job? | 22:14 |
corvus | oh there it is http://logs.openstack.org/52/633252/1/check/nodepool-functional-py35-src/8bc0569/ | 22:14 |
Shrews | yah, http://git.openstack.org/cgit/openstack-infra/nodepool/tree/devstack/plugin.sh#n464 is the config | 22:15 |
corvus | oh! | 22:16 |
corvus | we don't run it daemonized | 22:16 |
corvus | so it's the same as what i just did -- nodepool-builder -d | 22:16 |
corvus | okay, i can't falsify the hypothesis that dib requires something hooked up to stdout, so we need to add that back into the call | 22:17 |
Shrews | alrighty. that's a better theory than anything i've had | 22:17 |
corvus | Shrews: if we're certain there won't be any output, we can wait until the process terminates and then do a read(). but, the safest thing would be a select/read(1) loop. | 22:17 |
Shrews | corvus: ack. let's revert, restart, and i'll re-look monday | 22:18 |
corvus | Shrews: sounds good. i can take care of ensuring we restart with the revert if you want to get on with your EOD :) | 22:18 |
corvus | (i left a comment on the revert and +3d) | 22:19 |
Shrews | corvus: that would be much appreciated. | 22:19 |
*** ijw_ has quit IRC | 22:19 | |
*** bobh_ has joined #openstack-infra | 22:25 | |
*** bobh has quit IRC | 22:29 | |
openstackgerrit | Merged openstack-infra/nodepool master: Revert "Add a timeout for the image build" https://review.openstack.org/633252 | 22:37 |
*** slaweq has joined #openstack-infra | 22:38 | |
*** slaweq has quit IRC | 22:43 | |
*** bobh_ has quit IRC | 22:54 | |
*** smarcet has quit IRC | 22:55 | |
*** slaweq has joined #openstack-infra | 22:57 | |
*** kgiusti has left #openstack-infra | 23:04 | |
*** slaweq has quit IRC | 23:05 | |
*** rh-jelabarre has quit IRC | 23:09 | |
pabelanger | dmsimard: ianw: mind a late friday / early saturday review: https://review.openstack.org/633095/ would like to see about working on this over weekend | 23:11 |
*** bnemec has joined #openstack-infra | 23:12 | |
*** dklyle has joined #openstack-infra | 23:13 | |
*** smarcet has joined #openstack-infra | 23:15 | |
*** betherly has quit IRC | 23:16 | |
*** betherly has joined #openstack-infra | 23:16 | |
*** dklyle has quit IRC | 23:20 | |
*** kopecmartin|devc is now known as kopecmartin|off | 23:20 | |
*** wolverineav has quit IRC | 23:22 | |
*** wolverineav has joined #openstack-infra | 23:23 | |
*** efried has joined #openstack-infra | 23:23 | |
*** wolverineav has quit IRC | 23:27 | |
*** wolverineav has joined #openstack-infra | 23:27 | |
*** xek has quit IRC | 23:30 | |
*** betherly has quit IRC | 23:32 | |
*** eernst has quit IRC | 23:38 | |
*** smarcet has quit IRC | 23:39 | |
*** bnemec has quit IRC | 23:43 | |
*** bnemec has joined #openstack-infra | 23:44 | |
corvus | nb03 restarted with revert in place | 23:52 |
*** bnemec has quit IRC | 23:59 | |
*** bnemec has joined #openstack-infra | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!