sean-k-mooney | https://paste.centos.org/view/60dd37f7 | 00:01 |
---|---|---|
sean-k-mooney | seams ot work | 00:01 |
sean-k-mooney | cardoe: perhaps instead of tryign to use go install which will isntall the release i think form source | 00:03 |
sean-k-mooney | for this specific case using the static bin approch is better | 00:04 |
sean-k-mooney | by the way becareful with using tabs in makefiles | 00:05 |
sean-k-mooney | it does change the meaning vs spaces | 00:06 |
sean-k-mooney | the error is compelting about https://github.com/norwoodj/helm-docs/blob/master/go.mod#L3 | 00:07 |
cardoe | yeah a tab is a recipe prefix | 00:07 |
sean-k-mooney | which i have only seen fi that is a newer version fo go then installed | 00:07 |
cardoe | Yeah I'm not sure where 1.23 is coming from | 00:08 |
cardoe | This all works fine locally on a Ubuntu 24.04 box and on a Mac. | 00:09 |
sean-k-mooney | 24.04 has 1.23 in universe but the default in main is 1.22.2 | 00:11 |
clarkb | https://github.com/golang/go/issues/61888 makes it seem like 1.21 is the problem though not 1.22 | 00:12 |
opendevreview | Merged openstack/project-config master: Disable raxflex cloud https://review.opendev.org/c/openstack/project-config/+/935575 | 00:12 |
clarkb | oh that job ran on jammy not noble so ya that could still be it | 00:12 |
sean-k-mooney | looking at https://repology.org/project/go/versions jammy has 1.18.1 in main but 1.20 1.21 1.22 and 1.23 in updates/universe | 00:15 |
sean-k-mooney | that is the downside of go install. its fine if you ahve a workign toolcian but you can hit issues like this from time to time | 00:15 |
cardoe | So I should just use their binary package? | 00:18 |
cardoe | via the makefile? | 00:18 |
cardoe | or you guys want me to make a zuul role? | 00:18 |
sean-k-mooney | you could, they dont actully have a binary that is standalone but they have an rpm and deb file | 00:19 |
sean-k-mooney | there is also a docker contiener so anothe roption would be to use docker/podman to build the docs | 00:19 |
sean-k-mooney | it partly depend on what other tools you are using | 00:19 |
clarkb | the downside to that is docker hub may be reducing rate limits pretty significantly so you may replace one problem with another. However, you're already using docker so maybe that needs solving anyway | 00:20 |
sean-k-mooney | depending on yoru oter deps you coud coniser just moving that job to noble | 00:20 |
cardoe | They have a tarballed binary. | 00:20 |
sean-k-mooney | oh that is not the source | 00:20 |
cardoe | https://github.com/norwoodj/helm-docs/releases/download/v1.14.2/helm-docs_1.14.2_Linux_x86_64.tar.gz | 00:20 |
sean-k-mooney | ya i tought htat was the soruce tar | 00:21 |
sean-k-mooney | but thats an option yes | 00:21 |
cardoe | I can move the job to noble. I dunno how yet. | 00:21 |
clarkb | your job liekly defines a nodeset somewhere because the default nodest is noble now | 00:21 |
cardoe | https://opendev.org/openstack/openstack-helm/src/commit/bc2ebae06634051764ffddac96227ad956dd6437/zuul.d/project.yaml#L20 afaik that's what runs it. | 00:21 |
clarkb | you would update that nodeset to s/jammy/noble/ or remove it entirely and fall back to the default (which does update over time so up to you) | 00:22 |
cardoe | https://opendev.org/openstack/openstack-helm/src/branch/master/zuul.d/nodesets.yaml there's that. | 00:22 |
clarkb | oh this uses the standard openstack-tox-docs job (I didn't expect that) | 00:23 |
sean-k-mooney | it apprently has not be update yet | 00:24 |
clarkb | this makes the dance a little more complicated because you can't use the standard template if you want to modify what you are doing. You would drop that template then add openstack-tox-docs to these pipelines to match the template https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/project-templates.yaml#L46-L73 | 00:24 |
sean-k-mooney | that must be commign form publish-openstack-docs-pti | 00:24 |
clarkb | then for each of those jobs you append :\n nodeset: ubuntu-noble | 00:24 |
clarkb | that creates a local override in your config forcing noble to be hte nodeset | 00:24 |
*** __ministry is now known as Guest163 | 07:34 | |
*** elodilles_pto is now known as elodilles | 08:03 | |
opendevreview | Artem Goncharov proposed openstack/pbr master: add python 3.12 support https://review.opendev.org/c/openstack/pbr/+/924216 | 09:38 |
fungi | gmann: https://review.opendev.org/934648 is waiting on your review, as the oslo infra liaison | 14:16 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: End gating for Murano/Senlin/Sahara OSA roles https://review.opendev.org/c/openstack/project-config/+/935671 | 16:20 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: End gating for Murano/Senlin/Sahara OSA roles https://review.opendev.org/c/openstack/project-config/+/935671 | 16:41 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: Remove Murano/Senlin/Sahara OSA roles from infra https://review.opendev.org/c/openstack/project-config/+/935683 | 16:58 |
gmann | fungi: +1 | 17:04 |
stephenfin | M0weng[m]: Can you see this? | 17:13 |
M0weng[m] | stephenfin: test ping | 17:16 |
stephenfin | Sweet | 17:16 |
M0weng[m] | fungi or clarkb : I was trying to paste an output to paste.openstack.org that it flagged as potential spam, but for some reason the captcha images aren't fitting in the frames today - examples: | 17:21 |
* M0weng[m] uploaded an image: (88KiB) < https://matrix.org/oftc/media/v1/media/download/AZzOYlV3ybbfohPGAZOCsLTCrudgontqvCyVkPWBogjzvN1YslT5ZA20eF7JX9rogfXpM2DuMroHcvfsq4KBx19CeTjYsBpwAG1hdHJpeC5vcmcvU3ZoVmlrTGhWRXJtdXRDTW5VSXNtS0VM > | 17:21 | |
* M0weng[m] uploaded an image: (98KiB) < https://matrix.org/oftc/media/v1/media/download/AZqML3OZ8mIUtVQRKotUUYrWivFDvNDiyBAlfwoCRtFFiwHrNE2Z4n6wFNVj0_N0G_PA2q47iD9cJQ5B7NpmA0VCeTjYslmQAG1hdHJpeC5vcmcvQnVDV29MS21DR2ZpY3VYeG1DU0tQdXdS > | 17:22 | |
* M0weng[m] uploaded an image: (70KiB) < https://matrix.org/oftc/media/v1/media/download/AdJFODTstRuMkroSaRFgAyKl3RwKi53ESMSwkWBj7ZIldQSHXqmo8qg_QCN26ypP_axkCM1r4QmhMbJQehhCqLRCeTjYtIEwAG1hdHJpeC5vcmcvUE1hTkRJY0JkQ09MQWJUVEhkTUF1aVlq > | 17:22 | |
clarkb | the image lib must've updated again | 17:22 |
M0weng[m] | Eventually I figured out that you could see the whole image by opening it in a new tab, but Stephen suggested I bring this up in this channel. | 17:23 |
M0weng[m] | clarkb: Ahh makes sense | 17:23 |
clarkb | ya its good to know. Unfortunately its very likely to not be a high priority right now | 17:23 |
M0weng[m] | What repo would this be in? I can take a look | 17:23 |
clarkb | the code is all up for anyone that wants to debug it further though in https://opendev.org/opendev/lodgeit. | 17:23 |
M0weng[m] | Ohh nice, thank you! | 17:24 |
clarkb | https://opendev.org/opendev/lodgeit/src/branch/master/lodgeit/lib/captcha.py somewhere in there likely | 17:24 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: Add repository for OSA's httpd role https://review.opendev.org/c/openstack/project-config/+/935693 | 17:54 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: Add repository for OSA's httpd role https://review.opendev.org/c/openstack/project-config/+/935693 | 17:58 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: Add Zuul jobs for ansible-role-httpd project https://review.opendev.org/c/openstack/project-config/+/935695 | 18:00 |
opendevreview | Dmitriy Rabotyagov proposed openstack/project-config master: Add Zuul jobs for frrouting role https://review.opendev.org/c/openstack/project-config/+/935696 | 18:03 |
opendevreview | Clark Boylan proposed openstack/project-config master: Revert "Disable raxflex cloud" https://review.opendev.org/c/openstack/project-config/+/935698 | 18:29 |
opendevreview | Merged openstack/project-config master: End gating for Murano/Senlin/Sahara OSA roles https://review.opendev.org/c/openstack/project-config/+/935671 | 18:30 |
opendevreview | Merged openstack/project-config master: Revert "Disable raxflex cloud" https://review.opendev.org/c/openstack/project-config/+/935698 | 19:05 |
clarkb | M0weng[m]: I see the patch you pushed thanks! | 19:55 |
M0weng[m] | Great! No problem :D | 19:56 |
clarkb | I believe that opendev/system-config has functioanl testing of that repo too if we do a depends on maybe | 19:57 |
clarkb | lodgeit has its test suite then opendev/system-config tests a full container based deployment of it and inserts a paste? | 19:57 |
clarkb | I can help set up testing based on that if the built in testing is happy | 19:57 |
opendevreview | Clark Boylan proposed openstack/project-config master: Disable docker hub mirror use in jobs https://review.opendev.org/c/openstack/project-config/+/935725 | 22:02 |
clarkb | M0weng[m]: ok your change looks good to me. The CI -1 is unrelated and something we're trying to deal with (yay docker hub rate limits) | 23:11 |
clarkb | we'll be able to do a bit more testing before deployment once that is addressed but otherwise this looks great and thank you for the detailed commit message | 23:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!