| opendevreview | Dmitriy Chubinidze proposed openstack/openstack-manuals master: Update and add roles in Glossary https://review.opendev.org/c/openstack/openstack-manuals/+/947109 | 10:39 |
|---|---|---|
| opendevreview | Ivan Anfimov proposed openstack/openstack-manuals master: Update and add roles in Glossary https://review.opendev.org/c/openstack/openstack-manuals/+/947109 | 10:59 |
| opendevreview | Ivan Anfimov proposed openstack/openstack-manuals master: Update and add roles in Glossary https://review.opendev.org/c/openstack/openstack-manuals/+/947109 | 11:00 |
| opendevreview | Ivan Anfimov proposed openstack/openstack-manuals master: Update and add roles in Glossary https://review.opendev.org/c/openstack/openstack-manuals/+/947109 | 11:01 |
| *** gmaan_pto is now known as gmaan | 14:59 | |
| *** elodille1 is now known as elodilles | 15:31 | |
| noonedeadpunk | Can I ask for some reviews on https://review.opendev.org/c/openstack/governance/+/947622 ? As i'd really love to mark the repo as deprecated sooner then later | 17:17 |
| noonedeadpunk | Or, let me know if I'd need to submit some resolution along with it for exception for such late deprecation | 17:17 |
| gouthamr | noonedeadpunk: i missed this | 17:18 |
| noonedeadpunk | basically the caveat is that I want to deprecated it for Epoxy, as OSA has a trailing model, so Epoxy is not released yet | 17:20 |
| gouthamr | thanks i was going to ask | 17:20 |
| gouthamr | https://review.opendev.org/c/openstack/releases/+/947624 | 17:20 |
| noonedeadpunk | I'm not sure how "fine" is that, but repo is pretty much internal-only | 17:20 |
| noonedeadpunk | so from user prespective - there's no unexpected last-minute impact | 17:21 |
| gouthamr | i'm fine with merging the governance change if relmgmt folks were waiting on that to review | 17:22 |
| gouthamr | you've emptied the repo and made the project-config changes, so lgtm | 17:22 |
| opendevreview | Merged openstack/governance master: Deprecate openstack-ansible-tests repo https://review.opendev.org/c/openstack/governance/+/947622 | 17:26 |
| gouthamr | gmaan: o/ (when you see this) i see that you updated your nick/emails.. i was wondering if you can fix your gerrit emails, i think something still references "gmann<AT>ghanshyammann.com" | 17:31 |
| gouthamr | it's breaking the review script :) | 17:31 |
| gmaan | o/ | 17:37 |
| gmaan | yeah, I am fixing it now | 17:37 |
| gmaan | its updated in gerrit also, not sure where else missing | 17:38 |
| gouthamr | maybe the governance repo? | 17:39 |
| gmaan | I see | 17:40 |
| fungi | make sure you at least include it in one of the address fields in your openinfra foundation account profile as well | 17:43 |
| fungi | whatever preferred email you set in gerrit needs to be associated with your foundation profile in order for us to be able to match them up (it doesn't have to be in the primary address field in your profile, can be in the second or third and the lookups will still work) | 17:44 |
| gmaan | yeah, those are done | 17:45 |
| gouthamr | tc-members: looking to get this in this week, its had quite some soak time, and evolved in the positive direction | 17:58 |
| gouthamr | https://review.opendev.org/c/openstack/governance/+/944817 | 17:58 |
| gouthamr | please add your review/RC votes ^ | 17:59 |
| gouthamr | noonedeadpunk: hey, some late thought on https://review.opendev.org/c/openstack/governance/+/947622 - why is this deprecation, and not retirement? | 18:01 |
| gouthamr | noonedeadpunk: the repo changes you made suggest that we don't accept any bug fixes either, so i think you can move the repo to legacy.yaml instead of keeping it in project.yaml as "deprecated" | 18:02 |
| opendevreview | Goutham Pacha Ravi proposed openstack/governance master: Fix gmaan's email address https://review.opendev.org/c/openstack/governance/+/948342 | 18:12 |
| gouthamr | gmaan: ^ that was it | 18:12 |
| gmaan | gouthamr: yeah, thanks | 18:14 |
| opendevreview | Ghanshyam proposed openstack/governance master: Fix gmaan's email address https://review.opendev.org/c/openstack/governance/+/948342 | 18:16 |
| frickler | gmaan: gouthamr: ^^ please also update QA PTL data | 18:47 |
| gouthamr | ah! | 18:47 |
| opendevreview | Merged openstack/governance master: Fix gmaan's email address https://review.opendev.org/c/openstack/governance/+/948342 | 18:47 |
| gmaan | frickler: sure, I think that is why release PTL flag did not work | 18:47 |
| gouthamr | thanks, frickler .. gmaan: could do a replace all for gmann :) | 18:47 |
| gmaan | will do | 18:48 |
| jrosser | gouthamr: re openstack-ansible-tests doesn’t retirement break the stable branches that still rely on that repo? we made that mistake with accidentally retiring instead of deprecating some other OSA repos | 19:20 |
| gouthamr | jrosser: yes it would break older stable branches if they're relying on cloning from git.. we'd archive the repo if it were retired.. i didn't look at the stable branches for this tests repo .. deprecation sounds good in this case. thanks! | 19:33 |
| noonedeadpunk | gouthamr: we still have CI jobs on stable branches using this repo for testing | 19:34 |
| gouthamr | noonedeadpunk: ack, ty.. | 19:35 |
| noonedeadpunk | and we did quite some effort this cycle to re-implement CI in a different way fro remaining tests | 19:35 |
| noonedeadpunk | so I don't think we can backport that much to stable branches... | 19:35 |
| gouthamr | i see | 19:35 |
| gouthamr | makes sense | 19:35 |
| noonedeadpunk | gouthamr: pretty much the topic I'm talking about: https://review.opendev.org/q/topic:%22osa/molecule%22 | 19:41 |
| gouthamr | ++ | 19:42 |
| opendevreview | Ghanshyam proposed openstack/governance master: Updating Ghanshyam email/irc name https://review.opendev.org/c/openstack/governance/+/948354 | 19:56 |
| gmaan | gouthamr: frickler ^^ | 19:56 |
| frickler | noonedeadpunk: jrosser: also see https://zuul.opendev.org/t/openstack/config-errors?project=openstack%2Fopenstack-ansible-tests&skip=0 , how are you going to deal with that? | 20:01 |
| noonedeadpunk | frickler: I will check on that tomorrow morning.... | 20:02 |
| noonedeadpunk | But I don't know so far | 20:03 |
| noonedeadpunk | but as it deprecation -we still should be able to land changes to stable branches] | 20:03 |
| noonedeadpunk | so it should be resolvable | 20:03 |
| noonedeadpunk | I kinda wonder for how long these errors are there | 20:04 |
| fungi | yeah, that's the entire point of deprecating a repository vs retiring it | 20:14 |
| jrosser | getting the zed branch dealt with would likley deal with most of that | 21:32 |
| opendevreview | Ivan Anfimov proposed openstack/security-doc master: Fix for Operator (Enginner) https://review.opendev.org/c/openstack/security-doc/+/947198 | 22:09 |
| opendevreview | Ivan Anfimov proposed openstack/security-doc master: Fix for Operator (Enginner) https://review.opendev.org/c/openstack/security-doc/+/947198 | 22:09 |
| opendevreview | Ivan Anfimov proposed openstack/security-doc master: Fix for Operator (Enginner) https://review.opendev.org/c/openstack/security-doc/+/947198 | 22:09 |
| opendevreview | Ivan Anfimov proposed openstack/security-doc master: Remove tags from README https://review.opendev.org/c/openstack/security-doc/+/947198 | 22:10 |
| opendevreview | Merged openstack/security-doc master: Remove tags from README https://review.opendev.org/c/openstack/security-doc/+/947198 | 22:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!