| *** iurygregory_ is now known as iurygregory | 11:38 | |
| opendevreview | Dan Smith proposed openstack/governance master: Add watcher DPL for Epoxy https://review.opendev.org/c/openstack/governance/+/933018 | 14:23 |
|---|---|---|
| cardoe | https://review.opendev.org/c/openstack/governance/+/922507 if any more tc members want to follow up | 16:49 |
| cardoe | rpittau: when ya get some time could you weigh in on https://review.opendev.org/c/openstack/sushy/+/932096 | 16:51 |
| cardoe | wrong channel. | 16:51 |
| dansmith | fungi: do you know if the liaison roles in the yaml for DPL are basically just free-form? | 17:24 |
| dansmith | I found no examples of the "project update/onboarding | 17:24 |
| dansmith | " one in the file anywhere, | 17:24 |
| dansmith | so I just made something up and no tests seem to fail so I assume that's good enough? | 17:24 |
| dansmith | gmann^ | 17:25 |
| gmann | dansmith: they are defined in schema here. its 'project_update_onboarding' because additionalProperties is false you change pass - https://github.com/openstack/governance/blob/e5640e3533f49678c646b75ffe402ffda99eb2fe/openstack_governance/projects_schema.yaml#L70-L89 | 17:54 |
| gmann | I think keeping additionalProperties false is ok as we do not need to be so strict on not adding new type of liaison if needed | 17:54 |
| dansmith | gmann: ah okay thanks | 17:54 |
| dansmith | yeah, i figured it just had to be human-readable | 17:55 |
| dansmith | but I'll fix my thing to match this | 17:55 |
| opendevreview | Dan Smith proposed openstack/governance master: Add watcher DPL for Epoxy https://review.opendev.org/c/openstack/governance/+/933018 | 17:55 |
| gmann | ++ | 17:55 |
| gmann | dansmith: you need to add tc-liaison also there. I need to make the 4 liaisons as mandatory field in schema so that we do not miss it | 17:57 |
| dansmith | gmann: okay I was not sure if that was required because it didn't seem to be | 17:57 |
| dansmith | bauzas: can I put you there? I assume gmann could also do it... | 17:58 |
| dansmith | (probably too late in the day for bauzas at this point) | 17:58 |
| gmann | yeah I am fine or both name also ok | 17:58 |
| dansmith | oh okay I just saw your comment.. I'll add you to lower the percentage of @redhat.com there if you're willing thanks | 17:59 |
| gmann | ++ | 17:59 |
| opendevreview | Dan Smith proposed openstack/governance master: Add watcher DPL for Epoxy https://review.opendev.org/c/openstack/governance/+/933018 | 17:59 |
| fungi | dansmith: yeah i think it's somewhat necessary that tooling can read it, for example release jobs check dpl release liaisons similarly to ptl entries in order to determine who needs to acknowledge a release request for a given deliverable | 18:05 |
| dansmith | ah okay, well, that one was easy :) | 18:07 |
| opendevreview | Dan Smith proposed openstack/governance master: Add watcher DPL for Epoxy https://review.opendev.org/c/openstack/governance/+/933018 | 18:17 |
| opendevreview | Merged openstack/governance master: Propose a pop-up team for eventlet-removal https://review.opendev.org/c/openstack/governance/+/931978 | 20:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!