| *** harlowja_ has quit IRC | 00:08 | |
| *** xinliang has quit IRC | 00:17 | |
| *** xinliang has joined #zuul | 00:29 | |
| *** xinliang has quit IRC | 00:29 | |
| *** xinliang has joined #zuul | 00:29 | |
| *** odyssey4me has quit IRC | 00:56 | |
| *** odyssey4me has joined #zuul | 00:56 | |
| *** swest has quit IRC | 02:20 | |
| *** swest has joined #zuul | 02:35 | |
| *** dmellado has quit IRC | 02:46 | |
| *** openstackgerrit has joined #zuul | 02:50 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: do not uses builds query args for tenant filter https://review.openstack.org/553932 | 02:50 |
|---|---|---|
| tristanC | corvus: i'm not sure why the tests are not failing, but this secret refactor surely didn't pass sf-ci and i had to revert it. | 02:59 |
| corvus | tristanC: when you say it didn't pass sf-ci -- could you perhaps share the test that failed? | 03:18 |
| corvus | tristanC: i believe you that you say it didn't work, but i think i created a test based on what you described with the failure, and i can't see what's missing. without a failing test, i can't think of what else to do. i need your help. | 03:20 |
| tristanC | sf-ci deploys a full zuul/nodepool and run integration using fully-fledge base job, and with the refactor all the job failed with POST_FAILURE, and 'site_logs' is undefined in the logs | 03:22 |
| tristanC | unfortunately rdo-cloud network is having issues and i can't show you the logs atm | 03:23 |
| tristanC | test succedded with https://review.openstack.org/#/c/553144/, though i went with the revert as i'm not sure of the implication of that freezePlaybook method | 03:24 |
| corvus | tristanC: i wouldn't expect to need that since we should be freezing the playbooks each time we apply a variant (which includes inheritance) | 03:29 |
| *** dmellado has joined #zuul | 03:31 | |
| tristanC | corvus: perhaps the test needs "parent: null" to emulate a true base job using upload-log? | 03:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: WIP: test seret refactor https://review.openstack.org/553935 | 03:44 |
| tristanC | corvus: https://review.openstack.org/553935 should demonstrate the issue | 03:54 |
| corvus | tristanC: oh good idea. that test has a secret in the parent job, but the parent isn't a base job. | 03:58 |
| Diabelko | mmmm, is it normal that if I don't have a nodeset defined for a job, job result (NODE_FAILURE) won't be reported to mysql? | 04:01 |
| Diabelko | because, I believe that's what's happening | 04:01 |
| corvus | tristanC: that does indeed look like the thing i was missing, thanks! i should be able to update the tests with that next week. | 04:01 |
| tristanC | corvus: you're welcome again, it seems like we avoided mass post-failure errors ;) | 04:02 |
| corvus | Diabelko: you shouldn't get node_failure for a job without a nodeset (or a job with an empty nodeset) | 04:03 |
| Diabelko | well, that's what I got, should I look for it in the log and paste it in here and show you the code? | 04:08 |
| Diabelko | http://paste.openstack.org/show/DXtlbdNdB5WFAUMx3Gux/ and last two jobs here https://github.com/Juniper/contrail-project-config/blob/25de1cf90a81c54600c859e4a321bfed3774032c/zuul.d/jobs.yaml#L598 | 04:17 |
| Diabelko | fully reproducible | 04:18 |
| Diabelko | oh, my fault it - it actually gets reported to mysql | 04:29 |
| Diabelko | but it gets reported as node_failure anyway | 04:29 |
| Diabelko | any apparently my filters didn't pick this up, because start and end time have NULL values | 04:32 |
| *** dtruong has quit IRC | 05:31 | |
| *** AJaeger has quit IRC | 05:50 | |
| *** AJaeger has joined #zuul | 06:02 | |
| openstackgerrit | Sorin Sbarnea proposed openstack-infra/zuul-jobs master: Add tox-risk job template https://review.openstack.org/553958 | 08:00 |
| *** yolanda has joined #zuul | 08:06 | |
| *** JasonCL has quit IRC | 10:36 | |
| *** JasonCL has joined #zuul | 11:00 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add zookeeper driver and /{tenant}/labels controller https://review.openstack.org/553979 | 11:48 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add nodepool labels list https://review.openstack.org/553980 | 11:54 |
| *** odyssey4me has quit IRC | 12:53 | |
| *** odyssey4me has joined #zuul | 12:53 | |
| *** Wei_Liu1 has joined #zuul | 13:01 | |
| *** Wei_Liu has quit IRC | 13:01 | |
| *** Wei_Liu1 is now known as Wei_Liu | 13:01 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Refactor "emit-ara-html" into a new role: "ara-report" https://review.openstack.org/513874 | 14:56 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Refactor "emit-ara-html" into a new role: "ara-report" https://review.openstack.org/513874 | 15:00 |
| rcarrillocruz | dmsimard: yo, around? | 15:25 |
| dmsimard | rcarrillocruz: kinda | 15:26 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add zookeeper driver and /{tenant}/labels controller https://review.openstack.org/553979 | 16:08 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add nodepool labels list page https://review.openstack.org/553980 | 16:08 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/nodes controller https://review.openstack.org/553998 | 16:08 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add nodepool nodes status page https://review.openstack.org/553999 | 16:08 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: do not uses builds query args for tenant filter https://review.openstack.org/553932 | 16:23 |
| *** clarkb has quit IRC | 17:19 | |
| *** clarkb has joined #zuul | 17:19 | |
| *** jlvillal has joined #zuul | 17:57 | |
| *** yolanda_ has joined #zuul | 18:02 | |
| *** yolanda has quit IRC | 18:05 | |
| *** ToTodp has joined #zuul | 18:56 | |
| *** sc68cal has joined #zuul | 22:20 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!