| *** mhen_ is now known as mhen | 01:18 | |
| opendevreview | Merged openstack/devstack master: Avoid setting iso image in tempest config https://review.opendev.org/c/openstack/devstack/+/955970 | 06:51 |
|---|---|---|
| *** mhen_ is now known as mhen | 07:13 | |
| *** sfinucan is now known as stephenfin | 10:06 | |
| *** ykarel_ is now known as ykarel | 10:15 | |
| *** haleyb|out is now known as haleyb | 12:52 | |
| gmaan | sean-k-mooney: I replied to your comment on iso image test, did you check? https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/955950 | 15:23 |
| gmaan | if you have strong opinion on not doing volume attach/detach I can update but nothing harm to test the volume attachment also | 15:23 |
| sean-k-mooney | i would prefer ot have 2 tests | 15:24 |
| sean-k-mooney | one that did not need cinder and optionally a second that can have the volume | 15:24 |
| sean-k-mooney | that or make the volume part condtional on if cinder is deployed | 15:24 |
| sean-k-mooney | gmaan: for what its worth we do have a gap in cidner support | 15:25 |
| sean-k-mooney | today you can not set the device type when attachign a cinder volume | 15:25 |
| sean-k-mooney | we have some sepcial case code to set that for iso | 15:25 |
| sean-k-mooney | but that is what was broken | 15:25 |
| sean-k-mooney | gmaan: im ok with spliting this in a follow up if we want to proceed | 15:26 |
| gmaan | sean-k-mooney: I see I get your point now for cinder gap | 15:26 |
| sean-k-mooney | what i woudl like to see is 1 boot form iso with nova provisions storage, 2 boot form volume from an iso, 3 attahcign a volume coreated form an iso to any vm. | 15:26 |
| gmaan | I think let me update in this to do first one which is main goal of that bug | 15:27 |
| gmaan | writing separate tests is better than having cinder condition within test | 15:28 |
| sean-k-mooney | ya i perfer that also | 15:28 |
| sean-k-mooney | by the way we know volume detach in particar is sometime flaky in ci. so part of the reason i want the non cider version is to have a version that is very stable | 15:29 |
| sean-k-mooney | bfv also is stable | 15:29 |
| gmaan | yeah | 15:29 |
| sean-k-mooney | it just the 3rd one that can be flaky | 15:29 |
| sean-k-mooney | the third wone would need ot wait for SSHable right | 15:29 |
| sean-k-mooney | i didnt leave a comment about that | 15:30 |
| sean-k-mooney | https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/955950/3/whitebox_tempest_plugin/api/compute/test_iso_image.py#75 | 15:30 |
| sean-k-mooney | but your currently waiting for active before ding the atatch | 15:30 |
| sean-k-mooney | althouhg you have a couple of other operation in between so its probaly fine | 15:30 |
| gmaan | yeah, but that yes can be flaky if still server not ready | 15:31 |
| gmaan | it worked in few run on that change but we never know | 15:31 |
| sean-k-mooney | so, we have disucssed a few time baout starting to run a subset of whitebox tests in nova's gate | 15:31 |
| sean-k-mooney | i think this is oen that would be good to do | 15:32 |
| sean-k-mooney | but as we start doing that i woudl liek to be very tractical to make sure we keep thigns stable. | 15:32 |
| sean-k-mooney | having these split will allow use to add them incrementally or removign them incrementally | 15:33 |
| sean-k-mooney | anyway over all im hapy with the test i just would like to see the simipelr ones too | 15:33 |
| gmaan | sure, updating.. | 15:34 |
| opendevreview | Ghanshyam proposed openstack/whitebox-tempest-plugin master: Testing server boot from iso image https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/955950 | 15:43 |
| stephenfin | gmaan: frickler: Would I be able to get a review of https://review.opendev.org/c/openstack/devstack/+/955670 (and ideally the other two backports) when you get a chance? https://review.opendev.org/c/openstack/devstack/+/955670 | 17:32 |
| frickler | stephenfin: did you mean to paste the same link twice? | 17:40 |
| stephenfin | I did not, but I only meant to share one link | 17:41 |
| stephenfin | s/, but/:/ | 17:41 |
| frickler | ah, ok, I'll take a look after the TC meeting then | 17:43 |
| stephenfin | thanks | 17:46 |
| gmaan | stephenfin: ack, on your schema review, I will check these after that | 18:29 |
| opendevreview | Douglas Mendizábal proposed openstack/devstack master: Set password regex for security compliance https://review.opendev.org/c/openstack/devstack/+/956111 | 19:57 |
| opendevreview | Douglas Mendizábal proposed openstack/tempest master: Adding a password regex test for PCI DSS https://review.opendev.org/c/openstack/tempest/+/954029 | 20:01 |
| fungi | does anyone know what's still relying on the openinfra/python-tempestconf project? it's assigned to the defunct foundation interop working group, but tosky, mkopec, arxcruz and others seem to still be developing and reviewing changes on it | 22:51 |
| fungi | i'm working on winding down refstack and the other interop wg resources, but that stuck out as something that's not abandoned | 22:52 |
| *** adamcarthur56 is now known as adamcarthur5 | 23:51 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!