| *** mhen_ is now known as mhen | 02:48 | |
| opendevreview | Saurabh Agarwal proposed openstack/stackviz master: Fixed npm install errors, gulp dev errors and empty chart errors https://review.opendev.org/c/openstack/stackviz/+/973914 | 08:12 |
|---|---|---|
| abongale | gmaan: o/ | 09:20 |
| abongale | good morning qa o/ | 09:20 |
| *** haleyb|out is now known as haleyb | 13:59 | |
| JasonParoly[m] | gmaan: I just noticed that https://review.opendev.org/c/openstack/tempest/+/971327 merged with the wrong topic :-(. Is it ok or should I figure out how to change the topic? It looks like it would require another PR.... Have you faced this? Thanks. | 14:25 |
| opendevreview | Jason Paroly proposed openstack/tempest master: Add IPv6 native connectivity tests https://review.opendev.org/c/openstack/tempest/+/972411 | 14:37 |
| opendevreview | Jason Paroly proposed openstack/tempest master: Add IPv6 native connectivity tests https://review.opendev.org/c/openstack/tempest/+/972411 | 14:39 |
| clarkb | JasonParoly[m]: I can't speak for the qa team but as the person running the Gerrit I wouldn't worry about that too much. Topics are primarily useful before thinsg merge to help people organize their review work. If it merged anyway it is unlikely anyone will notice going forward | 15:58 |
| gmaan | JasonParoly[m]: yeah, as clarkb mentioned, topics are helpful before merge to track all changes for particular work. I do not think we can change topic after merger and we do not need to. do not worry about that. | 16:00 |
| *** tosky_ is now known as tosky | 21:51 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!