| tkajinam | mharley[m], hi. are you around ? | 14:01 |
|---|---|---|
| tkajinam | or dmendiza[m] ? | 14:02 |
| tkajinam | I'm wondering if one of you can merge https://review.opendev.org/c/openstack/barbican/+/975349 and https://review.opendev.org/c/openstack/barbican/+/978322 asap. this has been blocking requirement bump for some time | 14:06 |
| tkajinam | and the bump will be forcefully merged with breaking barbican tests now, unless these are merged soon... https://review.opendev.org/c/openstack/requirements/+/977783 | 14:06 |
| mharley[m] | Hello, tkajinam. Sorry for the late reply. I'm planning to go through all of those patches tomorrow. I'm separating Fridays to look after all upstream work. | 14:09 |
| tkajinam | mharley[m], hmm the requirements patch may be merged soon which will break CI and we have to squash these two changes into one to unblock gate, which should be merged before any others | 14:32 |
| tkajinam | mharley[m], so if we can merge these two *now* that saves you and us, for sure. | 14:33 |
| tkajinam | mharley[m], nevermind. the requirements patch is already in gate queue so it's too late to merge these two separately | 14:36 |
| tkajinam | mharley[m], I'll squash these into one. please be aware that change should be merged first before you approve any other changes | 14:36 |
| opendevreview | Takashi Kajinami proposed openstack/barbican master: Pass valid vault_url https://review.opendev.org/c/openstack/barbican/+/978322 | 14:38 |
| opendevreview | Merged openstack/barbican master: Pass valid vault_url https://review.opendev.org/c/openstack/barbican/+/978322 | 18:28 |
| opendevreview | Merged openstack/barbican master: Remove url tags from README https://review.opendev.org/c/openstack/barbican/+/961801 | 18:28 |
| mharley[m] | Thanks, tkajinam. Tomorrow I’ll go through other lingering patches. | 18:32 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!