*** mhen_ is now known as mhen | 01:37 | |
*** dtantsur_ is now known as dtantsur | 01:57 | |
opendevreview | Markus Hentsch proposed openstack/tempest master: Add credential generation for manager personas https://review.opendev.org/c/openstack/tempest/+/924133 | 08:08 |
---|---|---|
mhen | gmann: thanks for reviewing, please check my response on the patchset | 08:25 |
mhen | I was unable to figure out what tests to implement for the preprov creds, sadly | 08:25 |
frickler | kopecmartin: gmann: please check elod's comment on https://review.opendev.org/c/openstack/releases/+/927833 | 12:33 |
kopecmartin | omg, are we really gonna spend so much time on a thing that shouldn't take longer than 10 minutes? | 12:48 |
gmann | frickler: kopecmartin checking | 16:55 |
gmann | mhen: sure, will check | 16:56 |
gmann | kopecmartin: I think we can do major version bump too 40.0.0 to get it released soon. | 17:03 |
gmann | and final release can be minor version bump (depends on what all changes that will have) | 17:03 |
gmann | kopecmartin: and that will only need version change in release patch and no other change is needed anywhere else | 17:05 |
gmann | mhen: replied. thanks for updates and adding test for dynamic creds but I would like to add pre-provisioned creds also which are always missed and untested | 17:27 |
gmann | mhen: I wrote test logic there and some example of existing tests which you can refer | 17:27 |
kopecmartin | gmann: ack, i changed the version to 40.0.0 .. signing off now, have a great weekend | 18:05 |
gmann | kopecmartin: you too, thanks | 18:06 |
opendevreview | Brian Haley proposed openstack/tempest master: Wait for instance ports to become ACTIVE https://review.opendev.org/c/openstack/tempest/+/928471 | 20:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!