| vishalmanchanda | frickler: hello | 08:14 |
|---|---|---|
| vishalmanchanda | I discussed horizon stable/stein branch status about the zuul config errors. | 08:14 |
| vishalmanchanda | https://review.opendev.org/c/openstack/horizon/+/878846 | 08:15 |
| vishalmanchanda | We discussed this topic in yesterday horizon weekly meeting | 08:15 |
| vishalmanchanda | We decided to move stable/stein branch to EOL. | 08:16 |
| vishalmanchanda | that will fix the zuul config errors. | 08:16 |
| vishalmanchanda | I just want to check I need your help in force-merge few open patches in the stable/stein branch if you can. | 08:17 |
| vishalmanchanda | Could you force merge https://review.opendev.org/c/openstack/horizon/+/859754 and https://review.opendev.org/c/openstack/horizon/+/779015 | 08:18 |
| vishalmanchanda | I already w +1 on patches, but it will not merge as CI is broken | 08:21 |
| frickler | vishalmanchanda: hmm, in general we suggest to instead disable failing jobs in zuul, while I technically could do this, this could set a wrong precedent. so I'd need to discuss this with fellow infra-root people first | 08:21 |
| vishalmanchanda | frickler: ohh | 08:21 |
| frickler | vishalmanchanda: what I still could do is force-merge https://review.opendev.org/c/openstack/horizon/+/878846 and then you would be able to fix the remaining CI issues on your own | 08:22 |
| vishalmanchanda | actually I don't want spent too much time on stable/stein branch. | 08:23 |
| vishalmanchanda | I will abandon the open patches for stable/stein branch. | 08:23 |
| vishalmanchanda | as part of moving to EOL. | 08:23 |
| vishalmanchanda | about xstatic-font-awesome, I discussed in horizon meeting. I will push a patch soon to pin version in openstack/requirement and mention the detail why we don't need latest version of font-awosome | 08:25 |
| kozhukalov | Hi folks. Can you please tell me which node pool label should I use to get 32GB or 16GB Ubuntu focal nodes? All the ubuntu-focal labels are min-ram: 8000. And I also can see the ubuntu-bionic-32GB label defined in Vexxhost provider, but there is no ubuntu-focal-32GB. I also see that ubuntu-bionic-32GB is run with v3-standard-8 flavor which is commented as 32GB flavor | 08:47 |
| kozhukalov | https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.opendev.org.yaml#L168 | 08:47 |
| kozhukalov | Looks like I can use any focal label with this flavor v3-standard-8. | 08:48 |
| kozhukalov | For example the label nested-virt-ubuntu-focal seems to be suitable | 08:49 |
| kozhukalov | https://review.opendev.org/c/openstack/openstack-helm-infra/+/884989 | 09:00 |
| opendevreview | Merged openstack/project-config master: Stop testing github hosted projects in openstack tenant https://review.opendev.org/c/openstack/project-config/+/884563 | 09:23 |
| opendevreview | Merged openstack/project-config master: Stop caching infrequently-used CirrOS images https://review.opendev.org/c/openstack/project-config/+/873735 | 09:29 |
| opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Cache new cirros images https://review.opendev.org/c/openstack/project-config/+/885005 | 09:39 |
| fungi | kozhukalov: please propose the additions you want to see for the vexxhost-specific pool of vexxhost-ca-ymq-1 in https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.opendev.org.yaml and we'll review | 12:16 |
| fungi | it's better when the git history has some stakeholder demonstrating an interest in the additional labels since we prefer not to add things unless a project is actually planning to use them | 12:17 |
| opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: Remove build dirs with missing files; add more debug logs https://review.opendev.org/c/openstack/ci-log-processing/+/882270 | 13:39 |
| clarkb | kozhukalov: fungi: also note that those resources are extremely limited and from a single cloud so may go away | 14:48 |
| clarkb | they should probably not be relied on for gating | 14:48 |
| clarkb | fungi: you may want to weigh in on https://review.opendev.org/c/openstack/pbr/+/884791 before a pbr release is made | 14:49 |
| clarkb | however I suspect that this is just how its going to be since setuptools isn't working with build_sphinx anyway regardless of what pbr does | 14:50 |
| fungi | yeah, i was sort of following this saga and meant to look at that | 15:03 |
| fungi | thanks for the reminder | 15:04 |
| fungi | clarkb: one thing i haven't seen addresses in the discussion is what the original motivation was for calling sphinx at package build time. i guess so that docs could get embedded into sdists? | 15:05 |
| fungi | er, haven't seen addressed | 15:05 |
| opendevreview | Merged openstack/pbr master: Remove sphinx doc building integration https://review.opendev.org/c/openstack/pbr/+/884791 | 15:06 |
| clarkb | fungi: was it calling sphinx at package build time? | 15:06 |
| clarkb | fungi: I think this was all to support setup.py build_sphinx which is separate to package building | 15:07 |
| fungi | what other reason would there be to integrate it into a package build library? | 15:07 |
| clarkb | fungi: its so that build_sphinx would work aiui | 15:07 |
| fungi | ah, yeah presumably setuptools build_sphinx feature was there to support embedding built documentation in packages | 15:07 |
| fungi | so pbr was just providing the ability to integrate version information? | 15:08 |
| clarkb | I think pbr was integrating support for setup.cfg configuration of docs builds | 15:08 |
| clarkb | but it may have done more than that | 15:08 |
| fungi | aha | 15:09 |
| fungi | now i remember we did configure that in our projects some years back | 15:09 |
| fungi | but generally stopped once doc builds were redone to call the sphinx_build entrypoint directly | 15:10 |
| fungi | i suppose that's information worth touching on in the release note | 15:11 |
| clarkb | fungi: one thing that made this feel fairly safe is that there were already deprecations in place | 15:13 |
| clarkb | but ya more docs won't hurt | 15:13 |
| opendevreview | Clark Boylan proposed openstack/pbr master: Add release note for build_sphinx removal https://review.opendev.org/c/openstack/pbr/+/885064 | 15:25 |
| clarkb | fungi: ^ | 15:26 |
| clarkb | I think the key bit is "run sphinx tools directly" | 15:26 |
| opendevreview | Merged openstack/pbr master: Use _ instead of - in setup.cfg directives https://review.opendev.org/c/openstack/pbr/+/884938 | 15:29 |
| opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Cache new cirros images https://review.opendev.org/c/openstack/project-config/+/885005 | 16:26 |
| opendevreview | Merged openstack/pbr master: Add release note for build_sphinx removal https://review.opendev.org/c/openstack/pbr/+/885064 | 17:38 |
| opendevreview | Jay Faulkner proposed openstack/pbr master: Replace imp with importlib.machinery https://review.opendev.org/c/openstack/pbr/+/884789 | 17:40 |
| opendevreview | Jay Faulkner proposed openstack/pbr master: Replace imp with importlib.machinery https://review.opendev.org/c/openstack/pbr/+/884789 | 17:56 |
| JayF | I think that patchset will do the trick; py2.7 unit tests passing now. | 18:04 |
| JayF | (rest of CI still running) | 18:04 |
| *** JayF is now known as Guest1962 | 21:37 | |
| *** JasonF is now known as JayF | 21:37 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!