| *** jakeyip_ is now known as jakeyip | 00:20 | |
| mnasiadka | jakeyip: there will be no unmaintained/2023.2 - it has gone directly EOL since it's not a SLURP release - see https://review.opendev.org/c/openstack/releases/+/948580 | 07:01 |
|---|---|---|
| eolivare | fungi, o/ do you know anything about the outage of http://download.cirros-cloud.net/? IDK if it's part of openstack infra or not. But it is going to affect many opendev jobs | 08:11 |
| jakeyip | thanks mnasiadka | 08:11 |
| frickler | eolivare: https://download.cirros-cloud.net/ is not part of opendev, it is hosted on dreamhost and managed by a single person. you should not use it in CI directly, we have mirrored all (most?) relevant images into our CI images so they should be available directly in the local fs | 08:48 |
| frickler | also https://download.cirros-cloud.net/ is working for me currently, maybe they just had some maintenance earlier? | 08:49 |
| eolivare | frickler, yes, it seems it's back. Thanks for the advice anyway | 08:49 |
| eolivare | frickler, it is used from many projects and jobs, including devstack-tempest (I'd say many jobs inherit from it) | 08:51 |
| eolivare | https://opendev.org/openstack/tempest/src/branch/master/zuul.d/base.yaml#L21 | 08:51 |
| frickler | eolivare: devstack should have some logic built in to not actually download those images, but use the provided ones. if that doesn't work, that's a bug. (and also that line should likely be updated to refer to the latest images) | 09:02 |
| eolivare | frickler, ack, thanks | 09:03 |
| fungi | jakeyip: unmaintained status doesn't apply to non-slurp releases: https://releases.openstack.org/ | 12:49 |
| fungi | jakeyip: second bullet at https://governance.openstack.org/tc/resolutions/20230724-unmaintained-branches.html#unmaintained-branches "Only SLURP releases are eligible for having an Unmaintained branch." | 12:51 |
| fungi | oh, and i see mnasiadka already answered ;) | 12:51 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Revert "Temporarily require Signed-Off-By in the sandbox" https://review.opendev.org/c/openstack/project-config/+/950997 | 13:40 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Replace OpenStack's CLA enforcement with the DCO https://review.opendev.org/c/openstack/project-config/+/950998 | 13:43 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Add missing ACL inheritance for openstack/grian-ui https://review.opendev.org/c/openstack/project-config/+/951010 | 15:03 |
| fungi | sean-k-mooney: ^ fyi | 15:03 |
| sean-k-mooney | acjj u jubd if assumned that was the default | 15:05 |
| sean-k-mooney | but that looks ok with me | 15:05 |
| clarkb | fungi: question on that | 15:06 |
| fungi | yeah, it's not default, gerrit doesn't have a way to do implicit inheritence based on repository name patterns | 15:06 |
| sean-k-mooney | i guess we need to be explcit because we have multiple orgs | 15:06 |
| fungi | clarkb: my upcoming patches may answer your qeuestion, whatever it is | 15:06 |
| clarkb | fungi: basically I think you can remove the require cla line in the same change since it will be inherited then you have one less cleanup later | 15:06 |
| fungi | i need to remove a bunch of duplicate entries from openstack project acls | 15:07 |
| fungi | like ~230 | 15:07 |
| fungi | so easier to do that in a batch change | 15:07 |
| clarkb | ack | 15:07 |
| fungi | this was the only non-sig repo in the openstack namespace which was missing the inheritence entry | 15:08 |
| fungi | it'll get a little messy since some openstack sig repos don't inherit from meta-config and some do but then override those values instead | 15:09 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Replace OpenStack's CLA enforcement with the DCO https://review.opendev.org/c/openstack/project-config/+/950998 | 16:21 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Deduplicate OpenStack ACL content https://review.opendev.org/c/openstack/project-config/+/951014 | 16:21 |
| fungi | clarkb: ^ first attempt at it | 16:23 |
| clarkb | fungi: have we checked that mergeContent = true is an inherited field/value? I know the dco/cla stuff is | 16:26 |
| fungi | good question. i had assumed everything in the acl could be inherited | 16:27 |
| clarkb | ya I think 99% of things are | 16:27 |
| clarkb | but probably worth double checking before we make a massive update like this | 16:27 |
| fungi | we have it in meta-config, at least | 16:27 |
| clarkb | https://gerrit-review.googlesource.com/Documentation/config-project-config.html the default is INHERIT so ya should be fine | 16:28 |
| fungi | oh good, i wasn't sure where to start investigating that | 16:28 |
| clarkb | that page can also be used to check requireChangeId (which also defaults to INHERIT) | 16:30 |
| sean-k-mooney | you can see it in gerrit too i think | 16:57 |
| sean-k-mooney | on a per proejct basis | 16:57 |
| sean-k-mooney | so like here https://review.opendev.org/admin/repos/openstack/nova,general | 16:57 |
| sean-k-mooney | you can at lest see if it is inerited and the value | 16:58 |
| clarkb | in this case I don't think you'd see that because all these projects are explcitiyl setting it. But if you found one that isn't you'd be able to confirm | 17:00 |
| clarkb | but the docs have us covered. I'm not worried just wanted to double check | 17:01 |
| sean-k-mooney | which setting was it merge-content | 17:23 |
| sean-k-mooney | ya i thinik most have that hardcoded | 17:24 |
| sean-k-mooney | its in https://docs.opendev.org/opendev/infra-manual/latest/creators.html#minimal-acl-file | 17:24 |
| sean-k-mooney | so which does nto use ineritance | 17:24 |
| sean-k-mooney | so if they followed that guide it will be set | 17:24 |
| fungi | well, that's general guidance for projects in opendev, not guidance for openstack projects | 17:28 |
| fungi | https://docs.openstack.org/project-team-guide/repository.hcould stand to have a section on project creation that spells out openstack-specific project requirements | 17:30 |
| fungi | things like the meta-config inheritance | 17:30 |
| sean-k-mooney | i see well given the two docs dont cross link it make discoverablity of any openstack extra hard | 17:40 |
| fungi | they cross-link insofar as the project team guide refers to the infra manual for things like retirement steps | 17:41 |
| sean-k-mooney | right but this https://docs.openstack.org/project-team-guide/preface.html is the entirty of the project creation docs | 17:41 |
| sean-k-mooney | it just says follow the opendev one | 17:42 |
| fungi | and i think infra manual links to openstack governance and project team guide references at the begining of the document | 17:42 |
| fungi | https://docs.opendev.org/opendev/infra-manual/latest/creators.html#decide-status-and-namespace-of-your-project | 17:42 |
| sean-k-mooney | it does not callthe linke you pased above did not | 17:42 |
| sean-k-mooney | .. | 17:43 |
| sean-k-mooney | the link you pased above is a 404 by the way https://docs.openstack.org/project-team-guide/repository.hcould | 17:43 |
| sean-k-mooney | i assume you were referint to a subheading in https://docs.openstack.org/project-team-guide/repository | 17:43 |
| fungi | my ssh session is struggling with weak wifi in my office at the moment, so edits didn't resolve the way i intended | 17:44 |
| sean-k-mooney | that just has removal and retirement currently but it could have creation too | 17:44 |
| sean-k-mooney | fungi: no worries | 17:44 |
| fungi | https://docs.openstack.org/project-team-guide/repository.html is what i was trying to link earlier | 17:45 |
| fungi | i was saying that document has sections on renaming and retiring but no section on creation, and probably should | 17:45 |
| sean-k-mooney | ah yes i agree that would be good | 18:05 |
| sean-k-mooney | probaly move https://docs.openstack.org/project-team-guide/preface.html to that page as a sub section | 18:05 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!