| opendevreview | Takashi Kajinami proposed openstack/watcher master: Align default_log_level to global default https://review.opendev.org/c/openstack/watcher/+/995367 | 03:00 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Add Playwright-based E2E testing framework https://review.opendev.org/c/openstack/watcher-dashboard/+/970353 | 07:51 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Add Playwright integration test for skip action workflow https://review.opendev.org/c/openstack/watcher-dashboard/+/976594 | 07:52 |
| dviroel | amoralej: chandankumar: what is missing to get https://review.opendev.org/c/openstack/watcher-dashboard/+/970353 merged? | 14:13 |
| dviroel | we already had some reviews there and I think that is pretty close to merge it | 14:14 |
| amoralej | dviroel, chandankumar my main concern was https://review.opendev.org/c/openstack/watcher-dashboard/+/970353/comment/e5aa2cfa_4ee66a1b/ which is fixed | 14:29 |
| dviroel | nive | 14:31 |
| dviroel | nice | 14:31 |
| dviroel | sean-k-mooney: around? do you have any concern on merging playwright patch? your last vote was -1, but there were some updates... | 14:32 |
| dviroel | going to hold it a bit just in case | 14:32 |
| dviroel | https://review.opendev.org/c/openstack/watcher-dashboard/+/970353 | 14:32 |
| amoralej | wrt `playwright install` , comments from chandankumar clarifies it a bit and i was wrong wrt creating a separate target, we may do it in extra_setup_commands instead of commands but that's just minor optimization | 14:33 |
| dviroel | ack, marked as resolved | 14:33 |
| dviroel | amoralej: lets see if sean has any concern before W+1 | 14:33 |
| dviroel | otherwise we can w+1 later today | 14:33 |
| sean-k-mooney | dviroel: sot of im in the cybrog irc meeting | 14:34 |
| sean-k-mooney | as long as the tox env is not installing things like a broser | 14:34 |
| sean-k-mooney | i think i was mostly ok with it | 14:34 |
| sean-k-mooney | ill check back after the meeting | 14:34 |
| dviroel | sure, forgot about the meeting | 14:34 |
| sean-k-mooney | no worries | 14:34 |
| dviroel | no worries, please take a look later then | 14:34 |
| chandankumar | thank you everyone for reviewing it. :-) | 14:46 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Cache node utilization to fix vm_workload_consolidation at scale https://review.opendev.org/c/openstack/watcher/+/992676 | 14:57 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add allocation-based capacity checks to vm_workload_consolidation https://review.opendev.org/c/openstack/watcher/+/993088 | 14:57 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add allocation-based capacity checks to vm_workload_consolidation https://review.opendev.org/c/openstack/watcher/+/993088 | 15:30 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Use built-in method to build RequestContext https://review.opendev.org/c/openstack/watcher/+/995382 | 17:08 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Fix handling of X-Show-Deleted header https://review.opendev.org/c/openstack/watcher/+/995548 | 17:08 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!