| *** ysandeep|out is now known as ysandeep | 05:35 | |
| frickler | any ansible cores around? this seems to affect at least kolla and tripleo https://github.com/ansible-community/ansible-lint/pull/1797 | 06:57 |
|---|---|---|
| opendevreview | chandan kumar proposed zuul/zuul-jobs master: Introduce iptables_package local var https://review.opendev.org/c/zuul/zuul-jobs/+/822503 | 06:58 |
| *** ysandeep is now known as ysandeep|afk | 07:05 | |
| frickler | zuul-jobs linting is also broken by that https://zuul.opendev.org/t/zuul/build/12bb93d8586c4cd2a34a816c50b8d4b2 | 08:13 |
| *** ysandeep|afk is now known as ysandeep | 08:36 | |
| *** jpena|off is now known as jpena | 08:51 | |
| *** gibi_pto_back_on_10th is now known as gibi | 08:59 | |
| dtantsur | frickler: IIRC cirros did not have a real root partition for us to operate on. I'm looking at cirros-0.5.1-x86_64-rootfs.img now, and it does seem to have kernel/initramfs inside, as well as some grub files, but not grub-install or really any executables. | 10:23 |
| dtantsur | frickler: and the UEC image has a completely empty root partition | 10:25 |
| dtantsur | (it even has "blank" in its name) | 10:25 |
| dtantsur | frickler: for the context: Ironic needs to test (in the CI) writing a partition (= having only the root partition) image to the target disk and installing grub. | 10:39 |
| dtantsur | currently we only have one job that achieves that via a hand-crafted centos image. which is, at best, suboptimal. | 10:39 |
| frickler | dtantsur: hmm. we could publish the rootfs for cirros, but that doesn't include grub. the grub install is only being executed external to cirros during the bundling. do you need grub itself within the image? | 10:57 |
| dtantsur | frickler: yes | 10:57 |
| dtantsur | otherwise we cannot test installing the bootloader | 10:58 |
| *** ysandeep is now known as ysandeep|coffee | 11:08 | |
| *** rlandy|out is now known as rlandy|ruck | 11:16 | |
| *** dviroel|afk is now known as dviroel | 11:22 | |
| *** ysandeep|coffee is now known as ysandeep | 11:54 | |
| frickler | dtantsur: did a bit of testing, to build grub, buildroot needs wchar enabled, which increases the whole image size by more than 50%. so I don't think that this is feasible for a generic cirros image. one might make that a variant, but then we'd likely have to build it ourselves. | 12:19 |
| frickler | dtantsur: can you point me to how you are running these tests? I might want to try locally with some self-built cirros image | 12:19 |
| dtantsur | not exactly trivial, you'll need a proper ironic devstack environment.. lemme see if I can find a suitable job | 12:20 |
| dtantsur | frickler: so, basically https://opendev.org/openstack/ironic/src/branch/master/zuul.d/ironic-jobs.yaml#L349-L356 but removing IRONIC_DEFAULT_BOOT_OPTION or setting it to "local" | 12:21 |
| dtantsur | frickler: local.conf from it https://zuul.opendev.org/t/openstack/build/9cf357657eb344e6b698faa3fc9b5118/log/controller/logs/local_conf.txt | 12:23 |
| dtantsur | (again, IRONIC_DEFAULT_BOOT_OPTION=netboot is our workaround for not having local boot facilities, we're trying to remove that) | 12:24 |
| frickler | dtantsur: o.k., so you don't actually use this centos image you mentioned in the CI currently? or did I misunderstand that? anyway I think I can set something up with those hints, thx | 12:28 |
| dtantsur | ah, that, hold on, it's another job | 12:50 |
| dtantsur | frickler: https://opendev.org/openstack/metalsmith/src/branch/master/.zuul.yaml#L119-L145 using https://opendev.org/openstack/metalsmith/src/branch/master/playbooks/integration/centos8-image.yaml | 12:51 |
| *** pojadhav is now known as pojadhav|dr_appt | 13:17 | |
| fungi | frickler: it looks like the error you linked is consistent with what pr #1797 was fixing. do you know whether the error is continuing to surface? apparently we either need to use ansible_lint>=5.3.2 or pin rich<11 according to https://github.com/ansible-community/ansible-lint/issues/1795 | 13:26 |
| frickler | fungi: 5.3.2 was tagged only just now, it solves the issue in my local test, will drop some rechecks now | 13:41 |
| fungi | awesome | 13:41 |
| chandankumar | hello infra, whom can be bug to get https://zuul.opendev.org/t/openstack/builds?job_name=dib-functests&project=openstack/diskimage-builder this job fixed? | 13:44 |
| frickler | chandankumar: not sure if anyone is still interested in keeping opensuse builds working, possibly they should be disabled/made non-voting until that happens | 13:48 |
| fungi | agreed, it looks like the opensuse-specific job is already failing and set nonvoting, leading me to suspect support for it has been abandoned | 13:50 |
| fungi | i guess it began breaking the functest job around the start of the year, judging from the build history | 13:51 |
| chandankumar | thanks fungi frickler, proposing a patch to make it nv | 13:51 |
| fungi | chandankumar: dib-functests shouldn't be nonvoting, it should probably just stop testing the opensuste element | 13:51 |
| chandankumar | sure, will do that | 13:52 |
| fungi | and the readme for that element should probably be adjusted at the same time to say it's no longer tested and suspected to be broken, if we don't remove the element outright | 13:53 |
| dtantsur | would that lead to eventually removing opensuse nodes from the CI? | 14:05 |
| fungi | likely, yes. i expect we're already unable to update them | 14:06 |
| dtantsur | okay, we've deprecated it in bifrost already (because of python 3.6 mostly) | 14:07 |
| fungi | nevermind, looks like we're still able to build opensuse-15 images in nodepool as of 10 hours ago | 14:08 |
| fungi | however we don't have any booted | 14:08 |
| fungi | not sure if that's because they're failing to boot or because nobody's got any jobs currently running for it | 14:08 |
| fungi | nodepool logs indicate we're booting them and jobs are using them still, so i guess not completely broken yet | 14:10 |
| zigo | Hi there! One quick question: is it that it used to be that the refstack list contained test_name[id-2394872(smoke)] and now it's test_name[id-2394872,smoke] instead or something ? | 14:11 |
| zigo | I', seeing inconsistencies that leads to not found tests I believe... | 14:12 |
| fungi | zigo: you might have more luck asking in #refstack or #openstack-qa | 14:31 |
| zigo | Thanks, trying. | 14:32 |
| dtantsur | https://review.opendev.org/c/openstack/bifrost/+/819058/ ran a suse job, which did succeed | 14:32 |
| fungi | zigo: hopefully the tempest maintainers in #openstack-qa know how to identify test moves, but the refstack maintainers may also already be aware of that one | 14:32 |
| fungi | dtantsur: thanks, so yes i guess it's still working, maybe some corner case has broken impacting how dib's functtest exercises the opensuse element | 14:33 |
| zigo | I had a hard time figuring out that refstack insist soooo much on using venv, it was very annoying, but got that part tackled out, and now tests are running! :) | 14:33 |
| fungi | excellent | 14:35 |
| *** rlandy|ruck is now known as rlandy|ruck|brb | 14:36 | |
| chandankumar | fungi: Do We also need to remove opensuse/opensuse15-build-succeed funtest? | 14:36 |
| fungi | chandankumar: i'm not sure, hopefully the dib maintainers in #openstack-dib can provide better guidance on what they'll accept | 14:37 |
| opendevreview | chandan kumar proposed openstack/diskimage-builder master: Remove opensuse/build-succeeds funtests https://review.opendev.org/c/openstack/diskimage-builder/+/824002 | 14:40 |
| *** ykarel is now known as ykarel|away | 14:56 | |
| *** rlandy|ruck|brb is now known as rlandy|ruck | 14:59 | |
| *** dviroel is now known as dviroel|lunch | 15:02 | |
| *** dviroel|lunch is now known as dviroel | 16:07 | |
| opendevreview | chandan kumar proposed openstack/diskimage-builder master: Remove opensuse related funtests https://review.opendev.org/c/openstack/diskimage-builder/+/824002 | 16:16 |
| *** ysandeep is now known as ysandeep|out | 16:17 | |
| *** Guest2 is now known as prometheanfire | 16:35 | |
| clarkb | infra-root if you have time for https://review.opendev.org/c/opendev/system-config/+/823027 I think a change like that is a good one to start getting back into the swing of things in the new yaer (upgrade gitea to catch up on some bug fix relaeses) | 16:51 |
| clarkb | dtantsur: fungi: we've already committed to removing opensuse images baecuse no one has stepped into keep them | 16:51 |
| clarkb | (we said this almost a month ago now) | 16:51 |
| dtantsur | ack | 16:51 |
| fungi | oh, right, i forgot we announced that at the end of the year | 16:52 |
| clarkb | centos 8, opensuse 15 and tumbleweed, and gentoo were all on the chopping block. We got a volunteer for gentoo so it will stick around. centos 8 is EOL so not much we can do and the o[pensuse images just aren't cared for | 16:52 |
| clarkb | I run opensuse at home and would love to keep them going but I just don't have the time and it isn't that important for my home usage | 16:52 |
| clarkb | Removing tumblweed was on my "Hope to get that done this week" for this week list | 16:53 |
| fungi | yeah, loci folks noticed last week that something spontaneously broke in their centos-8 jobs, and concluded they should switch to stream | 16:53 |
| clarkb | I've still got some paperwork to do, but that is starting to get cleared off my desk so I can hopefully give time to things like the gitea upgrade above. There is also a gerrit project rename plugin that I'd like to add to our gerrit images and then test that it works with our CI stuff that already does renames | 16:56 |
| clarkb | (my major concern with that lpugin is it does a lot of stuff and does it online and I worry it might be very slow to rename individual projects) | 16:57 |
| fungi | i wonder if there would be a possible replication race condition between renaming a project in gerrit live and renaming in gitea | 17:17 |
| clarkb | I think there is | 17:18 |
| clarkb | But if we rename and then trigger replication we should be ok | 17:19 |
| fungi | as long as nothing updates the repo in between those steps | 17:19 |
| fungi | (including something as simple as someone pushing a revision of a change in review, since we replicate those to gitea as well) | 17:20 |
| opendevreview | Clark Boylan proposed openstack/project-config master: Add gerrit rename-project plugin to Zuul https://review.opendev.org/c/openstack/project-config/+/824038 | 17:21 |
| clarkb | fungi: right we have to replicate after we rename both gerrit and gitea, then we should be fine | 17:21 |
| clarkb | (and I'm not sure the order between gerrit and gitea matter much if you do the explicit replication at the end | 17:21 |
| fungi | my main worry is that if we rename in gitea first and then in gerrit but a change updates gerrit in between those, then we end up recreating the old repo on the gitea side, but maybe instead it just errors because gitea needs an explicit create step anyway? | 17:24 |
| clarkb | no, gerrit cannot create repos in gitea | 17:26 |
| clarkb | ya the push will be attempted and fail | 17:27 |
| fungi | seems fine in that case | 17:28 |
| clarkb | all of this should be tested and I'm not sure we should bother with the next rename since people want that soon | 17:31 |
| clarkb | But Ifigure we can start looking at it for the future | 17:32 |
| clarkb | and our CI can help a lot :) | 17:32 |
| fungi | yeah, i agree it's a great idea | 17:33 |
| fungi | does the new rename plugin actually move review history? acls? | 17:33 |
| clarkb | acls are in the repo but yes it is my understand it does what we do but also triggers reindexing internally while online. It may even updates project watches for users (not sure about that) | 17:34 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Better organize gerrit plugins in job defs https://review.opendev.org/c/opendev/system-config/+/824039 | 17:34 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Use rename-project plugin to rename projects https://review.opendev.org/c/opendev/system-config/+/824040 | 17:34 |
| clarkb | worst case we find it doesnt' work for us and we can keep doing what we do | 17:34 |
| frickler | zuul-jobs linter is still failing because ansible-lint is capped by https://review.opendev.org/c/zuul/zuul-jobs/+/773245. guess we would need to cap rich for now, unless there will be stable backports of the fix. zbr any idea on that? | 17:36 |
| fungi | oh, right, changes and acls are all trivially moved thanks to everything ending up in-repo with notedb | 17:37 |
| fungi | acls already were in fact | 17:37 |
| clarkb | frickler: the capping isn't complete I guess so some new library causes us to break? | 17:38 |
| frickler | clarkb: yes, some obscure dependency (rich) got an incompatible update | 17:40 |
| fungi | right, we'd have to add rich<11 to the test-requires | 17:41 |
| *** jpena is now known as jpena|off | 17:53 | |
| *** odyssey4me is now known as Guest135 | 17:59 | |
| clarkb | infra-root I've started putting our meeting agenda together. I trimmed out some stuff and added project renames to the list. Anything else to add today before I send that out? | 18:44 |
| clarkb | also infra-root any reason I should go ahead and attempt the refstack bullseye image update: https://review.opendev.org/c/opendev/system-config/+/821335 ? | 18:46 |
| clarkb | *any reason I should NOT go ahead? | 18:46 |
| fungi | clarkb: did they start testing with 3.9? | 19:06 |
| clarkb | not that I see. Last change was in november | 19:08 |
| clarkb | which is older than my image update | 19:08 |
| fungi | i don't object to trying anyway, it's _probably_ fine? | 19:09 |
| clarkb | and if it isn't will anyone notice? :P | 19:10 |
| clarkb | I guess I can ask kopecmartin to sign off on it first. kopecmartin can you review https://review.opendev.org/c/opendev/system-config/+/821335 and let us know if you have any objections to that image update for the refstack deployment? | 19:10 |
| clarkb | frickler: good catch | 19:19 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update refstack image to bullseye https://review.opendev.org/c/opendev/system-config/+/821335 | 19:20 |
| clarkb | ok with that largely out of the way I'm ready to look at the ansible lint rich thing. Do we have fixes yet or should I look into those? | 19:21 |
| fungi | i think it's just zuul-jobs where ansible-lint is pinned? | 19:22 |
| fungi | latest ansible-lint release already includes a fix | 19:23 |
| clarkb | openstack/project-config is similarly broken | 19:24 |
| fungi | ahh | 19:24 |
| clarkb | I'll try fixing openstack/project-config really quickly | 19:24 |
| fungi | but yeah, anywhere we pin ansible-lint and not rich we'll probably see the same error | 19:25 |
| *** artom__ is now known as artom | 19:25 | |
| opendevreview | Clark Boylan proposed openstack/project-config master: Add gerrit rename-project plugin to Zuul https://review.opendev.org/c/openstack/project-config/+/824038 | 19:30 |
| opendevreview | Clark Boylan proposed openstack/project-config master: Update ansible-lint to fix the rich dep https://review.opendev.org/c/openstack/project-config/+/824051 | 19:30 |
| clarkb | Lets see if that works | 19:30 |
| clarkb | then I can look at zuul-jobs next | 19:30 |
| opendevreview | Clark Boylan proposed opendev/base-jobs master: Update ansible-lint https://review.opendev.org/c/opendev/base-jobs/+/824052 | 19:31 |
| opendevreview | Clark Boylan proposed zuul/zuul-jobs master: Update ansible-lint pin to fix rich dep https://review.opendev.org/c/zuul/zuul-jobs/+/824057 | 19:45 |
| opendevreview | Clark Boylan proposed opendev/base-jobs master: Update ansible-lint https://review.opendev.org/c/opendev/base-jobs/+/824052 | 19:57 |
| clarkb | If ^ doesn't work I'll do what zuul-jobs apparently did and just bump ansible ahead for linting | 19:57 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Switch translate's IDP to OpenInfraID https://review.opendev.org/c/opendev/system-config/+/824059 | 20:07 |
| clarkb | all of those ansible-lint update changes are green now. reviews appreciated | 20:25 |
| fungi | all three in that topic lgtm, thanks! | 20:28 |
| clarkb | fungi: https://review.opendev.org/c/openstack/project-config/+/824051 is another. I'll update the topic on that one now | 20:29 |
| clarkb | last call on the infra meeting agenda. I'll get that setn out in the next 15 minutes or so | 20:53 |
| fungi | clarkb: i'm starting to suspect zanata doesn't store openids, and instead only checks the e-mail address they supply against its account table... do you happen to know? | 21:01 |
| clarkb | fungi: hrm I was pretty sure it did store them somewhere | 21:02 |
| fungi | i'm digging through its db and not finding any obvious openid urls in any of the tables, but i'll keep looking | 21:02 |
| clarkb | I want to say when I debugged the user who had login problems I found the openid urls. But I don't recall what table or column they were in | 21:02 |
| fungi | aha, found it! HCredentials | 21:04 |
| fungi | okay, so i can do an etl on this similar to the one i did for the lp transition | 21:04 |
| clarkb | and ya I do recall checking a lot of places :) | 21:04 |
| opendevreview | Merged openstack/diskimage-builder master: Remove opensuse related funtests https://review.opendev.org/c/openstack/diskimage-builder/+/824002 | 21:09 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Switch translate's IDP to OpenInfraID https://review.opendev.org/c/opendev/system-config/+/824059 | 21:21 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Monitor the HTTPS cert for id.openinfra.dev https://review.opendev.org/c/opendev/system-config/+/824063 | 21:21 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Switch translate-dev's IDP to OpenInfraID https://review.opendev.org/c/opendev/system-config/+/824064 | 21:21 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Switch refstack's IDP to OpenInfraID https://review.opendev.org/c/opendev/system-config/+/824065 | 21:21 |
| fungi | i'll mark the last two in the series wip for the time being | 21:21 |
| *** dviroel is now known as dviroel|out | 21:28 | |
| *** rlandy|ruck is now known as rlandy|dentist | 21:52 | |
| clarkb | using codesearch I think glean is the only project using opensuse-tumbleweed | 21:54 |
| clarkb | oh and that job isn't even running on tumblweed | 21:55 |
| clarkb | its a job that builds a tumblweed image and boots it with nodepool | 21:55 |
| opendevreview | Clark Boylan proposed opendev/base-jobs master: Remove tumbleweed nodeset https://review.opendev.org/c/opendev/base-jobs/+/824068 | 21:57 |
| clarkb | I think ^ is our first step to removing tumbleweed | 21:57 |
| opendevreview | Clark Boylan proposed openstack/project-config master: Set tumbleweed min-ready to 0 https://review.opendev.org/c/openstack/project-config/+/824070 | 22:03 |
| opendevreview | Clark Boylan proposed openstack/project-config master: Remove opensuse-tumbleweed from nodepool https://review.opendev.org/c/openstack/project-config/+/824071 | 22:03 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove opensuse tumbleweed from our mirrors https://review.opendev.org/c/opendev/system-config/+/824072 | 22:08 |
| clarkb | ok I think that is a complete set of changes to remove tumbleweed | 22:08 |
| fungi | lgtm, though the base-jobs change will need the ansible-lint fix, which i've single-core approved just now | 23:04 |
| clarkb | oh right. I got the order correct for the project-config changes but forgot about it on base-jobs | 23:05 |
| opendevreview | Merged opendev/base-jobs master: Update ansible-lint https://review.opendev.org/c/opendev/base-jobs/+/824052 | 23:12 |
| clarkb | https://review.opendev.org/c/opendev/base-jobs/+/824068 is green now. Thank you for the merge to fix that | 23:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!