| opendevreview | OpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/962557 | 02:13 |
|---|---|---|
| mnasiadka | Morning, finally rocky10 is passing in https://review.opendev.org/c/opendev/zuul-providers/+/962564 (hopefully it’s not just that lucky run) | 05:12 |
| fungi | congrats mnasiadka! i know that was a lot of work | 12:55 |
| mnasiadka | Well, I’ve done majority of zuul-providers work, added Rocky 10, so now Alma was not that complicated ;) | 12:59 |
| mnasiadka | Now I can go post a devstack job when that lands | 12:59 |
| opendevreview | Merged openstack/diskimage-builder master: Remove testing for f37 https://review.opendev.org/c/openstack/diskimage-builder/+/952954 | 13:36 |
| opendevreview | Piotr Parczewski proposed openstack/diskimage-builder master: Fix GRUB arm64 package names https://review.opendev.org/c/openstack/diskimage-builder/+/962793 | 13:37 |
| opendevreview | Piotr Parczewski proposed openstack/diskimage-builder master: Fix GRUB arm64 package names https://review.opendev.org/c/openstack/diskimage-builder/+/962793 | 13:39 |
| opendevreview | Jan Gutter proposed zuul/zuul-jobs master: Update ensure-helm role to add more functionality https://review.opendev.org/c/zuul/zuul-jobs/+/962794 | 13:44 |
| opendevreview | Jan Gutter proposed zuul/zuul-jobs master: Update ensure-helm role to add more functionality https://review.opendev.org/c/zuul/zuul-jobs/+/962794 | 13:46 |
| corvus | clarkb: fungi if it helps (probably not in this case, but just fyi) zuul uses a sphinx config setting to generate canonical link urls in the html | 14:00 |
| corvus | https://opendev.org/zuul/zuul/src/branch/master/doc/source/conf.py#L73-L74 | 14:00 |
| fungi | thanks, that's an excellent example | 14:01 |
| fungi | but yeah, not directly portable to the gitea backends vs proxied name discrepancy. i think we'll just set it in the apache vhost config | 14:01 |
| corvus | that was added mostly to tell search engines to ignore all the old versions of docs and focus on the latest | 14:01 |
| corvus | for gitea, i think the header is the way to go | 14:02 |
| corvus | (because gitea is in control of the canonical location there, it's generating the content (and apache is the next best thing)) | 14:02 |
| opendevreview | Jan Gutter proposed zuul/zuul-jobs master: Update ensure-helm role to add more functionality https://review.opendev.org/c/zuul/zuul-jobs/+/962794 | 14:21 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set canonical Link paths for gitea resources https://review.opendev.org/c/opendev/system-config/+/962826 | 14:59 |
| clarkb | infra-root ^ that is almost certainly incomplete and buggy but I figured we can start there as our testinfra test cases give us some decent feedback | 15:00 |
| clarkb | it isn't clear to me for example if the query string should be included (I think so?) and if so does teh query string variable include the ? prefix like the request_uri includes the root / | 15:01 |
| fungi | popping out for lunch, should be back around 16:30 | 15:19 |
| clarkb | rax-dfw continues to look good | 15:25 |
| clarkb | one concern I have is that s we use it more it may slowly snowball itno sadness but there is no evidence of that yet | 15:26 |
| clarkb | does the opendev zuul installation no longer have streaming logs that work for anyone else? cc corvus | 15:30 |
| clarkb | I'm using an up to date firefox installation and there are characters emitted but not coherently in anything that resembles a stream. I know there was an update to the ansi stuff recently maybe that is related? | 15:31 |
| clarkb | the job-output.txt file looks fine so it must be in the processing on the web side? | 15:32 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set canonical Link paths for gitea resources https://review.opendev.org/c/opendev/system-config/+/962826 | 15:33 |
| melwitt | does anyone know if there is a way to avoid using a hardcoded node name from hostvars to get a subnode name? I tried this but it seems to not work https://review.opendev.org/c/openstack/nova/+/957477/16/roles/run-evacuate-hook/tasks/main.yaml my problem is that I am trying to use a nodeset that has not named the subnode 'compute1' | 15:47 |
| clarkb | melwitt: if you look in the resulting job's zuul-info dir that should include the ansible inventory used to run the job playbooks | 15:49 |
| clarkb | melwitt: I would start with that datastructure and see if any lookup path from there looks workable | 15:49 |
| melwitt | ohh ok | 15:49 |
| melwitt | thank you | 15:49 |
| clarkb | but generally I suspect the ansibly solution to problems like this is to rely on groups more than hostnames | 15:50 |
| clarkb | so you might have a compute group and then expect that ndoes that fill that role are in that group | 15:50 |
| melwitt | yeah that makes sense. that's basically what I want to do | 15:50 |
| clarkb | melwitt: https://zuul.opendev.org/t/openstack/build/065d156f3a0c4db7a39d7021072132d3/log/zuul-info/inventory.yaml it does look liek you have subnode and compute in there already so you can in theory lookup from there | 15:52 |
| clarkb | you may need to convert the dict keys into a list to 0 index them? | 15:52 |
| melwitt | yeah I'm pretty ansible ignorant so I didn't even know where to look heh | 15:53 |
| clarkb | melwitt: https://www.reddit.com/r/ansible/comments/ozopzi/list_hosts_in_group_and_filter/ | 15:55 |
| clarkb | I think the select(match) filter is replacing the indexing there. So ya something along those lines or using a filter to get keys as a list then indexing them that way | 15:56 |
| clarkb | melwitt: I find this page is often helpful https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_filters.html | 15:56 |
| melwitt | thanks this all helps a lot :) | 15:56 |
| clarkb | I do wish that page more prominently linked to https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_filters.html as well | 15:57 |
| corvus | clarkb: https://zuul.opendev.org/t/zuul/stream/9ea0e70a8fec43228404c209068dcfc5?logfile=console.log looks like a legit stream to me | 15:57 |
| clarkb | basically you can use any of those built in filters or the ansible provided filters | 15:57 |
| clarkb | corvus: huh it doesn't for me. Let me try another browser | 15:57 |
| clarkb | corvus: it works in chrome and chromium but not firefox | 15:58 |
| clarkb | let me try firefox without any extensions | 15:58 |
| corvus | i will restart my firefox and try another | 15:58 |
| clarkb | firefox incognito mode also does not work so I think it is specific to firefox | 15:58 |
| corvus | ff 143.0.1 and ZOMG i can not copy and paste the text "canonical-002 - 1.0" | 15:59 |
| corvus | i can copy/paste the upstream version number, but not the downstream one. nice. | 16:00 |
| clarkb | I'm on 143.0.1 but there is new update available so I'll update and retest | 16:00 |
| corvus | i have another one on 143.0.3 (same downstream) and it works too | 16:01 |
| clarkb | corvus: ok 143.0.3 works for me now | 16:01 |
| clarkb | I wonder if the pending update installation broke some shared library stuff and preventing rendering | 16:01 |
| clarkb | its weird because everything else seemed to work except for when I tried to stream a log out of that gitea change | 16:02 |
| corvus | oh maybe. i've seen things get weird when that happens. | 16:02 |
| clarkb | sorry for the noise, I suspect this was an issue with the pending update impacting the old installation | 16:02 |
| opendevreview | Merged opendev/zuul-providers master: Add Alma Linux 10 image build jobs https://review.opendev.org/c/opendev/zuul-providers/+/962564 | 16:06 |
| mnasiadka | Yay | 16:07 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set canonical Link paths for gitea resources https://review.opendev.org/c/opendev/system-config/+/962826 | 16:11 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set canonical Link paths for gitea resources https://review.opendev.org/c/opendev/system-config/+/962826 | 17:31 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set canonical Link paths for gitea resources https://review.opendev.org/c/opendev/system-config/+/962826 | 18:28 |
| clarkb | I missed that it looks like you need the e suffix on the variable substitution stuff there | 18:29 |
| clarkb | though still not sure those variables are considered environment variables... | 18:29 |
| clarkb | ok that last run confirms that those variables are null in that context. I found https://andreas.scherbaum.la/post/2024-12-26_add-canonical-headers-for-files-in-apache2/ which uses mod rewrite to define variables with this sort of information for every request | 19:48 |
| clarkb | do we think ^ that is reasonable cc fungi | 19:49 |
| corvus | clarkb: i give up, what's with the `e`s near query_string? | 20:06 |
| clarkb | corvus: that is apparently the syntax you use in the mod_headers Header directive values | 20:10 |
| clarkb | corvus: you can use %{foo}e to lookup env vars or %{bar}s to look up special ssl vars | 20:10 |
| corvus | oh i see it in the docs there now. i was looking on the general substitution page | 20:12 |
| clarkb | ya I tripped over this quite a bit too. Its iteresting that tehy use their own syntax | 20:12 |
| corvus | clarkb: looks like the test is failing because it's getting Link: <https://testgitea.opendev.org(null)(null)>; rel="canonical" | 20:14 |
| corvus | oh and that's your question: should we use mod_rewrite to fix that | 20:14 |
| corvus | i think that's reasonable | 20:15 |
| clarkb | ya I guess I wasn't sure if setting env var values for every request is something we might expect to have a big impact on server operations | 20:15 |
| clarkb | we're essentially shuffling the data from mod rewrite vars to generic env vars that mod headers can read | 20:16 |
| clarkb | I'll work on making that update shortly | 20:18 |
| clarkb | I'm finally getting over my writers block and have some content going into a reveal.js html file finally | 20:18 |
| corvus | i think that's fine, mod_rewrite is made for that | 20:21 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set canonical Link paths for gitea resources https://review.opendev.org/c/opendev/system-config/+/962826 | 21:25 |
| clarkb | 962826 passes now. Our testinfra tests do test some git operations in addition to normal http requests for browser traffic so in theory this isn't breaking anything | 22:22 |
| clarkb | but probably still work some careful consideration of what the potential impacts are here and whether or not we need more test cases | 22:22 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!