| opendevreview | Merged openstack/glance master: Replace deprecated assertDictContainsSubset https://review.opendev.org/c/openstack/glance/+/802411 | 06:20 |
|---|---|---|
| opendevreview | Pranali Deore proposed openstack/glance-tempest-plugin master: Add protection testing for metadef namespaces https://review.opendev.org/c/openstack/glance-tempest-plugin/+/800902 | 12:51 |
| opendevreview | Pranali Deore proposed openstack/glance-tempest-plugin master: Implement API protection testing for metadef objects https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802793 | 12:51 |
| abhishekk | jokke_, rosmaita, dansmith glance weekly meeting in 10 minutes at #openstack- meeting | 13:49 |
| abhishekk | jokke_, added one comment on stable patch, https://review.opendev.org/c/openstack/glance/+/800102 | 14:23 |
| jokke_ | abhishekk: IIRC there was merge conflicts on the one you used as example preventing the automatic 3 way merge whichmade it needed to refer to the previous branch rather than just master | 14:34 |
| jokke_ | the stable branch policy literally instructs to use master | 14:35 |
| abhishekk | jokke_, ack, yeah make sense | 14:35 |
| abhishekk | it just tells that if you want to backport n-2 then it should be present in master as well as n-1 | 14:35 |
| abhishekk | so no worries | 14:36 |
| jokke_ | yes, but the cherrypick -x (and the ref there in the commit message) does not establish any depends on chain in gerrit | 14:36 |
| jokke_ | hmm-m that would be handy though and suddenly start making sense :D | 14:37 |
| jokke_ | Need to go and propose that | 14:37 |
| abhishekk | ack | 14:37 |
| jokke_ | it would be super handy if the cherry-picked-from could be treated as depends on and would not merge before all the references are merged | 14:38 |
| abhishekk | ++ | 14:38 |
| dansmith | that's a very interesting idea :) | 14:38 |
| jokke_ | abhishekk: but in general you refer to the previous branch only if you had merge conflict to solve so you don't need to solve it in ever branch below | 14:40 |
| jokke_ | but yean, would be handy way to help stable revies | 14:41 |
| abhishekk | right, I remember sometimes back rosmaita also explained this | 14:42 |
| jokke_ | I think in those doc patches it did not change the visible review at all | 14:42 |
| * abhishekk brb | 14:44 | |
| opendevreview | Merged openstack/glance-specs master: Add Yoga directory for specs https://review.opendev.org/c/openstack/glance-specs/+/807197 | 14:54 |
| * abhishekk back | 14:58 | |
| opendevreview | Merged openstack/glance stable/wallaby: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803618 | 15:24 |
| smcginnis | jokke_: Saw your comment about stable backports. Will try to take a look through today. | 15:41 |
| *** abhishekk is now known as abhishekk|afk | 15:45 | |
| jokke_ | smcginnis: cheers | 15:49 |
| opendevreview | Merged openstack/glance stable/victoria: Remove SSL configuration section from docs https://review.opendev.org/c/openstack/glance/+/803619 | 16:25 |
| *** akekane__ is now known as abhishekk | 17:39 | |
| * abhishekk signing out for the day | 19:03 | |
| opendevreview | Merged openstack/glance_store stable/victoria: Pass multipath config while creating connector object https://review.opendev.org/c/openstack/glance_store/+/786789 | 21:30 |
| opendevreview | Merged openstack/glance stable/victoria: Revert "Remove all usage of keystoneclient" https://review.opendev.org/c/openstack/glance/+/800102 | 23:03 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!