*** bhagyashris_ is now known as bhagyashris | 11:09 | |
pdeore | abhishekk, croelandt, dansmith, jokke_ , mrjoshi, rosmaita, glance weekly meeting in 10 minutes at #openstack-meeting | 13:50 |
---|---|---|
dansmith | I have a meeting conflict for this spot today, unfortunately | 13:50 |
rosmaita | me too | 13:50 |
abhishekk | me too! | 13:50 |
rosmaita | i thought we had cancelled this week's meeting? | 13:51 |
pdeore | ohh np, :) I just have PTG schedule updates for today | 13:51 |
pdeore | dansmith, abhishekk, rosmaita, should we cancel it ? | 13:52 |
abhishekk | no, you can just start the meeting | 13:52 |
rosmaita | works for me | 13:52 |
abhishekk | share updates and end it | 13:52 |
rosmaita | oh, what abhishekk said | 13:52 |
pdeore | ack | 13:52 |
*** pdeore is now known as pdeore|afk | 14:33 | |
opendevreview | Mridula Joshi proposed openstack/glance-specs master: Expanding stores-detail for other stores https://review.opendev.org/c/openstack/glance-specs/+/835606 | 14:56 |
opendevreview | Durga Malleswari Varanasi proposed openstack/glance master: Internal server error if shared member tries to stage data to image https://review.opendev.org/c/openstack/glance/+/834701 | 15:22 |
*** pdeore|afk is now known as pdeore | 16:31 | |
pdeore | dansmith, I just got to know that you were asking about the new boolean logic I added in my new patch sets of glance-tempest-plugin metadef rbac patches | 16:36 |
pdeore | https://review.opendev.org/c/openstack/glance-tempest-plugin/+/802794/12..14/glance_tempest_plugin/tests/rbac/v2/metadefs/test_properties.py#224 | 16:36 |
pdeore | So, on previous patch set I got a suggestion from Lance about removing the NOT from the boolean to make it easier to read | 16:37 |
dansmith | pdeore: okay, I see where lance suggested that on your link above, but not sure where I said something | 17:08 |
dansmith | but obviously doing "not foo == bar" is less readable than "foo != bar" so I definitely agree with lance's comment | 17:08 |
dansmith | did I say something contrary? | 17:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!