*** yamamoto has joined #openstack-infra | 00:02 | |
*** thorst has joined #openstack-infra | 00:07 | |
*** catintheroof has joined #openstack-infra | 00:09 | |
*** bobh has joined #openstack-infra | 00:09 | |
*** thorst has quit IRC | 00:12 | |
*** yamamoto has quit IRC | 00:21 | |
*** baoli has joined #openstack-infra | 00:23 | |
*** rcernin has quit IRC | 00:25 | |
*** lindycoder has joined #openstack-infra | 00:25 | |
*** lindycoder has quit IRC | 00:25 | |
*** nicolasbock has quit IRC | 00:29 | |
*** armax has joined #openstack-infra | 00:29 | |
*** rcernin has joined #openstack-infra | 00:33 | |
*** catintheroof has quit IRC | 00:42 | |
*** salv-orlando has joined #openstack-infra | 00:45 | |
*** salv-orlando has quit IRC | 00:50 | |
*** bobh has quit IRC | 00:58 | |
*** xarses_ has quit IRC | 01:01 | |
*** armax has quit IRC | 01:09 | |
*** xarses_ has joined #openstack-infra | 01:09 | |
*** thorst has joined #openstack-infra | 01:16 | |
openstackgerrit | James E. Blair proposed openstack-infra/openstack-zuul-jobs master: Zuul: add file extension to playbook path https://review.openstack.org/516035 | 01:18 |
---|---|---|
openstackgerrit | James E. Blair proposed openstack-infra/project-config master: Zuul: add file extension to playbook path https://review.openstack.org/516036 | 01:18 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Zuul: add file extension to playbook path https://review.openstack.org/516037 | 01:18 |
*** thorst has quit IRC | 01:21 | |
*** yamamoto has joined #openstack-infra | 01:21 | |
jeblair | i wrote a script to change all of the playbook paths to include extensions. there are 78 changes to do that. i'll be uploading them 20 minutes apart for the next 24 hours or so, to try to keep the memory use under control | 01:22 |
*** edmondsw has joined #openstack-infra | 01:22 | |
jeblair | https://review.openstack.org/#/q/status:open+topic:zuulv3-paths | 01:22 |
jeblair | it's worth noting that the changes to project-config and openstack-zuul-jobs are currently failing the syntax check because some of the jobs had invalid playbook paths. should be easy to fix those up. | 01:22 |
jeblair | (my script replaced the path with 'null' if it could not find the playbook) | 01:24 |
jeblair | it also occasionally made some minor changes to quotes and booleans, etc. i don't really care about those. it's just what ruamel output. | 01:25 |
*** edmondsw has quit IRC | 01:26 | |
*** yamamoto has quit IRC | 01:27 | |
*** andreww has quit IRC | 01:34 | |
*** rcernin has quit IRC | 01:37 | |
*** yamamoto has joined #openstack-infra | 01:49 | |
*** yamamoto has quit IRC | 02:04 | |
*** psachin has joined #openstack-infra | 02:10 | |
*** andreww has joined #openstack-infra | 02:14 | |
*** rcernin has joined #openstack-infra | 02:16 | |
*** yamamoto has joined #openstack-infra | 02:17 | |
*** andreww has quit IRC | 02:19 | |
*** xarses has joined #openstack-infra | 02:20 | |
*** yamamoto_ has joined #openstack-infra | 02:25 | |
*** yamamoto has quit IRC | 02:28 | |
*** salv-orlando has joined #openstack-infra | 02:46 | |
*** salv-orlando has quit IRC | 02:50 | |
*** yamamoto_ has quit IRC | 02:55 | |
*** thorst has joined #openstack-infra | 03:03 | |
*** thorst has quit IRC | 03:07 | |
*** edmondsw has joined #openstack-infra | 03:10 | |
*** HTTP_____GK1wmSU has joined #openstack-infra | 03:12 | |
*** HTTP_____GK1wmSU has left #openstack-infra | 03:13 | |
*** edmondsw has quit IRC | 03:14 | |
*** yamamoto has joined #openstack-infra | 03:20 | |
SamYaple | jeblair: whats up with the deprecating the file extention? | 03:42 |
SamYaple | just cause confusion or technical problems? | 03:42 |
*** psachin has quit IRC | 03:56 | |
*** yamamoto has quit IRC | 04:01 | |
*** yamamoto has joined #openstack-infra | 04:06 | |
*** yamamoto has quit IRC | 04:06 | |
*** yamamoto has joined #openstack-infra | 04:06 | |
*** ykarel|away has joined #openstack-infra | 04:07 | |
*** yamamoto has quit IRC | 04:15 | |
*** baoli has quit IRC | 04:29 | |
*** yamamoto has joined #openstack-infra | 04:29 | |
*** slaweq has joined #openstack-infra | 04:44 | |
*** yamamoto has quit IRC | 04:46 | |
*** salv-orlando has joined #openstack-infra | 04:47 | |
*** slaweq has quit IRC | 04:49 | |
*** salv-orlando has quit IRC | 04:51 | |
*** xarses has quit IRC | 04:52 | |
*** edmondsw has joined #openstack-infra | 04:58 | |
*** ykarel|away has quit IRC | 04:58 | |
*** edmondsw has quit IRC | 05:02 | |
*** thorst has joined #openstack-infra | 05:04 | |
*** rcernin has quit IRC | 05:07 | |
*** thorst has quit IRC | 05:08 | |
*** jokke_ has quit IRC | 05:16 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove unused jobs without playbooks https://review.openstack.org/516052 | 05:18 |
AJaeger | SamYaple: Makes life easier - we had people add the yaml to the file and than it broke... | 05:18 |
AJaeger | jeblair: 516052 removes the broken jobs, they are unused. I'll rebase 516035 on top of it | 05:19 |
SamYaple | AJaeger: thought so. yea it broke me at first too | 05:20 |
AJaeger | SamYaple: there might be more to it, let's see what jeblair adds | 05:22 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Zuul: add file extension to playbook path https://review.openstack.org/516035 | 05:22 |
AJaeger | jeblair: I'm fixing project-config the same way... | 05:27 |
*** ramishra has joined #openstack-infra | 05:27 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Zuul: add file extension to playbook path https://review.openstack.org/516036 | 05:28 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fix release-openstack-javascript https://review.openstack.org/516053 | 05:28 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove unused jobs without playbooks https://review.openstack.org/516052 | 05:33 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Zuul: add file extension to playbook path https://review.openstack.org/516035 | 05:33 |
*** sree has joined #openstack-infra | 05:34 | |
*** sree has quit IRC | 05:39 | |
*** armaan has quit IRC | 05:40 | |
AJaeger | config-core, please review and approve the changes above ^ | 05:40 |
*** armaan has joined #openstack-infra | 05:40 | |
*** ykarel|away has joined #openstack-infra | 05:45 | |
*** yamamoto has joined #openstack-infra | 05:47 | |
*** armaan has quit IRC | 05:53 | |
*** armaan has joined #openstack-infra | 05:53 | |
*** ykarel|away has quit IRC | 05:54 | |
*** jascott1 has quit IRC | 05:56 | |
*** yamamoto has quit IRC | 06:01 | |
*** ramishra has quit IRC | 06:04 | |
*** jascott1 has joined #openstack-infra | 06:07 | |
*** wolverineav has joined #openstack-infra | 06:10 | |
*** jascott1 has quit IRC | 06:12 | |
*** nikhil has quit IRC | 06:26 | |
*** Mutter_ has joined #openstack-infra | 06:38 | |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Zuul: add file extension to playbook path https://review.openstack.org/516037 | 06:39 |
*** ykarel|away has joined #openstack-infra | 06:42 | |
*** jascott1 has joined #openstack-infra | 06:45 | |
*** salv-orlando has joined #openstack-infra | 06:47 | |
*** Mutter_ has quit IRC | 06:51 | |
*** salv-orlando has quit IRC | 06:52 | |
*** vsaienk0 has joined #openstack-infra | 07:04 | |
*** thorst has joined #openstack-infra | 07:05 | |
dirk | infra-root: please remove jenkins-1 here: https://review.openstack.org/#/c/506848/ | 07:08 |
*** thorst has quit IRC | 07:09 | |
*** jascott1 has quit IRC | 07:11 | |
*** jascott1 has joined #openstack-infra | 07:12 | |
*** vsaienk0 has quit IRC | 07:16 | |
*** snapiri has quit IRC | 07:17 | |
AJaeger | dirk: it does not harm, just ignore it | 07:20 |
AJaeger | dirk: a rebase should remove it | 07:21 |
*** baoli has joined #openstack-infra | 07:28 | |
*** baoli has quit IRC | 07:33 | |
*** ykarel|away has quit IRC | 07:33 | |
*** snapiri has joined #openstack-infra | 07:46 | |
*** spiffxp has quit IRC | 07:47 | |
*** vipuls has quit IRC | 07:47 | |
*** salv-orlando has joined #openstack-infra | 07:48 | |
*** vipul has joined #openstack-infra | 07:49 | |
*** wolverineav has quit IRC | 07:50 | |
*** spiffxp has joined #openstack-infra | 07:50 | |
*** salv-orlando has quit IRC | 07:53 | |
*** HTTP_____GK1wmSU has joined #openstack-infra | 08:01 | |
*** HTTP_____GK1wmSU has left #openstack-infra | 08:02 | |
dirk | AJaeger: well, it seems that it causes people to ignore the review because they filter for jenkins-1 I guess | 08:17 |
*** wolverineav has joined #openstack-infra | 08:17 | |
dirk | AJaeger: rebase helped, thanks for the hint! | 08:19 |
*** wolverineav has quit IRC | 08:21 | |
*** salv-orlando has joined #openstack-infra | 08:23 | |
*** d0ugal_ has quit IRC | 08:28 | |
*** spiffxp has quit IRC | 08:31 | |
*** vipul has quit IRC | 08:32 | |
*** vipul has joined #openstack-infra | 08:34 | |
*** edmondsw has joined #openstack-infra | 08:34 | |
*** spiffxp has joined #openstack-infra | 08:36 | |
*** d0ugal_ has joined #openstack-infra | 08:38 | |
*** edmondsw has quit IRC | 08:39 | |
*** gmann_afk is now known as gmann | 08:39 | |
*** wolverineav has joined #openstack-infra | 08:59 | |
*** jascott1 has quit IRC | 09:01 | |
*** jascott1 has joined #openstack-infra | 09:02 | |
*** wolverineav has quit IRC | 09:03 | |
*** e0ne has joined #openstack-infra | 09:05 | |
*** thorst has joined #openstack-infra | 09:06 | |
*** salv-orlando has quit IRC | 09:07 | |
*** slaweq has joined #openstack-infra | 09:09 | |
*** salv-orlando has joined #openstack-infra | 09:09 | |
*** baoli has joined #openstack-infra | 09:10 | |
*** salv-orlando has quit IRC | 09:11 | |
*** ykarel|away has joined #openstack-infra | 09:11 | |
*** baoli has quit IRC | 09:11 | |
*** thorst has quit IRC | 09:12 | |
*** salv-orlando has joined #openstack-infra | 09:12 | |
*** gridinv has quit IRC | 09:17 | |
*** salv-orlando has quit IRC | 09:17 | |
*** gridinv has joined #openstack-infra | 09:18 | |
*** salv-orlando has joined #openstack-infra | 09:25 | |
*** baoli has joined #openstack-infra | 09:26 | |
*** d0ugal_ has quit IRC | 09:30 | |
*** baoli has quit IRC | 09:31 | |
*** d0ugal has joined #openstack-infra | 09:31 | |
*** d0ugal has quit IRC | 09:31 | |
*** d0ugal has joined #openstack-infra | 09:31 | |
*** e0ne has quit IRC | 09:47 | |
AJaeger | mnaser: what's up with all your puppet changes that they get RETRY_LIMIT? Example: https://review.openstack.org/515899 I see mirror-workspace-git-repos failing but don't see how... | 09:48 |
AJaeger | I wonder whether https://review.openstack.org/#/c/513233/ is the culprit here | 09:50 |
AJaeger | ianw, jeblair, mnaser ^ | 09:50 |
AJaeger | mmh, looking further: It wants to check out puppet-vitrage with stable/ocata branch - which does not exist. | 09:57 |
*** wolverineav has joined #openstack-infra | 10:05 | |
AJaeger | puppet-vitrage is in the required-projects list. Should we fallback to master here? | 10:05 |
*** bhavik1 has joined #openstack-infra | 10:07 | |
AJaeger | with here I mean: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/mirror-workspace-git-repos/tasks/main.yaml#n40 | 10:09 |
* AJaeger is not fixing that, just pointing out - and haven't added to etherpad yet | 10:10 | |
*** wolverineav has quit IRC | 10:10 | |
*** bhavik1 has quit IRC | 10:13 | |
*** salv-orlando has quit IRC | 10:17 | |
*** salv-orlando has joined #openstack-infra | 10:18 | |
*** andreww has joined #openstack-infra | 10:19 | |
*** andreww has quit IRC | 10:20 | |
*** andreww has joined #openstack-infra | 10:20 | |
*** slaweq has quit IRC | 10:22 | |
*** xarses_ has quit IRC | 10:22 | |
*** edmondsw has joined #openstack-infra | 10:22 | |
*** salv-orlando has quit IRC | 10:23 | |
*** edmondsw has quit IRC | 10:27 | |
dirk | AJaeger: whats the link to the zuulv3 breakages etherpad? | 10:30 |
dirk | https://review.openstack.org/#/c/514833/ introduced a massive regression breaking almost everything and we didn't catch it because the job didn't actually test the zuul change | 10:31 |
*** wolverineav has joined #openstack-infra | 10:31 | |
*** wolverineav has quit IRC | 10:36 | |
*** wolverineav has joined #openstack-infra | 10:42 | |
*** wolverineav has quit IRC | 10:46 | |
*** pbourke has quit IRC | 10:49 | |
*** e0ne has joined #openstack-infra | 10:50 | |
*** pbourke has joined #openstack-infra | 10:50 | |
*** e0ne has quit IRC | 10:51 | |
*** slaweq has joined #openstack-infra | 10:52 | |
*** thorst has joined #openstack-infra | 11:08 | |
*** thorst has quit IRC | 11:12 | |
*** salv-orlando has joined #openstack-infra | 11:18 | |
*** wolverineav has joined #openstack-infra | 11:20 | |
*** salv-orlando has quit IRC | 11:22 | |
*** wolverineav has quit IRC | 11:24 | |
*** ykarel|away has quit IRC | 11:25 | |
*** slaweq has quit IRC | 11:25 | |
*** slaweq has joined #openstack-infra | 11:27 | |
*** thorst has joined #openstack-infra | 11:29 | |
*** thorst has quit IRC | 11:33 | |
*** lathiat has quit IRC | 11:44 | |
*** dave-mccowan has joined #openstack-infra | 11:46 | |
*** wolverineav has joined #openstack-infra | 11:48 | |
*** lathiat has joined #openstack-infra | 11:51 | |
*** yamamoto has joined #openstack-infra | 11:55 | |
*** thorst has joined #openstack-infra | 11:58 | |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Zuul: add file extension to playbook path https://review.openstack.org/516082 | 11:59 |
*** thorst has quit IRC | 11:59 | |
*** slaweq has quit IRC | 12:00 | |
*** slaweq has joined #openstack-infra | 12:04 | |
frickler | dirk: https://etherpad.openstack.org/p/zuulv3-issues | 12:06 |
dirk | frickler: thanks, but that is only for reviews, not for bugs | 12:10 |
*** edmondsw has joined #openstack-infra | 12:10 | |
frickler | dirk: nope, bugs are below | 12:11 |
AJaeger | it's for bugs as well, see "Issues needing triage" and "Issues with jobs" | 12:11 |
*** edmondsw has quit IRC | 12:15 | |
dirk | frickler: ah, indeed, didn't scroll far enough | 12:15 |
dirk | Looks like I can add two more bugs | 12:16 |
AJaeger | dirk: I cleaned up a bit for you ;) | 12:16 |
*** dsariel has joined #openstack-infra | 12:17 | |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool master: Zuul: add file extension to playbook path https://review.openstack.org/516083 | 12:19 |
*** watanabe_isao has joined #openstack-infra | 12:24 | |
*** watanabe_isao has quit IRC | 12:24 | |
*** HTTP_____GK1wmSU has joined #openstack-infra | 12:31 | |
*** HTTP_____GK1wmSU has left #openstack-infra | 12:32 | |
*** slaweq has quit IRC | 12:38 | |
*** rkukura has joined #openstack-infra | 12:39 | |
*** nicolasbock has joined #openstack-infra | 12:44 | |
*** slaweq has joined #openstack-infra | 12:44 | |
*** hrubi has quit IRC | 12:45 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/puppet-subunit2sql master: Make subunit2sql install depend on netifaces https://review.openstack.org/516085 | 12:45 |
*** hrubi has joined #openstack-infra | 12:45 | |
*** Mutter_ has joined #openstack-infra | 13:13 | |
*** Mutter_ has quit IRC | 13:16 | |
*** thorst has joined #openstack-infra | 13:17 | |
*** e0ne has joined #openstack-infra | 13:18 | |
*** salv-orlando has joined #openstack-infra | 13:19 | |
*** salv-orlando has quit IRC | 13:24 | |
*** e0ne_ has joined #openstack-infra | 13:29 | |
*** e0ne has quit IRC | 13:33 | |
*** rtjure has quit IRC | 13:33 | |
*** HTTP_____GK1wmSU has joined #openstack-infra | 13:38 | |
*** thorst has quit IRC | 13:49 | |
*** jtomasek has joined #openstack-infra | 14:04 | |
*** ociuhandu has joined #openstack-infra | 14:13 | |
*** slaweq has quit IRC | 14:17 | |
*** dsariel has quit IRC | 14:17 | |
*** e0ne_ has quit IRC | 14:18 | |
*** slaweq has joined #openstack-infra | 14:20 | |
*** LindaWang has quit IRC | 14:24 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove unused jobs without playbooks https://review.openstack.org/516052 | 14:26 |
openstackgerrit | Merged openstack-infra/project-config master: Remove legacy bifrost jobs https://review.openstack.org/515728 | 14:26 |
*** yamahata has quit IRC | 14:28 | |
*** HenryG has joined #openstack-infra | 14:28 | |
*** HenryG has quit IRC | 14:30 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Zuul: add file extension to playbook path https://review.openstack.org/516035 | 14:32 |
*** HenryG has joined #openstack-infra | 14:32 | |
jeblair | dirk: i don't understand. what is the error related to 514833? | 14:46 |
dirk | jeblair: sec | 14:46 |
dirk | jeblair: http://logs.openstack.org/76/516076/1/check/cross-glance-py27/b5a011f/job-output.txt.gz | 14:50 |
dirk | jeblair: search for oslo.serialization=== | 14:50 |
*** slaweq has quit IRC | 14:50 | |
*** slaweq has joined #openstack-infra | 14:50 | |
jeblair | dirk: i can't find that in the log | 14:51 |
jeblair | dirk: can you just link to the line? | 14:51 |
jeblair | dirk: the timestamps are href anchors | 14:53 |
dirk | jeblair: trying | 14:55 |
dirk | http://logs.openstack.org/76/516076/1/check/cross-glance-py27/b5a011f/job-output.txt.gz#_2017-10-29_10_33_01_022143 | 14:57 |
dirk | jeblair: basically you see the upper-constraints.txt file that is used is from master, not from the review | 14:58 |
dirk | jeblair: it should install 2.19.1 instead of 2.19.2, which would make this job pass | 14:59 |
dirk | jeblair: apparently this is a general problem, because that's how we merged the breakage - the gate job didn't test the change, but tested master | 14:59 |
*** dsariel has joined #openstack-infra | 15:00 | |
jeblair | dirk: that says oslo.serialization==2.21.2 | 15:00 |
dirk | Now I cannot figure out how to undo the change as it had pretty desastrous effect | 15:00 |
dirk | Ah, then 2.21.1 is expected | 15:00 |
jeblair | dirk: it's a change to set oslo.serialization to 2.21.2 | 15:00 |
* dirk is on a phone without real internet and | 15:00 | |
dirk | jeblair: no, it's a revert of that change: https://review.openstack.org/#/c/516076/ | 15:02 |
jeblair | ah, now i understand, thanks | 15:03 |
*** jtomasek has quit IRC | 15:04 | |
*** dsariel_ has joined #openstack-infra | 15:04 | |
dirk | jeblair: it looks like it always uses the constraints file from master, which is how we managed to merge the regression in the first place and now cannot undo it | 15:05 |
*** thorst has joined #openstack-infra | 15:07 | |
*** thorst has quit IRC | 15:08 | |
*** dsariel has quit IRC | 15:08 | |
*** dsariel__ has joined #openstack-infra | 15:09 | |
*** armax has joined #openstack-infra | 15:12 | |
*** apetrich has quit IRC | 15:12 | |
*** apetrich has joined #openstack-infra | 15:12 | |
*** gmann is now known as gmann_afk | 15:13 | |
*** dsariel_ has quit IRC | 15:13 | |
*** dsariel_ has joined #openstack-infra | 15:14 | |
*** mriedem has joined #openstack-infra | 15:16 | |
*** armax has quit IRC | 15:16 | |
*** armax has joined #openstack-infra | 15:17 | |
*** dsariel__ has quit IRC | 15:19 | |
jeblair | dirk: mordred has a pending change to tox-siblings; i wonder if there's a bug in the current version, and if mordred's fix would address it. i've set 516076 to depends-on the fix to test that. | 15:19 |
*** dsariel__ has joined #openstack-infra | 15:19 | |
*** salv-orlando has joined #openstack-infra | 15:20 | |
*** dsariel_ has quit IRC | 15:23 | |
*** salv-orlando has quit IRC | 15:24 | |
*** dsariel_ has joined #openstack-infra | 15:24 | |
jeblair | dirk: it's looking like that is the fix. i my guess is that the initial 'run tox without tests' step either used the wrong constraints file, or no constraints at all. mordred has fixed that in his change. | 15:27 |
*** dsariel__ has quit IRC | 15:28 | |
*** slaweq has quit IRC | 15:28 | |
jeblair | dirk: i see oslo.serialization==2.21.1 in the console log now | 15:28 |
*** dsariel__ has joined #openstack-infra | 15:29 | |
jeblair | unfortunately, this job failed: http://logs.openstack.org/76/516076/2/check/openstack-tox-validate/b24b4a4/job-output.txt.gz#_2017-10-29_15_20_42_446144 | 15:29 |
jeblair | "Invalid constraint for does not have 3 "=":" | 15:30 |
jeblair | i don't see a problem with the constraints file it logged | 15:30 |
*** dsariel_ has quit IRC | 15:34 | |
AJaeger | dmsimard: please review https://review.openstack.org/#/c/516053/ again - the other change is based on top of it, so it should be fine! | 15:34 |
dmsimard | AJaeger: oh, huh | 15:35 |
*** mnaser_ has joined #openstack-infra | 15:36 | |
AJaeger | jeblair, dmsimard : I'm not aware how to test https://review.openstack.org/#/c/516025/1/roles/build-wheels/tasks/main.yaml - what do you think should be done here? | 15:37 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: Handle some git warnings https://review.openstack.org/516016 | 15:37 |
*** d0ugal has quit IRC | 15:37 | |
*** mnaser has quit IRC | 15:38 | |
*** dsariel_ has joined #openstack-infra | 15:38 | |
*** mnaser_ is now known as mnaser | 15:38 | |
openstackgerrit | Merged openstack-infra/project-config master: Fix release-openstack-javascript https://review.openstack.org/516053 | 15:40 |
jeblair | AJaeger: i think you'd have to merge the change and watch the next run of a job that uses that | 15:40 |
mnaser | AJaeger: i actually have no idea why the job is failing | 15:40 |
AJaeger | jeblair: yeah ;( Ok, then wait for reviews... | 15:41 |
mnaser | we recently cut puppet-vitrage stable/pike | 15:41 |
mnaser | so i just added it there again | 15:41 |
mnaser | and then pre is failing.. without much info | 15:41 |
dmsimard | AJaeger: there is kind of a way but it's a bit awkward and I want to find a better way | 15:41 |
AJaeger | mnaser: see my full backlog, you check out stable branches that don't exist in some cases | 15:41 |
mnaser | AJaeger: sorry let me look at eavesdrop | 15:41 |
AJaeger | mnaser: but I can't help further ;( | 15:41 |
AJaeger | mnaser: the logs are indeed sparse | 15:42 |
*** dsariel__ has quit IRC | 15:42 | |
dmsimard | AJaeger: if you look at https://github.com/openstack-infra/zuul/blob/feature/zuulv3/.zuul.yaml#L1-L29, this is how we ended up being able to test things that are otherwise not testable in-flight | 15:42 |
openstackgerrit | Merged openstack-infra/project-config master: Zuul: add file extension to playbook path https://review.openstack.org/516036 | 15:42 |
dmsimard | AJaeger: the tl;dr of that job is that it is multinode, installs Zuul on a nodepool VM, and then runs a playbook on other VMs in the nodeset with the (patched) zuul callback plugin | 15:43 |
mnaser | AJaeger: going a step above, https://review.openstack.org/#/c/512128/ -- this is in stable/pike and that branch exists in the repo | 15:44 |
*** dsariel__ has joined #openstack-infra | 15:44 | |
mnaser | AJaeger: ara saves the day :> http://logs.openstack.org/28/512128/2/check/puppet-openstack-integration-4-scenario001-tempest-centos-7/4022bc6/ara/result/abba525a-cc94-48a9-85e9-2d2fa6811aae/ | 15:45 |
mnaser | ctrl+f "failed": true | 15:45 |
dmsimard | AJaeger: I'd like to have a "framework" that makes doing things like that easy -- i.e, run an ansible-playbook from a "fake" executor | 15:45 |
AJaeger | mnaser: puppet-cloudkitty -> doesn't have the branch. Yes, found via ara as well | 15:45 |
mnaser | i think puppet-monasca and puppet-cloudkitty, yeah | 15:45 |
AJaeger | dmsimard: agreed. | 15:45 |
*** d0ugal has joined #openstack-infra | 15:46 | |
AJaeger | dmsimard: so, what should we do? Merge and monitor - or do you have a way of testing? Or do you prefer to leave the code as is and silence the ansible-lint warning? | 15:46 |
*** edmondsw has joined #openstack-infra | 15:46 | |
dmsimard | AJaeger: We've had exactly that kind of change break one of our playbooks and we had to add a comment to avoid people doing that exact change again.. let me try and find it | 15:47 |
AJaeger | config-core, could I get some reviews for https://review.openstack.org/514053 and https://review.openstack.org/515582 , please? | 15:47 |
*** dsariel_ has quit IRC | 15:47 | |
AJaeger | dmsimard: I'm fine leaving code as is ;) Just want to move forward one way or the other... Thanks for digging | 15:48 |
openstackgerrit | Merged openstack-infra/project-config master: ansible-lint: Handle some git warnings https://review.openstack.org/516016 | 15:48 |
*** dsariel_ has joined #openstack-infra | 15:50 | |
*** edmondsw has quit IRC | 15:51 | |
*** dsariel__ has quit IRC | 15:53 | |
dmsimard | AJaeger: hm, I can't find the change in question.. /me shrugs | 15:53 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy bifrost jobs https://review.openstack.org/515729 | 15:54 |
AJaeger | dmsimard: So, what's your advise? Leave as is and silence the warning or change? | 15:55 |
* AJaeger has to rebase anyhow... | 15:55 | |
*** armax has quit IRC | 15:57 | |
*** dsariel_ has quit IRC | 15:58 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: use command https://review.openstack.org/516025 | 15:58 |
AJaeger | that's the rebase - if you feel I should leave as is, please -1 and I'll change ^ | 15:58 |
dmsimard | AJaeger: I really, really dislike merging untested code.. and it's bitten us quite a bit, even with proper reviews I've ended up +3'ing broken things. I guess it's part of why I've been spending time on testing as many of the base roles as possible. | 15:59 |
*** slaweq has joined #openstack-infra | 15:59 | |
dmsimard | Maybe this one change is fine, maybe it's not, it's hard to tell without testing it | 15:59 |
*** armax has joined #openstack-infra | 15:59 | |
AJaeger | dmsimard: I've been too long in reviewing project-config that was untested ;( | 16:00 |
AJaeger | let me just silence ansible-lint so that we can get the jobs voting... | 16:00 |
dmsimard | AJaeger: oh, yeah, I guess you could argue that things are better now | 16:00 |
dmsimard | AJaeger: add a skip tag and add a TODO ? | 16:00 |
AJaeger | yes | 16:00 |
dmsimard | mnaser, EmilienM: btw I added a comment in https://review.openstack.org/#/c/515972/ | 16:01 |
dmsimard | AJaeger: looks like https://review.openstack.org/#/c/514053/ is in merge conflict | 16:02 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: Silence shell warnings https://review.openstack.org/516025 | 16:04 |
AJaeger | dmsimard: done as suggested ^ | 16:04 |
AJaeger | dmsimard: argh, will rebase. Thansk | 16:04 |
AJaeger | dmsimard: and it didn't include the .yaml files, fixed now | 16:07 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use native v3 publishing for project-team-guide https://review.openstack.org/514053 | 16:07 |
*** rtjure has joined #openstack-infra | 16:08 | |
dirk | jeblair: did you see http://logs.openstack.org/76/516076/2/check/openstack-tox-validate/b24b4a4/job-output.txt.gz#_2017-10-29_15_20_41_600130 ? | 16:08 |
AJaeger | dirk: known issue - there's no git remote and pip gets confused. Just ignore it... | 16:09 |
*** armax has quit IRC | 16:13 | |
jeblair | okay, the validate job is erroring because tox_siblings writes a new constraints file with an extra newline at the end | 16:14 |
jeblair | i pushed up a new version of https://review.openstack.org/516076 which disables tox_siblings for now | 16:20 |
jeblair | dirk: ^ i expect that to work. | 16:26 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use native v3 publishing for project-team-guide https://review.openstack.org/514053 | 16:27 |
AJaeger | dmsimard: changes to true ^ | 16:28 |
*** nicolasbock has quit IRC | 16:29 | |
AJaeger | dmsimard, pabelanger: for ansible-lint, we need https://review.openstack.org/516020 and https://review.openstack.org/516025 to pass the job | 16:29 |
*** [HeOS] has quit IRC | 16:30 | |
*** markvoelker has quit IRC | 16:31 | |
jeblair | dirk: https://review.openstack.org/516076 has tox-validate and cross-glance-py35 passing now | 16:31 |
*** slaweq has quit IRC | 16:32 | |
openstackgerrit | Merged openstack-infra/nodepool master: Zuul: add file extension to playbook path https://review.openstack.org/516083 | 16:32 |
openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Zuul: add file extension to playbook path https://review.openstack.org/516082 | 16:32 |
*** dsariel_ has joined #openstack-infra | 16:35 | |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Create new repo cinder tempest plugin https://review.openstack.org/486303 | 16:36 |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Add jobs for cinder-tempest-repo https://review.openstack.org/515797 | 16:37 |
*** slaweq has joined #openstack-infra | 16:39 | |
*** dsariel__ has joined #openstack-infra | 16:40 | |
*** dsariel_ has quit IRC | 16:43 | |
*** dsariel_ has joined #openstack-infra | 16:45 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: Silence shell warnings https://review.openstack.org/516025 | 16:45 |
*** thorst has joined #openstack-infra | 16:47 | |
*** dsariel__ has quit IRC | 16:48 | |
*** dsariel__ has joined #openstack-infra | 16:50 | |
*** thorst has quit IRC | 16:52 | |
*** dsariel_ has quit IRC | 16:53 | |
*** dsariel_ has joined #openstack-infra | 16:55 | |
*** dhajare|mtg has joined #openstack-infra | 16:56 | |
*** dsariel__ has quit IRC | 16:58 | |
*** dsariel__ has joined #openstack-infra | 17:00 | |
*** dsariel_ has quit IRC | 17:03 | |
*** dsariel_ has joined #openstack-infra | 17:05 | |
*** dsariel__ has quit IRC | 17:08 | |
*** dsariel__ has joined #openstack-infra | 17:10 | |
*** felipemonteiro_ has joined #openstack-infra | 17:11 | |
*** felipemonteiro__ has joined #openstack-infra | 17:12 | |
*** slaweq has quit IRC | 17:13 | |
*** dsariel_ has quit IRC | 17:13 | |
*** alex_xu has quit IRC | 17:14 | |
*** alex_xu has joined #openstack-infra | 17:14 | |
*** dsariel_ has joined #openstack-infra | 17:15 | |
*** felipemonteiro_ has quit IRC | 17:16 | |
dmsimard | AJaeger: we merged a regression somewhere | 17:17 |
*** wolverineav has quit IRC | 17:18 | |
*** dsariel__ has quit IRC | 17:18 | |
dmsimard | hmm, maybe not, have you seen http://logs.openstack.org/97/516097/3/check/openstack-tox-cover/c58d50b/ara/result/27fd6ce1-76fe-40c8-a93a-93dfdf6db875/ before ? | 17:19 |
*** felipemonteiro__ has quit IRC | 17:19 | |
*** felipemonteiro_ has joined #openstack-infra | 17:19 | |
dmsimard | HEAD is now at 7fea153 Merge \"update constraint for oslo.serialization to new release 2.21.2\"\nerror: pathspec 'feature/1.0' did not match any file(s) known to git. | 17:19 |
*** wolverineav has joined #openstack-infra | 17:19 | |
dmsimard | it looks like it's trying to checkout the ara branch (feature/1.0) for the requirements repo | 17:19 |
*** dsariel_ has quit IRC | 17:19 | |
dmsimard | jeblair: ^ rings a bell ? is that related to the branch issues ? | 17:19 |
*** slaweq has joined #openstack-infra | 17:20 | |
dmsimard | interestingly all the other patches in that chain passed, just that one failed. I'll try a recheck just to see if it passes. | 17:20 |
*** salv-orlando has joined #openstack-infra | 17:21 | |
*** felipemonteiro_ has quit IRC | 17:23 | |
*** salv-orlando has quit IRC | 17:25 | |
*** slaweq has quit IRC | 17:27 | |
*** slaweq has joined #openstack-infra | 17:28 | |
*** Apoorva has joined #openstack-infra | 17:30 | |
*** Apoorva has quit IRC | 17:30 | |
*** markvoelker has joined #openstack-infra | 17:31 | |
*** edmondsw has joined #openstack-infra | 17:35 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Revert "Move to dictionary list of projects zuul._projects" https://review.openstack.org/516117 | 17:37 |
jeblair | dmsimard, AJaeger, ianw: ^ that may also explain the "chef checked out swift" item from the etherpad | 17:38 |
*** edmondsw has quit IRC | 17:39 | |
jeblair | dmsimard: if i'm reading ara correctly fram the build you linked, we did requirements,ara for 'determine local head', and then ara,requirements for 'update remote repository state' | 17:39 |
jeblair | so i'm guessing that with_dict and with_together don't order things the same way | 17:40 |
*** slaweq has quit IRC | 17:43 | |
*** eroux has joined #openstack-infra | 17:53 | |
AJaeger | jeblair, dmsimard: argh ;( that's sad. Do we have to revert all of these and give up on the idea? | 17:57 |
mnaser | is there a way to "pin" for required-project to a specific branch | 18:00 |
mnaser | stable/pike for puppet-openstack-integration is failing bc its trying to check out stable/pike tempest | 18:00 |
jeblair | AJaeger: no, we just need to alter it slightly to make sure we use the same order. i'm just not going to figure it out on a sunday | 18:01 |
jeblair | mnaser: link? | 18:01 |
mnaser | jeblair: http://logs.openstack.org/28/512128/3/check/puppet-openstack-integration-4-scenario001-tempest-centos-7/0fa1dd5/ara/result/594bee67-9787-4703-a3ac-1ddee2fc1ac5/ | 18:03 |
mnaser | https://review.openstack.org/#/c/512128/ | 18:03 |
mnaser | ctrl+f "failed" will show it attempting to checkout stable/pike of tempest | 18:03 |
jeblair | mnaser: same bug ^ | 18:03 |
mnaser | oh | 18:03 |
mnaser | ok cool | 18:04 |
jeblair | mnaser: fix is 516117 | 18:04 |
*** thorst has joined #openstack-infra | 18:04 | |
mnaser | jeblair: i see it, cool, i'll recheck once things merge | 18:04 |
mnaser | thank you :> | 18:05 |
jeblair | mnaser: np | 18:05 |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Revert "Move to dictionary list of projects zuul._projects" https://review.openstack.org/516117 | 18:05 |
*** markvoelker has quit IRC | 18:06 | |
AJaeger | mnaser: now ^ | 18:06 |
mnaser | AJaeger: i guess i can re-add cloudkitty and co | 18:07 |
AJaeger | jeblair: you can abandon https://review.openstack.org/516116 - pabelanger beat you to it... | 18:07 |
AJaeger | seems he was too impatient to wait for your automatic submission ;) | 18:08 |
mnaser | submitted my change again, ill report back for updates (512128 if anyone wants to watch) | 18:08 |
*** thorst has quit IRC | 18:16 | |
*** slaweq has joined #openstack-infra | 18:18 | |
mnaser | looks like it went past that stage | 18:19 |
mnaser | thanks everyone | 18:19 |
AJaeger | yeah! | 18:19 |
*** armax has joined #openstack-infra | 18:32 | |
*** thorst has joined #openstack-infra | 18:44 | |
*** thorst has quit IRC | 18:44 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fix openstack-infra publishing https://review.openstack.org/516010 | 18:45 |
*** thorst has joined #openstack-infra | 18:45 | |
*** slaweq has quit IRC | 18:48 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove project-team-guide-publish https://review.openstack.org/514054 | 18:49 |
*** thorst has quit IRC | 18:49 | |
openstackgerrit | Merged openstack-infra/project-config master: neutron-dynamic-routing: Use releasenote and publish job template https://review.openstack.org/514495 | 18:51 |
*** salv-orlando has joined #openstack-infra | 18:51 | |
*** slaweq has joined #openstack-infra | 18:52 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove migrated legacy-octavia-dashboard-* https://review.openstack.org/513568 | 18:52 |
*** armax has quit IRC | 18:52 | |
*** armax has joined #openstack-infra | 18:53 | |
*** [HeOS] has joined #openstack-infra | 18:54 | |
*** salv-orlando has quit IRC | 18:56 | |
*** markvoelker has joined #openstack-infra | 19:02 | |
*** armax has quit IRC | 19:05 | |
*** edmondsw has joined #openstack-infra | 19:23 | |
*** slaweq has quit IRC | 19:26 | |
*** edmondsw has quit IRC | 19:27 | |
*** slaweq has joined #openstack-infra | 19:33 | |
*** markvoelker has quit IRC | 19:36 | |
*** hongbin has joined #openstack-infra | 19:42 | |
openstackgerrit | James E. Blair proposed openstack/diskimage-builder master: Zuul: add file extension to playbook path https://review.openstack.org/516129 | 19:43 |
*** felipemonteiro_ has joined #openstack-infra | 19:46 | |
*** slaweq has quit IRC | 19:47 | |
*** slaweq has joined #openstack-infra | 19:47 | |
*** felipemonteiro__ has joined #openstack-infra | 19:47 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove migrated legacy-octavia-dashboard-* https://review.openstack.org/513568 | 19:51 |
*** felipemonteiro_ has quit IRC | 19:51 | |
*** felipemonteiro_ has joined #openstack-infra | 19:54 | |
*** felipemonteiro__ has quit IRC | 19:54 | |
*** dhajare|mtg has quit IRC | 19:55 | |
prometheanfire | jeblair: can you create a bug for https://review.openstack.org/#/c/514058/ and add the reqs team to it so we can track it (making a note that we'd need to un-disable tox_install_siblings) | 20:02 |
prometheanfire | other than that the reqs patch looks acceptable | 20:02 |
*** armaan has quit IRC | 20:03 | |
openstackgerrit | Merged openstack-infra/project-config master: add more trace info to process_release_requests.py https://review.openstack.org/514818 | 20:09 |
openstackgerrit | Merged openstack-infra/project-config master: Remove tempest job https://review.openstack.org/514769 | 20:12 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove non-voting from legacy-puppet-beaker-rspec https://review.openstack.org/515582 | 20:15 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Run legacy-tripleo-ci-centos-7-ovb-ha-tempest-oooq on tripleo-rh1 https://review.openstack.org/514718 | 20:15 |
*** markvoelker has joined #openstack-infra | 20:34 | |
*** thorst has joined #openstack-infra | 20:46 | |
*** eroux has quit IRC | 20:49 | |
*** thorst has quit IRC | 20:50 | |
*** salv-orlando has joined #openstack-infra | 20:52 | |
*** salv-orlando has quit IRC | 20:56 | |
*** felipemonteiro_ has quit IRC | 20:57 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Zuul: add file extension to playbook path https://review.openstack.org/516135 | 21:03 |
*** markvoelker has quit IRC | 21:07 | |
*** edmondsw has joined #openstack-infra | 21:11 | |
*** edmondsw has quit IRC | 21:15 | |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Simplify ansible invocations in tox.ini https://review.openstack.org/516022 | 21:15 |
*** askb has joined #openstack-infra | 21:24 | |
*** jascott1 has quit IRC | 21:34 | |
*** dsariel_ has joined #openstack-infra | 21:52 | |
mnaser | gah, not very lucky | 21:58 |
mnaser | http://zuulv3.openstack.org/static/stream.html?uuid=699035fc25f74eeeaf7b99039327af59&logfile=console.log | 21:58 |
mnaser | this job failed after timing out (io issues on system?) | 21:58 |
mnaser | but now its stuck in uploading logs | 21:58 |
mnaser | is there a tiemout for post? | 21:58 |
mnaser | post is stuck for 1h15m if my math is right | 21:59 |
*** markvoelker has joined #openstack-infra | 22:04 | |
ianw | jeblair: hmm, thanks, i'll take a look | 22:13 |
ianw | mnaser: hmm, this is one where ssh responds instantly, but then logging in looks hung | 22:15 |
mnaser | ianw: bummer, because i've been trying to get this merged for a while :( | 22:15 |
ianw | sometimes this has been logind type problems, or dns | 22:16 |
mnaser | ianw: i suspect fs too maybe? cause it was stuck in rpm install before timing out | 22:17 |
*** ociuhandu has quit IRC | 22:19 | |
ianw | yeah, i guess it's gone now. console might help if it comes back | 22:21 |
*** e0ne has joined #openstack-infra | 22:25 | |
openstackgerrit | Merged openstack-infra/tripleo-ci master: Fail on curl error when setting up repo https://review.openstack.org/515695 | 22:27 |
*** thorst has joined #openstack-infra | 22:31 | |
*** thorst has quit IRC | 22:36 | |
*** markvoelker has quit IRC | 22:37 | |
*** salv-orlando has joined #openstack-infra | 22:46 | |
*** e0ne has quit IRC | 22:52 | |
*** e0ne has joined #openstack-infra | 22:53 | |
*** e0ne has quit IRC | 22:53 | |
*** e0ne has joined #openstack-infra | 22:55 | |
*** e0ne has quit IRC | 22:55 | |
*** e0ne has joined #openstack-infra | 22:55 | |
*** e0ne has quit IRC | 22:56 | |
*** e0ne has joined #openstack-infra | 22:56 | |
*** e0ne has quit IRC | 22:56 | |
*** e0ne has joined #openstack-infra | 22:57 | |
*** e0ne has quit IRC | 22:57 | |
*** edmondsw has joined #openstack-infra | 22:59 | |
*** rcernin has joined #openstack-infra | 23:00 | |
*** edmondsw has quit IRC | 23:03 | |
openstackgerrit | James E. Blair proposed openstack-infra/shade master: Zuul: add file extension to playbook path https://review.openstack.org/516145 | 23:04 |
*** hongbin has quit IRC | 23:14 | |
*** gmann_afk is now known as gmann | 23:29 | |
*** markvoelker has joined #openstack-infra | 23:34 | |
*** kazsh has joined #openstack-infra | 23:34 | |
*** jascott1 has joined #openstack-infra | 23:35 | |
*** thorst has joined #openstack-infra | 23:36 | |
*** thorst has quit IRC | 23:37 | |
*** jascott1 has quit IRC | 23:39 | |
*** salv-orlando has quit IRC | 23:45 | |
*** calebb has quit IRC | 23:48 | |
*** kzaitsev1pi has quit IRC | 23:51 | |
*** threestrands has joined #openstack-infra | 23:54 | |
*** kzaitsev_pi has joined #openstack-infra | 23:56 | |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Add new backup hosts to puppet https://review.openstack.org/516148 | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!