| kopecmartin | gmann: does the new comment answer your concern in https://review.opendev.org/c/openstack/tempest/+/891123 ? | 13:33 |
|---|---|---|
| opendevreview | Balazs Gibizer proposed openstack/whitebox-tempest-plugin master: Add barbican-tempest-plugin to requirements.txt https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/903680 | 14:57 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Add barbican-tempest-plugin to requirements.txt https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/903680 | 18:10 |
| opendevreview | melanie witt proposed openstack/devstack master: Add swap to default flavors https://review.opendev.org/c/openstack/devstack/+/903908 | 19:01 |
| melwitt | sean-k-mooney: ^ | 19:03 |
| sean-k-mooney | so those are not actully the falvor we use for tempest | 19:04 |
| sean-k-mooney | tempest creates its own | 19:05 |
| sean-k-mooney | here https://github.com/openstack/devstack/blob/master/lib/tempest#L294-L353 | 19:05 |
| sean-k-mooney | also i think we dont need to add that much swap say 64-128mb should be plenty | 19:06 |
| sean-k-mooney | its relly just https://github.com/openstack/devstack/blob/master/lib/tempest#L294-L306 that needs to be updated | 19:06 |
| sean-k-mooney | melwitt: those are the lines that creates the tempest flavor for ci^ | 19:07 |
| melwitt | sean-k-mooney: oh, oops. thanks | 19:08 |
| opendevreview | melanie witt proposed openstack/devstack master: Add swap to Tempest flavors https://review.opendev.org/c/openstack/devstack/+/903908 | 19:13 |
| gmann | kopecmartin: no, that condition is wrong to check and that is why test are skipped. replied there | 19:14 |
| opendevreview | melanie witt proposed openstack/devstack master: Add swap to Tempest flavors https://review.opendev.org/c/openstack/devstack/+/903908 | 23:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!