| *** rlandy|bbl is now known as rlandy | 00:01 | |
| *** gmann_afk is now known as gmann | 00:06 | |
| *** jamesmcarthur has quit IRC | 00:09 | |
| *** Goneri has quit IRC | 00:28 | |
| *** jamesmcarthur has joined #zuul | 00:31 | |
| *** jamesmcarthur has quit IRC | 00:35 | |
| *** jamesmcarthur has joined #zuul | 00:40 | |
| *** saneax has quit IRC | 01:09 | |
| mnaser | hmm | 01:11 |
|---|---|---|
| mnaser | {% if some_var %} in ansible will not fail if that variable is not defined, right? | 01:11 |
| *** wuchunyang has joined #zuul | 01:15 | |
| openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul-jobs master: Fix namespace speicification in collect-kubernetes-logs role https://review.opendev.org/735755 | 01:17 |
| *** rlandy has quit IRC | 01:24 | |
| *** swest has quit IRC | 01:25 | |
| *** jamesmcarthur has quit IRC | 01:31 | |
| *** jamesmcarthur has joined #zuul | 01:31 | |
| *** threestrands_ has joined #zuul | 01:32 | |
| *** threestrands has quit IRC | 01:33 | |
| *** threestrands_ has quit IRC | 01:33 | |
| *** threestrands_ has joined #zuul | 01:33 | |
| *** threestrands_ has quit IRC | 01:34 | |
| *** threestrands_ has joined #zuul | 01:35 | |
| *** threestrands_ has quit IRC | 01:36 | |
| *** threestrands_ has joined #zuul | 01:36 | |
| *** jamesmcarthur has quit IRC | 01:37 | |
| *** threestrands_ has quit IRC | 01:37 | |
| *** threestrands_ has joined #zuul | 01:38 | |
| *** threestrands_ has quit IRC | 01:39 | |
| *** threestrands_ has joined #zuul | 01:39 | |
| *** swest has joined #zuul | 01:40 | |
| *** threestrands_ has quit IRC | 01:40 | |
| *** threestrands_ has joined #zuul | 01:41 | |
| *** threestrands_ has quit IRC | 01:42 | |
| *** threestrands_ has joined #zuul | 01:42 | |
| *** threestrands_ has quit IRC | 01:43 | |
| *** threestrands_ has joined #zuul | 01:44 | |
| *** threestrands_ has quit IRC | 01:45 | |
| *** threestrands_ has joined #zuul | 01:45 | |
| *** threestrands_ has quit IRC | 01:46 | |
| *** threestrands_ has joined #zuul | 01:47 | |
| *** threestrands_ has quit IRC | 01:48 | |
| *** threestrands_ has joined #zuul | 01:48 | |
| *** threestrands_ has quit IRC | 01:49 | |
| *** threestrands_ has joined #zuul | 01:50 | |
| *** threestrands_ has quit IRC | 01:51 | |
| *** threestrands_ has joined #zuul | 01:51 | |
| *** threestrands_ has quit IRC | 01:52 | |
| *** threestrands_ has joined #zuul | 01:53 | |
| *** threestrands_ has quit IRC | 01:54 | |
| *** threestrands_ has joined #zuul | 01:54 | |
| *** threestrands_ has quit IRC | 01:55 | |
| *** threestrands_ has joined #zuul | 01:56 | |
| *** threestrands_ has quit IRC | 01:57 | |
| *** threestrands_ has joined #zuul | 01:57 | |
| *** threestrands_ has quit IRC | 01:58 | |
| *** rfolco|rover has quit IRC | 01:58 | |
| *** threestrands_ has joined #zuul | 01:59 | |
| *** threestrands_ has quit IRC | 02:00 | |
| *** threestrands_ has joined #zuul | 02:00 | |
| *** threestrands_ has quit IRC | 02:01 | |
| *** threestrands_ has joined #zuul | 02:02 | |
| *** ysandeep is now known as ysandeep|away | 02:03 | |
| *** jamesmcarthur has joined #zuul | 02:11 | |
| *** jamesmcarthur has quit IRC | 02:15 | |
| *** jamesmcarthur has joined #zuul | 02:15 | |
| *** jamesmcarthur has quit IRC | 02:21 | |
| *** bhavikdbavishi has joined #zuul | 02:29 | |
| *** bhavikdbavishi1 has joined #zuul | 02:36 | |
| *** bhavikdbavishi has quit IRC | 02:38 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 02:38 | |
| *** jamesmcarthur has joined #zuul | 02:48 | |
| *** jamesmcarthur has quit IRC | 02:54 | |
| *** jamesmcarthur has joined #zuul | 02:54 | |
| *** jamesmcarthur has quit IRC | 03:10 | |
| *** jamesmcarthur has joined #zuul | 03:10 | |
| *** jamesmcarthur has quit IRC | 03:15 | |
| *** jamesmcarthur has joined #zuul | 03:15 | |
| *** wuchunyang has quit IRC | 03:16 | |
| *** jamesmcarthur has quit IRC | 03:30 | |
| *** jamesmcarthur has joined #zuul | 03:30 | |
| *** jamesmcarthur has quit IRC | 03:37 | |
| *** jamesmcarthur has joined #zuul | 03:43 | |
| *** wuchunyang has joined #zuul | 03:43 | |
| *** wuchunyang has quit IRC | 04:06 | |
| *** wuchunyang has joined #zuul | 04:07 | |
| *** wuchunyang has quit IRC | 04:11 | |
| *** ysandeep|away is now known as ysandeep | 04:17 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** wuchunyang has joined #zuul | 04:38 | |
| *** saneax has joined #zuul | 04:44 | |
| *** jamesmcarthur has quit IRC | 04:48 | |
| *** jamesmcarthur has joined #zuul | 04:49 | |
| *** jamesmcarthur has quit IRC | 04:53 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix namespace speicification in collect-kubernetes-logs role https://review.opendev.org/735755 | 05:00 |
| *** bhavikdbavishi has quit IRC | 05:29 | |
| *** jamesmcarthur has joined #zuul | 05:31 | |
| *** wuchunyang has quit IRC | 05:38 | |
| *** wuchunyang has joined #zuul | 05:39 | |
| *** felixedel has joined #zuul | 05:43 | |
| *** jamesmcarthur has quit IRC | 05:43 | |
| *** bhavikdbavishi has joined #zuul | 05:45 | |
| *** threestrands_ has quit IRC | 05:52 | |
| *** threestrands_ has joined #zuul | 05:53 | |
| *** threestrands_ has quit IRC | 05:54 | |
| *** threestrands_ has joined #zuul | 05:54 | |
| *** threestrands_ has quit IRC | 05:55 | |
| *** threestrands_ has joined #zuul | 05:56 | |
| *** wuchunyang has quit IRC | 05:57 | |
| *** threestrands_ has quit IRC | 05:57 | |
| *** threestrands_ has joined #zuul | 05:57 | |
| *** threestrands_ has quit IRC | 05:58 | |
| *** threestrands_ has joined #zuul | 05:59 | |
| *** threestrands_ has quit IRC | 06:00 | |
| *** threestrands_ has joined #zuul | 06:00 | |
| *** threestrands_ has quit IRC | 06:01 | |
| *** threestrands_ has joined #zuul | 06:02 | |
| *** hashar has joined #zuul | 06:02 | |
| masterpe | Hi, I'm attaching a second project in zuul, I have attached it to untrusted-projects in /etc/zuul/main.yaml, In that repository I created a zuul.d with files jobs.yaml and projects.yaml. But in zuul/scheduler.log I don't see the project get loaded with the log line "zuul.TenantParser: Loading configuration from ......" | 06:02 |
| *** threestrands_ has quit IRC | 06:03 | |
| *** threestrands_ has joined #zuul | 06:03 | |
| *** threestrands_ has quit IRC | 06:04 | |
| *** threestrands_ has joined #zuul | 06:05 | |
| *** threestrands_ has quit IRC | 06:06 | |
| *** threestrands_ has joined #zuul | 06:06 | |
| *** threestrands_ has quit IRC | 06:07 | |
| *** threestrands_ has joined #zuul | 06:08 | |
| *** threestrands_ has quit IRC | 06:09 | |
| *** threestrands_ has joined #zuul | 06:09 | |
| *** threestrands_ has quit IRC | 06:10 | |
| *** threestrands_ has joined #zuul | 06:11 | |
| *** threestrands_ has quit IRC | 06:12 | |
| *** jamesmcarthur has joined #zuul | 06:12 | |
| *** threestrands_ has joined #zuul | 06:12 | |
| *** threestrands_ has quit IRC | 06:13 | |
| *** threestrands_ has joined #zuul | 06:14 | |
| *** threestrands_ has quit IRC | 06:15 | |
| *** threestrands_ has joined #zuul | 06:15 | |
| *** threestrands_ has quit IRC | 06:16 | |
| *** threestrands_ has joined #zuul | 06:17 | |
| *** wuchunyang has joined #zuul | 06:17 | |
| *** threestrands_ has quit IRC | 06:18 | |
| *** threestrands_ has joined #zuul | 06:18 | |
| *** threestrands_ has quit IRC | 06:19 | |
| *** threestrands_ has joined #zuul | 06:20 | |
| *** hashar has quit IRC | 06:21 | |
| *** threestrands_ has quit IRC | 06:21 | |
| tobiash | masterpe: did you reconfigure or restart zuul? | 06:21 |
| *** threestrands_ has joined #zuul | 06:21 | |
| masterpe | I restarted zuul | 06:22 |
| *** threestrands_ has quit IRC | 06:22 | |
| masterpe | tobiash: and changed /etc/zuul/main.yaml | 06:23 |
| *** threestrands_ has joined #zuul | 06:23 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Remove the -plain job variants https://review.opendev.org/735774 | 06:23 |
| *** threestrands_ has quit IRC | 06:24 | |
| *** threestrands_ has joined #zuul | 06:24 | |
| *** threestrands_ has quit IRC | 06:25 | |
| *** threestrands_ has joined #zuul | 06:26 | |
| *** threestrands_ has quit IRC | 06:27 | |
| *** threestrands_ has joined #zuul | 06:27 | |
| *** jamesmcarthur has quit IRC | 06:28 | |
| *** threestrands_ has quit IRC | 06:28 | |
| *** threestrands_ has joined #zuul | 06:29 | |
| masterpe | I removed alot of branches, it had 44 branches and then it loaded the config | 06:30 |
| *** wuchunyang has quit IRC | 06:30 | |
| *** threestrands_ has quit IRC | 06:30 | |
| *** threestrands_ has joined #zuul | 06:30 | |
| *** threestrands_ has quit IRC | 06:31 | |
| masterpe | What is the max number branches it loads? | 06:32 |
| *** threestrands_ has joined #zuul | 06:32 | |
| *** threestrands_ has quit IRC | 06:33 | |
| *** threestrands_ has joined #zuul | 06:33 | |
| *** threestrands_ has quit IRC | 06:34 | |
| *** threestrands_ has joined #zuul | 06:35 | |
| AJaeger | zuul-jobs maintainer, please review https://review.opendev.org/735774 to remove some node tests we don't need anymore | 06:36 |
| *** threestrands_ has quit IRC | 06:36 | |
| *** threestrands_ has joined #zuul | 06:36 | |
| *** threestrands_ has quit IRC | 06:37 | |
| *** threestrands_ has joined #zuul | 06:38 | |
| *** threestrands_ has quit IRC | 06:39 | |
| *** threestrands_ has joined #zuul | 06:39 | |
| *** threestrands_ has quit IRC | 06:40 | |
| *** threestrands_ has joined #zuul | 06:41 | |
| *** threestrands_ has quit IRC | 06:42 | |
| *** threestrands_ has joined #zuul | 06:42 | |
| *** threestrands_ has quit IRC | 06:43 | |
| *** threestrands_ has joined #zuul | 06:44 | |
| *** threestrands_ has quit IRC | 06:45 | |
| *** threestrands_ has joined #zuul | 06:45 | |
| *** bhavikdbavishi has quit IRC | 06:46 | |
| *** threestrands_ has quit IRC | 06:46 | |
| *** threestrands_ has joined #zuul | 06:47 | |
| *** threestrands_ has quit IRC | 06:48 | |
| *** hashar has joined #zuul | 06:48 | |
| *** threestrands_ has joined #zuul | 06:48 | |
| *** threestrands_ has quit IRC | 06:49 | |
| *** threestrands_ has joined #zuul | 06:50 | |
| *** threestrands_ has quit IRC | 06:51 | |
| *** threestrands_ has joined #zuul | 06:51 | |
| *** threestrands_ has quit IRC | 06:52 | |
| *** threestrands_ has joined #zuul | 06:53 | |
| *** threestrands_ has quit IRC | 06:54 | |
| *** threestrands_ has joined #zuul | 06:54 | |
| *** threestrands_ has quit IRC | 06:55 | |
| *** threestrands_ has joined #zuul | 06:56 | |
| *** threestrands_ has quit IRC | 06:57 | |
| *** rpittau|afk is now known as rpittau | 06:57 | |
| *** threestrands_ has joined #zuul | 06:57 | |
| *** threestrands_ has quit IRC | 06:58 | |
| *** threestrands_ has joined #zuul | 06:59 | |
| *** threestrands_ has quit IRC | 07:00 | |
| *** threestrands_ has joined #zuul | 07:00 | |
| *** threestrands_ has quit IRC | 07:01 | |
| *** sgw1 has quit IRC | 07:01 | |
| *** threestrands_ has joined #zuul | 07:02 | |
| *** threestrands_ has quit IRC | 07:03 | |
| *** threestrands_ has joined #zuul | 07:03 | |
| *** threestrands_ has quit IRC | 07:04 | |
| *** threestrands_ has joined #zuul | 07:05 | |
| *** threestrands_ has quit IRC | 07:06 | |
| *** threestrands_ has joined #zuul | 07:06 | |
| *** jcapitao has joined #zuul | 07:07 | |
| *** threestrands_ has quit IRC | 07:07 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Return upload_results in test-upload-logs-swift role https://review.opendev.org/735503 | 07:11 |
| felixedel | ianw: Thanks for the review :) | 07:13 |
| rpittau | good morning everyone! Please can I get attention on https://review.opendev.org/735174 it's actually a blocker to get tests on Ubuntu Focal with mysql running correctly, thanks! | 07:16 |
| *** bhavikdbavishi has joined #zuul | 07:21 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Fix mqtt log url reporting when report-build-page is active https://review.opendev.org/703983 | 07:24 |
| *** jamesmcarthur has joined #zuul | 07:25 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Fix mqtt log url reporting when report-build-page is active https://review.opendev.org/703983 | 07:27 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Fix mqtt log url reporting when report-build-page is active https://review.opendev.org/703983 | 07:29 |
| *** tosky has joined #zuul | 07:31 | |
| *** jamesmcarthur has quit IRC | 07:31 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: Fix errors with the REST client https://review.opendev.org/728061 | 07:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: fix discrepancies between RPC and REST outputs for autohold https://review.opendev.org/728073 | 07:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add simple testing for Zuul CLI & REST API https://review.opendev.org/728098 | 07:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 07:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 07:42 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add promote endpoint https://review.opendev.org/728489 | 07:42 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add promote endpoint https://review.opendev.org/728489 | 07:45 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: support promote via the REST API https://review.opendev.org/728907 | 07:45 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Builds: add "held" attribute https://review.opendev.org/728994 | 07:46 |
| mhu | (it's rebase time!) | 07:46 |
| *** wuchunyang has joined #zuul | 07:49 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: separate Zuul CLI, Zuul Web CLI https://review.opendev.org/729236 | 07:52 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: add checkbox filters to TableFilters component, filter builds by held status https://review.opendev.org/729265 | 07:53 |
| *** wuchunyang has quit IRC | 07:53 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: add Autoholds Requests page https://review.opendev.org/729307 | 07:53 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 07:53 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-web: support OPTIONS for protected endpoints https://review.opendev.org/734134 | 07:53 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-web: refactor auth token handling code https://review.opendev.org/734139 | 07:54 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/734850 | 07:54 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/735586 | 07:54 |
| *** jpena|off is now known as jpena | 07:56 | |
| avass | AJaeger: should we just merge that? the error does not seem relevant in any way | 08:02 |
| avass | since we're just removing jobs it shouldn't affect anything else | 08:04 |
| AJaeger | avass: yes, let's merge - but we need to fix the error to be able to merge ;) | 08:07 |
| avass | :) | 08:09 |
| *** nils has joined #zuul | 08:10 | |
| *** fbo|off is now known as fbo|eod|afk | 08:14 | |
| *** fbo|eod|afk is now known as fbo|afk | 08:14 | |
| *** hashar_ has joined #zuul | 08:26 | |
| *** hashar has quit IRC | 08:27 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: Fix errors with the REST client https://review.opendev.org/728061 | 08:30 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: fix discrepancies between RPC and REST outputs for autohold https://review.opendev.org/728073 | 08:31 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add simple testing for Zuul CLI & REST API https://review.opendev.org/728098 | 08:32 |
| *** hashar_ is now known as hashar | 08:43 | |
| *** ysandeep is now known as ysandeep|lunch | 08:46 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 08:49 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 08:50 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add promote endpoint https://review.opendev.org/728489 | 08:58 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: support promote via the REST API https://review.opendev.org/728907 | 08:58 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: separate Zuul CLI, Zuul Web CLI https://review.opendev.org/729236 | 08:59 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: add Autoholds Requests page https://review.opendev.org/729307 | 09:00 |
| *** jamesmcarthur has joined #zuul | 09:27 | |
| *** jamesmcarthur has quit IRC | 09:32 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 09:48 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: test: add 'uuid' to 'src_dir' https://review.opendev.org/735576 | 09:50 |
| *** felixedel has quit IRC | 10:01 | |
| *** ysandeep|lunch is now known as ysandeep | 10:05 | |
| *** hashar_ has joined #zuul | 10:10 | |
| *** hashar has quit IRC | 10:11 | |
| *** dmellado has quit IRC | 10:18 | |
| *** hashar__ has joined #zuul | 10:20 | |
| *** bhavikdbavishi has quit IRC | 10:21 | |
| *** hashar__ is now known as hashar | 10:24 | |
| *** hashar_ has quit IRC | 10:24 | |
| *** rpittau is now known as rpittau|bbl | 10:34 | |
| *** dmellado has joined #zuul | 10:41 | |
| mugsie | did the nodepool CI images need to be updated for the removal of tox / pip / etc ? https://review.opendev.org/#/c/554432/ seems toi have failed due to it (or I missed something else) | 10:44 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: test: add 'uuid' to 'src_dir' https://review.opendev.org/735576 | 10:44 |
| *** sshnaidm is now known as sshnaidm|afk | 10:45 | |
| *** hashar has quit IRC | 10:50 | |
| *** tflink has quit IRC | 10:50 | |
| *** tflink has joined #zuul | 10:51 | |
| *** wuchunyang has joined #zuul | 11:01 | |
| *** jcapitao is now known as jcapitao_lunch | 11:03 | |
| *** wuchunyang has quit IRC | 11:05 | |
| *** bhavikdbavishi has joined #zuul | 11:06 | |
| AJaeger | mugsie: not the images - the jobs | 11:06 |
| mugsie | AJaeger: sorry, yes, thats what I meant -_- | 11:07 |
| AJaeger | mugsie: So, yes, there might be same fallout where our jobs/playbooks/roles are not correct | 11:07 |
| *** tflink has quit IRC | 11:08 | |
| *** tflink has joined #zuul | 11:09 | |
| AJaeger | mugsie: so, "ensure-pip" role needs to be added there most likely, see http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015425.html | 11:10 |
| mugsie | yeah, we just reading that email - will look at it now | 11:10 |
| mugsie | AJaeger: that would have to go into all the playbooks/<job>/run.yaml right? | 11:16 |
| openstackgerrit | Albin Vass proposed zuul/nodepool master: WIP: aws: add support for uploading diskimages https://review.opendev.org/735217 | 11:22 |
| *** swest has quit IRC | 11:25 | |
| *** jamesmcarthur has joined #zuul | 11:28 | |
| masterpe | I got a RESULT_TIMED_OUT on a task, but the task was running fine | 11:29 |
| *** jpena is now known as jpena|lunch | 11:32 | |
| *** jamesmcarthur has quit IRC | 11:33 | |
| masterpe | The task was running for 22 minutes, and the the last output was 9 seconds before it gave a "RUN END RESULT_TIMED_OUT" | 11:35 |
| masterpe | is there a timeout setting? | 11:35 |
| masterpe | or what does RESULT_TIMED_OUT mean? | 11:36 |
| *** sshnaidm|afk is now known as sshnaidm | 11:40 | |
| *** swest has joined #zuul | 11:42 | |
| *** rlandy has joined #zuul | 11:46 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 11:53 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/734850 | 11:56 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/735586 | 11:56 |
| *** rfolco|rover has joined #zuul | 11:59 | |
| *** felixedel has joined #zuul | 12:12 | |
| AJaeger | mugsie: adding to run.yaml is fine if it fails there | 12:20 |
| *** bhavikdbavishi has quit IRC | 12:20 | |
| *** jcapitao_lunch is now known as jcapitao | 12:27 | |
| *** rpittau|bbl is now known as rpittau | 12:28 | |
| *** jpena|lunch is now known as jpena | 12:37 | |
| mordred | mugsie: I'd recommend adding it to a pre playbook instead - it's environmental setup | 12:50 |
| *** ysandeep is now known as ysandeep|afk | 12:51 | |
| mordred | mugsie, AJaeger: I can get a nodepool job update patch up - one sec | 12:51 |
| *** sgw1 has joined #zuul | 12:56 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Add ensure-pip to build-python-release https://review.opendev.org/735904 | 12:57 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Add ensure-pip to quick-start job https://review.opendev.org/735905 | 12:58 |
| openstackgerrit | Monty Taylor proposed zuul/nodepool master: Add ensure-tox to k8s and openshift jobs https://review.opendev.org/735906 | 12:59 |
| mordred | AJaeger, mugsie: ^^ that stack should fix nodepoo | 13:00 |
| mordred | zuul-maint: ^^ those three patches should, I hope, fix zuul jobs after the opendev image change | 13:01 |
| AJaeger | wow, three changes ;( | 13:01 |
| AJaeger | +2 on the first one | 13:01 |
| mordred | rpittau: thanks! +A | 13:02 |
| rpittau | mordred: no problem :) | 13:02 |
| *** swest has quit IRC | 13:02 | |
| *** Goneri has joined #zuul | 13:03 | |
| *** bhavikdbavishi has joined #zuul | 13:03 | |
| *** wuchunyang has joined #zuul | 13:03 | |
| sshnaidm | cores, please review collection roles for zuul: https://review.opendev.org/730360 | 13:05 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Add ensure-pip to build-sphinx-docs https://review.opendev.org/735908 | 13:10 |
| fungi | masterpe: yes, specific timeouts can be configured on individual jobs or inherited from parent jobs: https://zuul-ci.org/docs/zuul/reference/job_def.html#attr-job.timeout https://zuul-ci.org/docs/zuul/tutorials/quick-start.html#configure-a-base-job | 13:11 |
| mordred | AJaeger, tristanC: thanks - 735908 is a similar one | 13:12 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Shift docker and pip setup to pre-playbook for quick-start https://review.opendev.org/735910 | 13:16 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Shift docker and pip setup to pre-playbook for quick-start https://review.opendev.org/735910 | 13:16 |
| mordred | tristanC: ^^ there's a cleanup based on your review | 13:17 |
| *** swest has joined #zuul | 13:18 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Add ensure-pip and ensure-virtualenv to build-sphinx-docs https://review.opendev.org/735908 | 13:18 |
| openstackgerrit | Merged zuul/zuul-jobs master: Add ensure-pip to build-python-release https://review.opendev.org/735904 | 13:20 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Add ensure-pip and ensure-virtualenv to build-sphinx-docs https://review.opendev.org/735908 | 13:21 |
| *** fbo|afk is now known as fbo | 13:25 | |
| *** ysandeep|afk is now known as ysandeep | 13:27 | |
| *** jamesmcarthur has joined #zuul | 13:29 | |
| masterpe | fungi: Thanks I missed that | 13:32 |
| *** jamesmcarthur has quit IRC | 13:34 | |
| AJaeger | tristanC: could you check https://review.opendev.org/735908 - Software Factory CI failed; do you know what needs to be done? | 13:38 |
| tristanC | AJaeger: hum, it's dulwich that fails to install because of missing python3-devel file | 13:43 |
| tristanC | https://softwarefactory-project.io/logs/08/735908/3/third-party-check/test-job-build-reno-releasenotes-el7/7e2a469/ara-report/ | 13:44 |
| AJaeger | tristanC: do we need to add something to bindep.txt - or is that something you need to do in Software Factory CI? | 13:44 |
| *** bhavikdbavishi has quit IRC | 13:44 | |
| AJaeger | tristanC: could you also review the change, please? | 13:45 |
| tristanC | AJaeger: i'm not sure, the failure is starting from opendev.org/zuul/zuul-jobs/playbooks/releasenotes/pre.yaml | 13:46 |
| tristanC | I guess we can pre-install python3-devel on our image, but it's odd that this is failing now | 13:47 |
| AJaeger | tristanC: indeed, I'm not aware of changes. Maybe a new dulwich release that has no wheels? | 13:48 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: DNM: test ensure-sphinx role https://review.opendev.org/735919 | 13:49 |
| mordred | tristanC, AJaeger: we have python3-devel in the bindep file already | 13:49 |
| mordred | perhaps those jobs aren't doing a bindep? | 13:49 |
| tristanC | mordred: right, but does reno projects also needs python3-devel ? | 13:50 |
| mordred | (also I agree, it's weird) | 13:50 |
| mordred | tristanC: one would hope not, if people are publishing wheels properly | 13:50 |
| mordred | but yeah - maybe there was a dulwich release issue | 13:50 |
| mordred | dulwich released last 2 days ago | 13:51 |
| tristanC | in anycase, this is not related to mordred changes, https://review.opendev.org/735919 reproduces the issue | 13:51 |
| mordred | and it looks like dulwich has wheels | 13:51 |
| mordred | tristanC: "awesome" | 13:51 |
| mordred | tristanC: I love it when stuff just stops working | 13:52 |
| mordred | oh! | 13:52 |
| mordred | dulwich 0.20.3 only has wheels for >=python3.6 | 13:53 |
| mordred | but we're trying to install dulwich 0.20.2 which was released a month ago | 13:54 |
| mordred | tristanC: that reno venv is python2 | 13:54 |
| mordred | tristanC: stdout: created virtual environment CPython2.7.5.final.0-64 in 513ms | 13:54 |
| tristanC | mordred: yeah, and dulwich is also now using py3.6 syntax | 13:55 |
| mordred | do we need to set something in ensure-sphinx to tell it to use python3 instead of python2? | 13:55 |
| mordred | tristanC: sphinx_python: python2 | 13:55 |
| mordred | we're explicitly setting python2 in the role | 13:56 |
| tristanC | got a syntax error here: https://softwarefactory-project.io/logs/19/735919/1/third-party-check/test-job-build-reno-releasenotes-f31/f79314d/ara-report/result/ac841b95-61c4-46f9-8fa3-c7164ded1bf0/ | 13:56 |
| AJaeger | and in OpenStack that's the value we use | 13:56 |
| mordred | how about we try updating that to python3 | 13:56 |
| AJaeger | would be fine for openstack... | 13:56 |
| tristanC | mordred: that would means reno can no longer be used for py2 projects | 13:57 |
| mordred | tristanC: no - it just means reno itself needs to be run with python3 | 13:57 |
| mordred | you can run python3 reno venv in a python2 project | 13:57 |
| fungi | though also, older versions can still be run on python2 as well, you just miss new features | 13:58 |
| mordred | I mean, to be fair, we could also probably put in some python2-related version pins into reno so that reno in python2 could just use older versions | 13:58 |
| tristanC | mordred: isn't the sphinx python runtime needs to be the same as the one used by the project? | 13:58 |
| mordred | tristanC: possibly - but still, the zuul-jobs default should be python3 not python2 these days. it's a variable, so an individual project that needs it to be python2 for code import reasons can still override it | 13:59 |
| mordred | tristanC: but it's more likely that python2 sphinx won't be able to parse python3 code than the other way around | 13:59 |
| *** Goneri has quit IRC | 14:00 | |
| felixedel | Regarding the python3 error: I saw a similar issue today which makes the zuul-quick-start job fail sporadically https://zuul.opendev.org/t/zuul/build/7cd31005a59940888802217a175699e4 Does anybody have an idea about this? | 14:01 |
| felixedel | We had a similar issue today in our alpine docker images, which changed from alpine:3.11 to alpine:3.12 and with the new version, the pip module was removed from the default python3 installation in alpine. | 14:03 |
| mordred | felixedel: that should be fixed by https://review.opendev.org/735905 | 14:05 |
| mordred | felixedel: we made a change to our base vm images in opendev yesterday and are missing a setup task in that job | 14:06 |
| fungi | as we've discovered time and again, ripping things out of your images is a great way to find out the numerous places you've assumed those things are present | 14:06 |
| mordred | fungi: :) | 14:07 |
| felixedel | mordred: Ah, thanks. Good to know. | 14:07 |
| felixedel | So the error occurs only sporadically because there are still some old images out there which are taken for the builds? | 14:08 |
| mordred | yeah - probably so | 14:10 |
| *** sanjayu_ has joined #zuul | 14:10 | |
| *** piotrowskim_ has joined #zuul | 14:12 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Build sphinx with python3 instead https://review.opendev.org/735923 | 14:12 |
| *** hashar has joined #zuul | 14:12 | |
| felixedel | zuul-maint: Totally different topic: Are there any plans to update zuul-web from using Patternfly 3 to Patternfly 4? | 14:12 |
| *** dmellado_ has joined #zuul | 14:12 | |
| *** ianychoi_ has joined #zuul | 14:12 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Shift docker and pip setup to pre-playbook for quick-start https://review.opendev.org/735910 | 14:13 |
| *** jbryce_ has joined #zuul | 14:13 | |
| mordred | felixedel: I don't know of any active plans - but that's just because nobody has it on their todo list - I imagine it's not something we're opposed to either | 14:13 |
| mordred | iirc, that'll require updating bootstrap as well, which would both be nice, as well as a non-zero amount of work I think | 14:14 |
| *** Goneri has joined #zuul | 14:14 | |
| mordred | we did just update react itself though | 14:14 |
| *** mnaser has quit IRC | 14:14 | |
| *** zbr_ has joined #zuul | 14:15 | |
| *** wxy has quit IRC | 14:15 | |
| *** ChrisShort has quit IRC | 14:15 | |
| *** tflink_ has joined #zuul | 14:15 | |
| *** kmalloc has quit IRC | 14:15 | |
| *** ianychoi has quit IRC | 14:15 | |
| *** saneax has quit IRC | 14:16 | |
| *** piotrowskim has quit IRC | 14:16 | |
| *** dmellado has quit IRC | 14:16 | |
| *** jbryce has quit IRC | 14:16 | |
| *** zbr has quit IRC | 14:16 | |
| *** portdirect has quit IRC | 14:16 | |
| *** jbryce_ is now known as jbryce | 14:16 | |
| *** dmellado_ is now known as dmellado | 14:16 | |
| *** piotrowskim_ is now known as piotrowskim | 14:16 | |
| *** zbr_ is now known as zbr | 14:16 | |
| *** tflink has quit IRC | 14:16 | |
| *** mnaser has joined #zuul | 14:16 | |
| tristanC | mordred: so i'm adding sphinx_python variable to sf third party ci jobs here: https://softwarefactory-project.io/r/18599 | 14:17 |
| *** wxy has joined #zuul | 14:17 | |
| *** kmalloc has joined #zuul | 14:18 | |
| *** ChrisShort has joined #zuul | 14:18 | |
| tristanC | and I guess reno is no longer working on py2 with the latest dulwich since it dropped support for older pythons. | 14:19 |
| mordred | tristanC: not surprising - I think supporting python2 is becoming increasingly harder by the day | 14:21 |
| felixedel | mordred: Patternfly 4 isn't relying on Bootstrap anymore. I found some blog posts saying that it "will" (they are from 2018) be possible to update parts of the website independently of each other and use both versions together for the transition phase: https://blog.patternfly.org/patternfly/patternfly-4-what-does-it-mean-for-existing-users/ and | 14:21 |
| felixedel | https://blog.patternfly.org/patternfly/patternfly-4-migration-update/ But I couldn't find any further information/examples on that in the Patternfly 4 documentation | 14:21 |
| *** wuchunyang has quit IRC | 14:22 | |
| felixedel | But since I'm not familiar with that framework at all, I thought that maybe someone here knows more about this :) | 14:22 |
| mordred | felixedel: yeah - I think that's likely one of the reasons we haven't done anything - I don't think any of us are _so_ familiar with the framework that doing teh transition seems straightforward | 14:24 |
| mordred | "PatternFly-React users will be provided with codemods (scripts) to migrate components from PatternFly 3 to 4" | 14:25 |
| mordred | that seems like a thing we'd want :) | 14:25 |
| *** jamesmcarthur has joined #zuul | 14:30 | |
| *** wuchunyang has joined #zuul | 14:32 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Record artifact checksums and signatures to stdout https://review.opendev.org/735929 | 14:40 |
| *** bhavikdbavishi has joined #zuul | 14:45 | |
| *** jamesmcarthur has quit IRC | 14:49 | |
| *** jamesmcarthur has joined #zuul | 14:49 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 14:50 |
| openstackgerrit | Merged zuul/zuul master: Correct syntax to be compatible with mysql8 https://review.opendev.org/735174 | 14:51 |
| AJaeger | mordred: I was fine for reno to use python3 in openstack - but not sphinx in general. We have the build-sphinx-docs jobs on stable (python2) branches, so if we switch sphinx to python3, will those fail? | 14:54 |
| AJaeger | mordred: btw. https://zuul-ci.org/docs/zuul-jobs/python-jobs.html#jobvar-build-sphinx-docs.sphinx_python | 14:55 |
| *** felixedel has quit IRC | 15:00 | |
| *** wuchunyang has quit IRC | 15:04 | |
| *** wuchunyang has joined #zuul | 15:05 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Add ensure-pip and ensure-virtualenv to build-sphinx-docs https://review.opendev.org/735908 | 15:09 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Build sphinx with python3 instead https://review.opendev.org/735923 | 15:09 |
| mordred | AJaeger: good catch | 15:09 |
| mordred | AJaeger: I do not believe the stable build-sphinx-docs jobs should fail - sphinx installed in python3 should still be able to process | 15:10 |
| mordred | python2 | 15:10 |
| AJaeger | mordred: We use requirements from the repos, those might be old. What about setting in openstack-zuul-jobs for the sphinx jobs the default to python2. So, that for openstack this change is a noop? | 15:11 |
| mordred | AJaeger: I think we can potentially do that - although I'd suggest we only do it for stable branches - we should really not be building docs using python2 anymore by default | 15:11 |
| mordred | AJaeger: the openstack jobs are using build-openstack-sphinx-docs right? | 15:12 |
| AJaeger | mordred: agreed. | 15:12 |
| AJaeger | Yes, build-openstack-sphinx-docs is defined in openstack-zuul-jobs | 15:12 |
| AJaeger | And nobody should run that on master ;) | 15:12 |
| AJaeger | we could even add a branches to not run on master, we deprecated that some time ago... | 15:13 |
| mordred | AJaeger: remote: https://review.opendev.org/735937 Use python2 and xenial for sphinx on stable branches | 15:15 |
| mordred | AJaeger: oh - great! let me update that | 15:15 |
| AJaeger | mordred: run it only on stable branches upto rocky | 15:16 |
| mordred | AJaeger: done | 15:17 |
| mordred | AJaeger: because openstack in general is using tox -edocs now yes? | 15:18 |
| AJaeger | mordred: still two or three user in project-config ;( All in x/ namespace | 15:18 |
| AJaeger | mordred: yes, tox -e docs via the tox-docs or openstack-tox-docs job | 15:18 |
| mordred | AJaeger: well - if those users can't handle python3 in their sphinx, that's on them :) | 15:20 |
| AJaeger | mordred: let's update the docs as well... | 15:20 |
| AJaeger | mordred: agreed | 15:20 |
| mordred | AJaeger: what docs do you think should be updated? | 15:21 |
| AJaeger | mordred: I left a comment | 15:23 |
| *** tflink_ is now known as tflink | 15:24 | |
| AJaeger | mordred: let me do an update - ok? | 15:24 |
| *** wuchunyang has quit IRC | 15:25 | |
| mordred | AJaeger: please! | 15:28 |
| AJaeger | mordred: on it... | 15:28 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 15:32 |
| *** wuchunyang has joined #zuul | 15:48 | |
| *** ysandeep is now known as ysandeep|brb | 15:51 | |
| *** bhavikdbavishi has quit IRC | 16:04 | |
| *** bhavikdbavishi has joined #zuul | 16:09 | |
| *** harrymichal has joined #zuul | 16:11 | |
| *** rpittau is now known as rpittau|afk | 16:12 | |
| *** ysandeep|brb is now known as ysandeep | 16:23 | |
| *** ysandeep is now known as ysandeep|away | 16:30 | |
| *** jamesmcarthur has quit IRC | 16:33 | |
| *** sgw1 has quit IRC | 16:40 | |
| -openstackstatus- NOTICE: Zuul is being restarted for an urgent configuration change and may be offline for 15-30 minutes. Patches uploaded or approved during that time will need to be rechecked. | 16:40 | |
| *** nils has quit IRC | 16:59 | |
| *** jamesmcarthur has joined #zuul | 17:01 | |
| *** sgw1 has joined #zuul | 17:05 | |
| *** harrymichal has quit IRC | 17:09 | |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab: support for the tag_push event https://review.opendev.org/735976 | 17:25 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab: implement git push support https://review.opendev.org/734159 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - add the merge request updated test https://review.opendev.org/734869 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - add dynamic reconfig test https://review.opendev.org/735186 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - add test checking tenant reconfig when ref updated https://review.opendev.org/735187 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab: add enqueue/dequeue unit tests https://review.opendev.org/735188 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - Add support for depends-on dependency https://review.opendev.org/735189 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab: support for the tag_push event https://review.opendev.org/735976 | 17:32 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab: support for the tag_push event https://review.opendev.org/735976 | 17:34 |
| *** jpena is now known as jpena|off | 17:34 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul-jobs master: prepare-workspace: Set root dir to zuul build uuid https://review.opendev.org/735980 | 17:35 |
| *** jcapitao has quit IRC | 17:36 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Rework quick-start and prepare for other tutorials https://review.opendev.org/732066 | 17:37 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate your first patch" https://review.opendev.org/732067 | 17:37 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs" https://review.opendev.org/732068 | 17:37 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline" https://review.opendev.org/732069 | 17:37 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets" https://review.opendev.org/732070 | 17:37 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies" https://review.opendev.org/732071 | 17:37 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul_run_command ret when command is not found https://review.opendev.org/735475 | 17:38 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: zuul_stream: close streamer when task is skipped https://review.opendev.org/735476 | 17:38 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: execute zuul tutorial jobs with pending zuul-jobs changes https://review.opendev.org/735477 | 17:38 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: test job multinode (zuul_stream+callback test) https://review.opendev.org/735483 | 17:38 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: test: add 'uuid' to 'src_dir' https://review.opendev.org/735576 | 17:39 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add 'uuid' to 'src_dir' in order to allow parallel jobs for a static node https://review.opendev.org/735981 | 17:39 |
| noonedeadpunk | hi there:) | 17:44 |
| noonedeadpunk | need some help with required-project stuff | 17:44 |
| noonedeadpunk | so we have some problem with it and a bit stuck with finding a good solution | 17:45 |
| noonedeadpunk | we have defined a big amount of projects as required ones https://opendev.org/openstack/openstack-ansible/src/branch/master/zuul.d/jobs.yaml#L23 | 17:46 |
| noonedeadpunk | as we really want to depends-on work for them | 17:46 |
| noonedeadpunk | later we just symlink repos and use them for testing | 17:46 |
| *** wuchunyang has quit IRC | 17:46 | |
| noonedeadpunk | however, we also do have specific SHA's we want to deploy, which are here https://opendev.org/openstack/openstack-ansible/src/branch/stable/train/ansible-role-requirements.yml | 17:47 |
| noonedeadpunk | like for master things are ok. | 17:47 |
| *** sgw has quit IRC | 17:47 | |
| noonedeadpunk | but on specific branches we kind of in front of choice - either working depends-on or valid sha's we expect to see... | 17:48 |
| noonedeadpunk | I saw option to override-checkout, but we kinda need this only for projects, that doesn't have depends-on... | 17:48 |
| noonedeadpunk | and pretty stuck on this | 17:49 |
| mordred | noonedeadpunk: it is a deep question you ask and one that doens't have an immediate easy answer | 17:49 |
| mordred | I am about to run out to get a sandwich, but would love to chat about it when I get back | 17:50 |
| mordred | (I have some thoughts - I've thought about your pattern there before- but none of them are quick) | 17:50 |
| clarkb | I think that problem is similar to how openstack sometimes wants to test with publishes releases of libraries and sometimes want to test future changes to libraries | 17:50 |
| clarkb | openstack addressed that with two different sets of jobs. Forward looking (where you're depends on would work) and release based to ensure we continue to work against releases with the current context | 17:51 |
| noonedeadpunk | I was thinking about that... but we have so much scenarious, that testing each twice is so resource consuming... | 17:52 |
| mordred | yeah - a short-ish answer is to have something that edits that file - like a pre-playbook | 17:52 |
| mordred | and have it edit the file if there is a depends-on change for a given project in the stack | 17:52 |
| mordred | so normally you just have the shas work as they are | 17:52 |
| mordred | but if there is a depends-on change, you just edit the sha for that project - which is nice because it's just an ansible file | 17:53 |
| *** sgw has joined #zuul | 17:53 | |
| noonedeadpunk | clarkb: also the thing is that is tarted breaking stable branches.. Like we have branchless project https://opendev.org/openstack/ansible-config_template | 17:53 |
| noonedeadpunk | and with recent edit, which is not supported with earlier ansible, stable branches just fail | 17:54 |
| clarkb | noonedeadpunk: ya branchless projects should run tests against all the branches | 17:54 |
| clarkb | tempest and devstack-gate have had to do that | 17:54 |
| clarkb | and yes at some point you sort of have to decide where you've done enough testing and can address things that slip through mroe easily than trying to keep lots and lots of testing workign at all times | 17:54 |
| noonedeadpunk | but the thing is, that old behaviour is going to be deprecated in the upcoming ansible releases... | 17:54 |
| noonedeadpunk | so we can't implent new attitude and keep an old one.... excpet doing nasty things in the code :( | 17:55 |
| noonedeadpunk | mordred: under "that file" you mean jobs.yaml? | 17:56 |
| clarkb | noonedeadpunk: for cases like that I think you either update old code to use new ansible or need more branches | 17:56 |
| mordred | noonedeadpunk: no - ansible-role-requirements | 17:56 |
| mordred | noonedeadpunk: have a zuul pre-playbook mutate shas in that file if there are depends-on changes | 17:56 |
| noonedeadpunk | and we drop required projects, right? | 17:57 |
| mordred | no, I think you still want required projects | 17:58 |
| noonedeadpunk | that can actually work, but I didn't realize how exactly we should override ansible-role-requirements? in terms how we get data to override it with? | 17:58 |
| mordred | noonedeadpunk: look at zuul.items | 17:58 |
| mordred | noonedeadpunk: each repo that exists in zuul.items means that zuul has prepared a change for it | 17:58 |
| mordred | so you want to use the sha of the tip of teh git repo on disk for that project | 17:59 |
| mordred | but if a project is not mentioned in zuul.items, you can leave it alone | 17:59 |
| noonedeadpunk | oh, so it has not all required-projects, but just what is stated in depends-on? | 17:59 |
| mordred | yeah - zuul.items is the list of changes invovled | 18:00 |
| noonedeadpunk | that sounds like a good solution | 18:00 |
| mordred | you should be able to do that just in a zuul pre-playbook so that the actual prod code doesn't have to change - and luckil ansible is good at reading adn writing yaml :) | 18:00 |
| mordred | noonedeadpunk: https://zuul.opendev.org/t/zuul/build/75432cf45e7f48a899af84181869daae/log/zuul-info/inventory.yaml#56 is an example from https://review.opendev.org/#/c/735905/ which has a depends-on on it | 18:02 |
| noonedeadpunk | so currently we do in actual code https://opendev.org/openstack/openstack-ansible/src/branch/master/scripts/get-ansible-role-requirements.yml#L36 | 18:02 |
| noonedeadpunk | but yeh, I got the point | 18:03 |
| noonedeadpunk | ok, that is super useful infom thanks | 18:03 |
| noonedeadpunk | we can work it out I think | 18:03 |
| *** wuchunyang has joined #zuul | 18:04 | |
| *** jamesmcarthur_ has joined #zuul | 18:05 | |
| *** jamesmcarthur has quit IRC | 18:08 | |
| -openstackstatus- NOTICE: Zuul is back online; changes uploaded or approved between 16:40 and 18:00 will need to be rechecked. | 18:09 | |
| *** mugsie has quit IRC | 18:11 | |
| *** mugsie has joined #zuul | 18:15 | |
| *** tosky_ has joined #zuul | 18:26 | |
| *** tosky has quit IRC | 18:28 | |
| *** iurygregory has quit IRC | 18:33 | |
| *** tosky_ is now known as tosky | 18:38 | |
| *** mattw4 has joined #zuul | 18:47 | |
| *** wuchunyang has quit IRC | 18:47 | |
| *** mattw4 has quit IRC | 18:47 | |
| *** bhavikdbavishi has quit IRC | 19:03 | |
| *** bhavikdbavishi has joined #zuul | 19:04 | |
| mhu | something wrong with the zuul-build-python-release job? | 19:13 |
| mhu | 2020-06-16 19:02:31.858635 | ubuntu-bionic | error: invalid command 'bdist_wheel' | 19:15 |
| mhu | in task [build-python-release : Build a tarball and wheel] | 19:15 |
| *** hashar has quit IRC | 19:15 | |
| clarkb | the wheel package is likely missing? | 19:16 |
| mordred | yes | 19:17 |
| mordred | that would be the wheel package missing | 19:17 |
| mordred | we should probalby add ensure-pip to a pre-playbook for that job | 19:18 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Make sure pip is installed for python releases https://review.opendev.org/736001 | 19:21 |
| mordred | mhu, clarkb: ^^ something like taht | 19:21 |
| mordred | nope. that will not fix it - we need an explicit wheel install | 19:22 |
| mordred | mhu, clarkb: CAN OF WORMS | 19:26 |
| mhu | aww man | 19:28 |
| mordred | mhu: I'll get something soon - it'll just be slightly longer than 5 minutes :) | 19:28 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Make sure pip and wheel are installed for python releases https://review.opendev.org/736001 | 19:30 |
| mordred | mhu: that ^^ is the bad version - but I think it's showing that we to restructure there so taht we're installing wheel into a virtualenv | 19:30 |
| mordred | and then using that virtualenv to run bdist_wheel | 19:31 |
| mordred | it's not super hard - we do it elsewhere- but I'm in a meeting | 19:31 |
| *** bhavikdbavishi has quit IRC | 19:41 | |
| *** rfolco|rover is now known as rfolco|rover|brb | 19:56 | |
| *** iurygregory has joined #zuul | 19:59 | |
| AJaeger | mordred: https://review.opendev.org/#/c/736001 is in merge-conflict and needs a rebase | 20:02 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Build sphinx with python3 instead https://review.opendev.org/735923 | 20:05 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Make sure pip and wheel are installed for python releases https://review.opendev.org/736001 | 20:05 |
| mordred | AJaeger: fixed, thanks | 20:05 |
| *** rfolco|rover|brb is now known as rfolco|rover | 20:25 | |
| openstackgerrit | Gonéri Le Bouder proposed zuul/zuul master: github: use change.message in squahsed commit message https://review.opendev.org/736019 | 20:27 |
| *** wuchunyang has joined #zuul | 20:52 | |
| *** y2kenny has joined #zuul | 20:53 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: test job multinode (zuul_stream+callback test) https://review.opendev.org/735483 | 20:54 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add 'uuid' to 'src_dir' in order to allow parallel jobs for a static node https://review.opendev.org/735981 | 20:54 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: test 'uuid' in 'src_dir', 1 static/2 parallel jobs https://review.opendev.org/735576 | 20:54 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API, Web UI: implement nodes filtering https://review.opendev.org/736042 | 21:01 |
| y2kenny | After creating a driver under nodepool/nodepool/driver, is there any other place I need to register the driver? I wrote a driver the got some unit test passing but things seem to get stuck during production deployment (log stopped after "Creating new ProviderManager object for <bla>" | 21:01 |
| clarkb | y2kenny: I think you need to set the driver name in your config and that should match what the driver is called in the drivers dir? | 21:05 |
| y2kenny | clarkb: it does | 21:06 |
| clarkb | y2kenny: hrm the code that does the loading is nodepool/drivers/__init__.py:Drivers | 21:06 |
| clarkb | and I thought it was supposed to be dynamic | 21:06 |
| clarkb | its doing for drivers_path in drivers_paths + [os.path.dirname(__file__)] | 21:06 |
| tristanC | clarkb: y2kenny: it should be dynamic, adding a new directory in the drivers folder should be enough | 21:07 |
| y2kenny | clarkb: that's how I understand it as well. Right now I am trying to find the drivers_path in the container | 21:07 |
| clarkb | I wonder if it is having an error loading the driver and so bails out early? | 21:07 |
| tristanC | y2kenny: have you seen https://zuul-ci.org/docs/nodepool/devguide.html#writing-a-new-provider-driver ? | 21:07 |
| clarkb | (that should be logged fwiw) | 21:07 |
| y2kenny | the nodepool log didn't say much. It loaded the Kubernetes driver fine (it's in the same nodepool.yaml just for santity check.) | 21:08 |
| y2kenny | I am wondering if I need to package the driver somehow in the container building? | 21:09 |
| clarkb | y2kenny: you shouldn't if you are building the container out of the source dir where you've addedthe driver | 21:09 |
| clarkb | oh except ther ecould be one thing there | 21:09 |
| clarkb | have you git added your new files yet? | 21:09 |
| y2kenny | yes | 21:09 |
| clarkb | the source installation is going to look for files that are in the git tree so you need to git add them first I think | 21:09 |
| y2kenny | clarkb: there wouldn't be a 'driver' directory in the container would it? | 21:17 |
| *** rfolco|rover has quit IRC | 21:18 | |
| clarkb | y2kenny: it should be in /usr/lib/python3.6/site-packages/nodepool/driver or similar to that | 21:20 |
| clarkb | might be /usr/local/lib | 21:21 |
| y2kenny | yes... found it | 21:21 |
| clarkb | or dist-packages instead of site-packages but something along those lines | 21:21 |
| y2kenny | and the driver is there... um... | 21:21 |
| clarkb | is it executable? | 21:21 |
| clarkb | that shouldn't matter but I wonder if file perms are realted | 21:21 |
| y2kenny | the permission aligns with the k8s and gce drivers | 21:22 |
| y2kenny | OH... looks like it's my driver. something timed out during start | 21:24 |
| y2kenny | AHHHH... sorry... I put my functions in the wrong place (in __init__ instead of start.) | 21:25 |
| y2kenny | my bad. | 21:25 |
| *** wuchunyang has quit IRC | 21:28 | |
| *** wuchunyang has joined #zuul | 21:29 | |
| *** jamesmcarthur_ has quit IRC | 21:36 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-website master: Add user story summaries for Wazo and T-Systems https://review.opendev.org/736053 | 21:39 |
| *** sugaar1 has joined #zuul | 21:56 | |
| *** sugaar has quit IRC | 21:56 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul-jobs master: WIP: prepare-workspace: Set root dir to zuul build uuid https://review.opendev.org/735980 | 22:04 |
| *** irclogbot_2 has quit IRC | 22:07 | |
| *** tobberydberg has quit IRC | 22:08 | |
| *** y2kenny has quit IRC | 22:09 | |
| *** tobberydberg has joined #zuul | 22:09 | |
| *** irclogbot_0 has joined #zuul | 22:14 | |
| *** jkt has quit IRC | 22:14 | |
| *** jkt has joined #zuul | 22:14 | |
| *** rlandy is now known as rlandy|bbl | 22:16 | |
| *** iurygregory has quit IRC | 22:33 | |
| *** mugsie has quit IRC | 22:33 | |
| *** ChrisShort has quit IRC | 22:33 | |
| *** yoctozepto has quit IRC | 22:33 | |
| *** ttx has quit IRC | 22:33 | |
| *** masterpe has quit IRC | 22:33 | |
| *** Tahvok has quit IRC | 22:33 | |
| *** amotoki has quit IRC | 22:33 | |
| *** bhagyashris has quit IRC | 22:33 | |
| *** dmsimard has quit IRC | 22:33 | |
| *** fdegir has quit IRC | 22:33 | |
| *** corvus has quit IRC | 22:33 | |
| *** pleia2 has quit IRC | 22:33 | |
| *** ysandeep|away has quit IRC | 22:33 | |
| *** evgenyl has quit IRC | 22:33 | |
| *** tristanC has quit IRC | 22:33 | |
| *** PrinzElvis has quit IRC | 22:33 | |
| *** mhu has quit IRC | 22:33 | |
| *** irclogbot_0 has quit IRC | 22:33 | |
| *** johnsom has quit IRC | 22:33 | |
| *** mwhahaha has quit IRC | 22:33 | |
| *** maxamillion has quit IRC | 22:33 | |
| *** tdasilva has quit IRC | 22:33 | |
| *** iamweswilson has quit IRC | 22:33 | |
| *** tributarian has quit IRC | 22:33 | |
| *** SotK has quit IRC | 22:33 | |
| *** tobiash has quit IRC | 22:33 | |
| *** dpawlik6 has quit IRC | 22:33 | |
| *** wxy has quit IRC | 22:33 | |
| *** kmalloc has quit IRC | 22:33 | |
| *** mnaser has quit IRC | 22:33 | |
| *** tflink has quit IRC | 22:33 | |
| *** piotrowskim has quit IRC | 22:33 | |
| *** gmann has quit IRC | 22:33 | |
| *** leoluk has quit IRC | 22:33 | |
| *** andreaf has quit IRC | 22:33 | |
| *** Pilou has quit IRC | 22:33 | |
| *** wuchunyang has quit IRC | 22:33 | |
| *** sgw has quit IRC | 22:33 | |
| *** vblando has quit IRC | 22:33 | |
| *** mnasiadka has quit IRC | 22:33 | |
| *** jkt has quit IRC | 22:33 | |
| *** sugaar1 has quit IRC | 22:33 | |
| *** Goneri has quit IRC | 22:33 | |
| *** dmellado has quit IRC | 22:33 | |
| *** olaph has quit IRC | 22:33 | |
| *** avass has quit IRC | 22:33 | |
| *** pots has quit IRC | 22:33 | |
| *** erbarr has quit IRC | 22:33 | |
| *** ericsysmin has quit IRC | 22:33 | |
| *** logan- has quit IRC | 22:33 | |
| *** sanjayu_ has quit IRC | 22:33 | |
| *** yolanda has quit IRC | 22:33 | |
| *** paladox has quit IRC | 22:33 | |
| *** panda has quit IRC | 22:33 | |
| *** arxcruz has quit IRC | 22:33 | |
| *** kgz has quit IRC | 22:33 | |
| *** EmilienM has quit IRC | 22:33 | |
| *** nhicher has quit IRC | 22:33 | |
| *** jhesketh has quit IRC | 22:33 | |
| *** jlk has quit IRC | 22:33 | |
| *** AJaeger has quit IRC | 22:33 | |
| *** johanssone has quit IRC | 22:33 | |
| *** tobberydberg has quit IRC | 22:33 | |
| *** rlandy|bbl has quit IRC | 22:33 | |
| *** decimuscorvinus has quit IRC | 22:33 | |
| *** reiterative has quit IRC | 22:33 | |
| *** webknjaz has quit IRC | 22:33 | |
| *** gundalow has quit IRC | 22:33 | |
| *** mgoddard has quit IRC | 22:33 | |
| *** dcastellani has quit IRC | 22:33 | |
| *** mrcirca has quit IRC | 22:33 | |
| *** ChanServ has quit IRC | 22:33 | |
| *** smyers has quit IRC | 22:33 | |
| *** stevthedev has quit IRC | 22:33 | |
| *** adam_g has quit IRC | 22:33 | |
| *** persia has quit IRC | 22:33 | |
| *** mgagne has quit IRC | 22:33 | |
| *** jbryce has quit IRC | 22:33 | |
| *** sshnaidm has quit IRC | 22:33 | |
| *** cloudnull has quit IRC | 22:33 | |
| *** ianw has quit IRC | 22:33 | |
| *** bstinson has quit IRC | 22:33 | |
| *** openstackgerrit has quit IRC | 22:33 | |
| *** frickler has quit IRC | 22:33 | |
| *** zbr has quit IRC | 22:33 | |
| *** swest has quit IRC | 22:33 | |
| *** evrardjp has quit IRC | 22:33 | |
| *** Open10K8S has quit IRC | 22:33 | |
| *** samccann has quit IRC | 22:33 | |
| *** lseki has quit IRC | 22:33 | |
| *** guilhermesp has quit IRC | 22:33 | |
| *** donnyd has quit IRC | 22:33 | |
| *** noonedeadpunk has quit IRC | 22:33 | |
| *** kklimonda has quit IRC | 22:33 | |
| *** rpittau|afk has quit IRC | 22:33 | |
| *** mordred has quit IRC | 22:33 | |
| *** ironfoot has quit IRC | 22:33 | |
| *** fungi has quit IRC | 22:33 | |
| *** tosky has quit IRC | 22:33 | |
| *** marvs has quit IRC | 22:33 | |
| *** fbo has quit IRC | 22:33 | |
| *** SpamapS has quit IRC | 22:33 | |
| *** weshay|ruck has quit IRC | 22:33 | |
| *** shanemcd has quit IRC | 22:33 | |
| *** clarkb has quit IRC | 22:33 | |
| *** aluria has quit IRC | 22:33 | |
| *** sgw1 has quit IRC | 22:33 | |
| *** guillaumec has quit IRC | 22:33 | |
| *** jpena|off has quit IRC | 22:33 | |
| *** Miouge- has quit IRC | 22:33 | |
| *** andreykurilin has quit IRC | 22:33 | |
| *** freefood has quit IRC | 22:33 | |
| *** mmedvede has quit IRC | 22:33 | |
| *** gothicmindfood has quit IRC | 22:33 | |
| *** pots has joined #zuul | 22:37 | |
| *** avass has joined #zuul | 22:37 | |
| *** olaph has joined #zuul | 22:37 | |
| *** dmellado has joined #zuul | 22:37 | |
| *** Goneri has joined #zuul | 22:37 | |
| *** sugaar1 has joined #zuul | 22:37 | |
| *** jkt has joined #zuul | 22:37 | |
| *** fungi has joined #zuul | 22:37 | |
| *** ironfoot has joined #zuul | 22:37 | |
| *** mordred has joined #zuul | 22:37 | |
| *** rpittau|afk has joined #zuul | 22:37 | |
| *** kklimonda has joined #zuul | 22:37 | |
| *** noonedeadpunk has joined #zuul | 22:37 | |
| *** guilhermesp has joined #zuul | 22:37 | |
| *** lseki has joined #zuul | 22:37 | |
| *** donnyd has joined #zuul | 22:37 | |
| *** samccann has joined #zuul | 22:37 | |
| *** Open10K8S has joined #zuul | 22:37 | |
| *** evrardjp has joined #zuul | 22:37 | |
| *** swest has joined #zuul | 22:37 | |
| *** zbr has joined #zuul | 22:37 | |
| *** johanssone has joined #zuul | 22:37 | |
| *** AJaeger has joined #zuul | 22:37 | |
| *** jlk has joined #zuul | 22:37 | |
| *** jhesketh has joined #zuul | 22:37 | |
| *** nhicher has joined #zuul | 22:37 | |
| *** EmilienM has joined #zuul | 22:37 | |
| *** kgz has joined #zuul | 22:37 | |
| *** arxcruz has joined #zuul | 22:37 | |
| *** panda has joined #zuul | 22:37 | |
| *** paladox has joined #zuul | 22:37 | |
| *** yolanda has joined #zuul | 22:37 | |
| *** sanjayu_ has joined #zuul | 22:37 | |
| *** tdasilva has joined #zuul | 22:37 | |
| *** maxamillion has joined #zuul | 22:37 | |
| *** mwhahaha has joined #zuul | 22:37 | |
| *** johnsom has joined #zuul | 22:37 | |
| *** irclogbot_0 has joined #zuul | 22:37 | |
| *** sgw has joined #zuul | 22:37 | |
| *** wuchunyang has joined #zuul | 22:37 | |
| *** logan- has joined #zuul | 22:37 | |
| *** ericsysmin has joined #zuul | 22:37 | |
| *** erbarr has joined #zuul | 22:37 | |
| *** dpawlik6 has joined #zuul | 22:37 | |
| *** tobiash has joined #zuul | 22:37 | |
| *** SotK has joined #zuul | 22:37 | |
| *** iamweswilson has joined #zuul | 22:37 | |
| *** gothicmindfood has joined #zuul | 22:37 | |
| *** mmedvede has joined #zuul | 22:37 | |
| *** freefood has joined #zuul | 22:37 | |
| *** andreykurilin has joined #zuul | 22:37 | |
| *** Miouge- has joined #zuul | 22:37 | |
| *** jpena|off has joined #zuul | 22:37 | |
| *** guillaumec has joined #zuul | 22:37 | |
| *** sgw1 has joined #zuul | 22:37 | |
| *** mhu has joined #zuul | 22:37 | |
| *** PrinzElvis has joined #zuul | 22:37 | |
| *** ChanServ has joined #zuul | 22:37 | |
| *** Tahvok has joined #zuul | 22:37 | |
| *** amotoki has joined #zuul | 22:37 | |
| *** bhagyashris has joined #zuul | 22:37 | |
| *** dmsimard has joined #zuul | 22:37 | |
| *** fdegir has joined #zuul | 22:37 | |
| *** corvus has joined #zuul | 22:37 | |
| *** pleia2 has joined #zuul | 22:37 | |
| *** tepper.freenode.net sets mode: +o ChanServ | 22:37 | |
| *** ysandeep|away has joined #zuul | 22:37 | |
| *** evgenyl has joined #zuul | 22:37 | |
| *** tristanC has joined #zuul | 22:37 | |
| *** iurygregory has joined #zuul | 22:37 | |
| *** mugsie has joined #zuul | 22:37 | |
| *** ChrisShort has joined #zuul | 22:37 | |
| *** yoctozepto has joined #zuul | 22:37 | |
| *** jbryce has joined #zuul | 22:38 | |
| *** sshnaidm has joined #zuul | 22:38 | |
| *** cloudnull has joined #zuul | 22:38 | |
| *** ianw has joined #zuul | 22:38 | |
| *** bstinson has joined #zuul | 22:38 | |
| *** openstackgerrit has joined #zuul | 22:38 | |
| *** frickler has joined #zuul | 22:38 | |
| *** masterpe has joined #zuul | 22:38 | |
| *** ttx has joined #zuul | 22:38 | |
| *** smyers has joined #zuul | 22:38 | |
| *** stevthedev has joined #zuul | 22:38 | |
| *** adam_g has joined #zuul | 22:38 | |
| *** mgagne has joined #zuul | 22:38 | |
| *** persia has joined #zuul | 22:38 | |
| *** tobberydberg has joined #zuul | 22:38 | |
| *** rlandy|bbl has joined #zuul | 22:38 | |
| *** decimuscorvinus has joined #zuul | 22:38 | |
| *** reiterative has joined #zuul | 22:38 | |
| *** webknjaz has joined #zuul | 22:38 | |
| *** gundalow has joined #zuul | 22:38 | |
| *** mrcirca has joined #zuul | 22:38 | |
| *** mgoddard has joined #zuul | 22:38 | |
| *** dcastellani has joined #zuul | 22:38 | |
| *** kmalloc has joined #zuul | 22:39 | |
| *** wxy has joined #zuul | 22:39 | |
| *** mnaser has joined #zuul | 22:39 | |
| *** tflink has joined #zuul | 22:39 | |
| *** piotrowskim has joined #zuul | 22:39 | |
| *** leoluk has joined #zuul | 22:39 | |
| *** gmann has joined #zuul | 22:39 | |
| *** andreaf has joined #zuul | 22:39 | |
| *** Pilou has joined #zuul | 22:39 | |
| *** erbarr has quit IRC | 22:39 | |
| *** mwhahaha has quit IRC | 22:39 | |
| *** masterpe has quit IRC | 22:40 | |
| *** tosky has joined #zuul | 22:40 | |
| *** ericsysmin has quit IRC | 22:40 | |
| *** dcastellani has quit IRC | 22:40 | |
| *** gundalow has quit IRC | 22:41 | |
| *** mnaser has quit IRC | 22:41 | |
| *** gmann has quit IRC | 22:41 | |
| *** weshay|ruck has joined #zuul | 22:42 | |
| *** marvs has joined #zuul | 22:42 | |
| *** fbo has joined #zuul | 22:42 | |
| *** SpamapS has joined #zuul | 22:42 | |
| *** shanemcd has joined #zuul | 22:42 | |
| *** clarkb has joined #zuul | 22:42 | |
| *** aluria has joined #zuul | 22:42 | |
| *** ericsysmin has joined #zuul | 22:42 | |
| *** mwhahaha has joined #zuul | 22:43 | |
| *** gundalow has joined #zuul | 22:43 | |
| *** erbarr has joined #zuul | 22:43 | |
| *** dcastellani has joined #zuul | 22:44 | |
| *** gmann has joined #zuul | 22:44 | |
| *** wuchunyang has quit IRC | 22:47 | |
| *** mnaser has joined #zuul | 22:48 | |
| *** wuchunyang has joined #zuul | 22:48 | |
| *** vblando has joined #zuul | 22:51 | |
| mordred | zuul-maint: https://review.opendev.org/#/c/735923/ and https://review.opendev.org/#/c/736001/ are fixes for issues that came up in jobs today | 22:52 |
| corvus | mordred: re 923 -- do you think we need to announce that? | 22:52 |
| mordred | we'll need to land them so that we can recheck and land https://review.opendev.org/#/c/735905/ so that we can be all green | 22:52 |
| mordred | corvus: maybe? it _shouldn't_ be noticable to fokls? but maybe we should (we can just set sphinx_python to python3 in zuul for the short-term if we want an announcement period) | 22:53 |
| corvus | mordred: how did we end up noticing the problem? | 22:54 |
| mordred | corvus: why don't I rebase that to put the pip/wheel change first - it's the one we need to unstick the other zuul job | 22:54 |
| mordred | corvus: in a job - dulwich now only supports python >= 3.6 | 22:55 |
| corvus | mordred: re pip/wheel -- is become needed? and are we sure that'll work in all cases? | 22:55 |
| mordred | so trying to build docs with sphinx on python2 failed | 22:55 |
| corvus | mordred: ack | 22:55 |
| mordred | corvus: it's a pip install that isn't in a virtualenv - so I think it needs the become for now - it didn't break the SF jobs | 22:55 |
| corvus | mordred: re pip -- i think we've been going in the direction of not requiring root for python things? | 22:55 |
| mordred | corvus: I do think we should rework that to use a virtualenv | 22:55 |
| mordred | yeah | 22:55 |
| mordred | do you think we should rework that for this one? | 22:56 |
| mordred | (this is one of those "this job worked without root when the base image had wheel pre-installed" joys) | 22:57 |
| corvus | mordred: hrm, i just worry that someone has that working in some env that doesn't require root | 22:57 |
| mordred | fair enough. let me take another swing at it real quick | 22:57 |
| corvus | mordred: i see 2 options | 22:57 |
| mordred | ooh. I like options | 22:57 |
| corvus | mordred: 1) your virtualenv rework; or 2) protect that with a 'is wheel installed' check | 22:58 |
| corvus | 2 might be easier/faster? 1 is better | 22:58 |
| tristanC | i think it's fair that zuul-jobs uses py3 by default now, in particular for reno that breaks otherwise. perhaps this could use an announces, like, `if you use zuul to test python2 project, then you need to set sphinx_python, ... to "python3"` | 22:58 |
| corvus | tristanC, mordred: yeah, i'm leaning toward an announcement for that, but since we think it probably should not break anyone, maybe we can just do a 1 week notice? | 22:59 |
| mordred | corvus: yeah - let's do 2 today and 1 tomorrow though - because I think 1 is going to be a big patch | 22:59 |
| corvus | looking further at 923 -- i think the fact that openstack would be broken by it argues rather strongly for a notice | 23:01 |
| corvus | (not for openstack, there's a patch for that, but for anyone else) | 23:01 |
| *** mnasiadka has joined #zuul | 23:01 | |
| *** wuchunyang has quit IRC | 23:02 | |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Make sure wheel is installed for python releases https://review.opendev.org/736001 | 23:03 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Build sphinx with python3 instead https://review.opendev.org/735923 | 23:03 |
| mordred | corvus: kk. there's an update on the whee patch | 23:03 |
| *** masterpe has joined #zuul | 23:03 | |
| mordred | tristanC: does 736001 look ok to you (for now - I'll rework it more better tomorrow) | 23:04 |
| mordred | that _should_ allow https://review.opendev.org/#/c/735905/ to land | 23:05 |
| corvus | mordred: typo | 23:06 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Make sure wheel is installed for python releases https://review.opendev.org/736001 | 23:08 |
| mordred | corvus: it's almost like I copied that from another role | 23:08 |
| corvus | mordred: lgtm | 23:09 |
| *** Goneri has quit IRC | 23:24 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Make sure wheel is installed for python releases https://review.opendev.org/736001 | 23:26 |
| mordred | corvus: I've re-enqueued https://review.opendev.org/#/c/735905 | 23:28 |
| corvus | mordred: thanks! | 23:32 |
| *** rlandy|bbl is now known as rlandy | 23:35 | |
| *** tosky has quit IRC | 23:50 | |
| *** wuchunyang has joined #zuul | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!