| ianw | the fedora mirror should be released from kernel.org now | 00:03 |
|---|---|---|
| opendevreview | Ian Wienand proposed opendev/system-config master: mirror-update: clean up Fedora ignores https://review.opendev.org/c/opendev/system-config/+/837637 | 00:07 |
| ianw | hrw/fungi: ^ yeah the other thing about offical cloning is that we aren't really a full mirror, we just choose what we want | 00:08 |
| ianw | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_374/837637/1/check/system-config-run-mirror-update/37429f5/job-output.txt | 00:58 |
| ianw | testinfra is broken by this to :/ | 00:58 |
| ianw | this seems less straight forward. i'm unsure why this would be running as a different user | 01:14 |
| ianw | python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/home/zuul/src/opendev.org/opendev/system-config/setup.py | 01:16 |
| ianw | i guess tox is "installing" system-config? | 01:16 |
| ianw | ohhh, hrm, testinfra runs on bridge | 01:18 |
| ianw | i bet we copy the repos over into bridge as root... | 01:21 |
| *** rlandy|bbl is now known as rlandy|out | 01:23 | |
| fungi | sounds probable | 01:32 |
| ianw | hrm, maybe not https://zuul.opendev.org/t/openstack/build/37429f53db61499ab1d66a5b752807d7/console#0/4/8/bridge.openstack.org is just plain prepare-workspace-git | 02:00 |
| ianw | - hosts: bridge.openstack.org | 02:07 |
| ianw | become: true | 02:07 |
| ianw | i think that would do it. we're running it all as root now | 02:07 |
| opendevreview | Ian Wienand proposed opendev/system-config master: run-base: allow git into system-config https://review.opendev.org/c/opendev/system-config/+/837645 | 02:17 |
| opendevreview | Ian Wienand proposed opendev/system-config master: mirror-update: clean up Fedora ignores https://review.opendev.org/c/opendev/system-config/+/837637 | 03:05 |
| ianw | <tibbs> ianw: Sadly pubmirror1 is out of date for alt but up to date for everything else, because the repository got way too big. New disks are on the way. | 07:14 |
| ianw | <tibbs> pubmirror2 is up to date as far as I can tell. | 07:14 |
| ianw | a follow up on the mirror situation | 07:14 |
| ianw | fedora mirror situation | 07:14 |
| *** pojadhav is now known as pojadhav|lunch | 08:00 | |
| *** pojadhav|lunch is now known as pojadhav | 08:40 | |
| *** rlandy_ is now known as rlandy | 10:32 | |
| *** dviroel|out is now known as dviroel | 11:11 | |
| fungi | noted, thanks | 11:35 |
| *** soniya29 is now known as soniya29|out | 13:51 | |
| *** artom__ is now known as artom | 14:00 | |
| *** pojadhav is now known as pojadhav|afk | 14:06 | |
| opendevreview | yatin proposed zuul/zuul-jobs master: ensure-sphix: upgrade setuptools in venv https://review.opendev.org/c/zuul/zuul-jobs/+/837735 | 15:08 |
| ykarel | fungi, ianw clarkb when u get chance please see ^ | 15:20 |
| ykarel | needed for translation job failures in neutron | 15:21 |
| ykarel | https://zuul.openstack.org/builds?job_name=propose-translation-update&project=openstack%2Fneutron | 15:21 |
| *** dviroel is now known as dviroel|lunch | 15:28 | |
| clarkb | ykarel is gone apparnetly | 15:45 |
| clarkb | I'm not quite sure how the pip constraints conflict thing maps to needing file: support but having file: support doesn't seem problematic | 15:46 |
| clarkb | basically the change is fine, but may not address the issue unless I'm missing some key piece of info | 15:47 |
| fungi | clarkb: see my comment | 15:47 |
| clarkb | ah thank you. we need toget better at including that info :) | 15:47 |
| fungi | newer pip started using the new dep solver, causing it to not install the requested version of pyroute2 because it needed a newer setuptools | 15:48 |
| clarkb | ya 0.6.1 of pyroute2 or newer needs newer setuptools and we constrain to 0.6.9 | 15:48 |
| fungi | and yes, i would have liked to see it explained in the commit message | 15:48 |
| fungi | but rather than -1 i just added a comment with the results of my research there | 15:49 |
| clarkb | yup that works. I was just digging into it after my review and confused as to why we thought this would fix it. The change itself was fine and added functionality I had no problem with :) | 15:49 |
| fungi | right, the change meant we used a version of pip which performed more correct dependency solving (or any at all really). not a bug, but a feature | 15:50 |
| opendevreview | Merged zuul/zuul-jobs master: ensure-sphix: upgrade setuptools in venv https://review.opendev.org/c/zuul/zuul-jobs/+/837735 | 15:58 |
| *** marios is now known as marios|out | 16:03 | |
| *** dviroel|lunch is now known as dviroel | 16:35 | |
| clarkb | fungi: if you have a moment https://review.opendev.org/c/opendev/system-config/+/837645 should unstick opendev's CI | 17:06 |
| clarkb | fungi: note I think that only affects testinfra because we have to do installations of packages. I don't think it affects our production runs because ansible isn't really installing system-config just referring to it. And that playbook is zuul only I think | 17:08 |
| fungi | i'm confused by that change | 17:08 |
| clarkb | if we see that the production stuff is impacted too we can move it up a level and have it affect testing and prod but I suspect that isn't necessary. | 17:08 |
| clarkb | fungi: aiui its only the testinfra tox target that has the problem because we run that as root (it needs access to all the ansible stuff) | 17:09 |
| fungi | yet we're not using become on the git config call | 17:10 |
| clarkb | fungi: because it is already running as root | 17:10 |
| fungi | playbooks/zuul/run-base.yaml is run as root? | 17:10 |
| fungi | okay, that explains it | 17:10 |
| clarkb | yes, in system-config we connect as root to our systems. This is part of the "test like production" (but not quite heh) and so ansible is ssh'd back into itself as root there aiui | 17:11 |
| fungi | i wasn't sure what "This section" in the commit message referred to. i guess it should have said "this playbook" | 17:11 |
| clarkb | there is the top level zuul ansible that ssh's in to the test node called bridge as zuul and drives the test job. That drives a nested ansible which runs as root and sshs into bridge as root to perform those actions to mimic production | 17:11 |
| clarkb | oh ya there is a become: true at the top of that play | 17:12 |
| clarkb | so that makes it explicit | 17:12 |
| fungi | yep, i just got confused because i was trying to find where the section of the playbook that ran as root was | 17:12 |
| fungi | but it was all of the playbook | 17:12 |
| clarkb | yup | 17:12 |
| fungi | just me taking commit messages too literally again, i guess | 17:12 |
| fungi | did we rule out group ownership as a solution? | 17:13 |
| clarkb | we did | 17:13 |
| clarkb | the git patch onyl checks user ownership (and there are accusations on hacker news that tehy did it insufficiently too for extra fun) | 17:13 |
| fungi | git is equally displeased by the user being a member of the group owning the files? | 17:13 |
| clarkb | and i tested that on my held node by adding root to the stack group and then retrying | 17:13 |
| clarkb | yes | 17:14 |
| fungi | too bad | 17:14 |
| clarkb | if you want to test that you can check on my held node. root is already part of the stack group which should own the keystone repo which you can sudo pip install to see teh failure | 17:14 |
| clarkb | ianw's analysis ended up checking that groups are ignored on the source side too and both sides agreed. Group membership isn't evaluated and is ignored for this | 17:14 |
| opendevreview | Merged opendev/system-config master: run-base: allow git into system-config https://review.opendev.org/c/opendev/system-config/+/837645 | 17:27 |
| fungi | got it, thanks | 17:27 |
| opendevreview | Clark Boylan proposed opendev/jeepyb master: DNM want to run jeepyb testing against gerrit https://review.opendev.org/c/opendev/jeepyb/+/837753 | 17:51 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update our gitea test to handle new git behavior https://review.opendev.org/c/opendev/system-config/+/837761 | 20:08 |
| clarkb | fungi: ^ I think that will make gitea testing happy | 20:10 |
| fungi | thanks! | 20:10 |
| *** dviroel is now known as dviroel|afk | 20:22 | |
| *** arxcruz is now known as arxcruz|out | 20:28 | |
| *** timburke__ is now known as timburke | 20:55 | |
| clarkb | https://review.opendev.org/c/opendev/system-config/+/837761 seems to fix the gitea testing issue if infra-root have time to review it | 21:12 |
| clarkb | ianw: we've also found a couple issues in opendev land including ^ and the one your found | 21:20 |
| clarkb | s/your/you/ | 21:20 |
| *** dhill is now known as Guest1864 | 22:00 | |
| opendevreview | James E. Blair proposed zuul/zuul-jobs master: Add per-build WinRM cert generation https://review.opendev.org/c/zuul/zuul-jobs/+/837416 | 22:05 |
| opendevreview | Merged opendev/system-config master: Update our gitea test to handle new git behavior https://review.opendev.org/c/opendev/system-config/+/837761 | 22:27 |
| clarkb | https://review.opendev.org/c/opendev/system-config/+/827774 has been rechecked to hopefully flip its -1 to a +1 now that ^ has merged | 22:27 |
| opendevreview | Michael Johnson proposed openstack/diskimage-builder master: Mark our source trees as safe for git to use as other users https://review.opendev.org/c/openstack/diskimage-builder/+/837776 | 23:18 |
| opendevreview | Luciano Lo Giudice proposed openstack/project-config master: Add the cinder-three-par to Openstack charms https://review.opendev.org/c/openstack/project-config/+/837782 | 23:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!