| opendevreview | Merged openstack/tempest master: Fix tempest-slow-py3: use the correct inheritance chain https://review.opendev.org/c/openstack/tempest/+/800614 | 01:36 |
|---|---|---|
| *** abhishekk is now known as akekane|home | 04:44 | |
| *** akekane|home is now known as abhishekk | 04:44 | |
| *** bhagyashris_ is now known as bhagyashris|ruck | 06:25 | |
| opendevreview | Merged openstack/tempest master: Fix test_network_quota_exceeding test https://review.opendev.org/c/openstack/tempest/+/797466 | 11:04 |
| *** dviroel|out is now known as dviroel | 11:24 | |
| opendevreview | Dan Smith proposed openstack/tempest master: Add keystone unified limits client https://review.opendev.org/c/openstack/tempest/+/788345 | 13:36 |
| opendevreview | Dan Smith proposed openstack/tempest master: Add tests for glance unified quotas https://review.opendev.org/c/openstack/tempest/+/788346 | 13:36 |
| gmann | kopecmartin: ianw can you please check these system scope config bits https://review.opendev.org/c/openstack/devstack/+/778975/1 https://review.opendev.org/c/openstack/devstack/+/778945/2 https://review.opendev.org/c/openstack/devstack/+/779085/1 | 14:20 |
| *** dviroel is now known as dviroel|lunch | 14:50 | |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use older run-tempest for stable branches https://review.opendev.org/c/openstack/tempest/+/787455 | 15:34 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: reproduce run-tempest bug https://review.opendev.org/c/openstack/tempest/+/794685 | 15:34 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Use older run-tempest for stable branches https://review.opendev.org/c/openstack/tempest/+/787455 | 15:38 |
| *** dviroel|lunch is now known as dviroel | 15:39 | |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: reproduce run-tempest bug https://review.opendev.org/c/openstack/tempest/+/794685 | 15:43 |
| dansmith | is depends-on still broken? | 17:39 |
| dansmith | (seems like it) | 17:39 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add keystone unified limits client https://review.opendev.org/c/openstack/tempest/+/788345 | 17:52 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add tests for glance unified quotas https://review.opendev.org/c/openstack/tempest/+/788346 | 17:52 |
| gmann | dansmith: no, its my wrong comment for api_version in client | 17:52 |
| gmann | dansmith: fixed that and rebased test patch | 17:52 |
| gmann | sorry about confusion | 17:53 |
| dansmith | gmann: ah, hah :D | 17:56 |
| mtreinish | gmann: just a heads up, I'm preparing to release a new testtools today. Just running through some sanity check unit test runs on nova and tempest to make sure everything looks fine | 18:35 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: testing https://review.opendev.org/c/openstack/tempest/+/793632 | 18:35 |
| gmann | mtreinish: ok. does stestr always use latest testtools or it is versioned there ? | 18:37 |
| mtreinish | it uses whichever version is installed. Let me double check the min version in the requirements file | 18:38 |
| mtreinish | it's 'testtools>=2.2.0' for stestr | 18:38 |
| mtreinish | so it'll pull this new version in | 18:38 |
| mtreinish | fwiw the unit tests for nova and tempest passed without issue using the new version | 18:38 |
| mtreinish | I can run some more sanity checks before I push it if you want | 18:39 |
| gmann | mtreinish: ok. I think nova, tempest should be enough. if anything else we will get to know while constraint update in requirements - https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L347 | 18:42 |
| gmann | requirements gate run many project unit tests before updating constarint | 18:42 |
| mtreinish | ok sure I figured u-c would block it if it was bad, but I wanted to give you a heads up anyway | 18:51 |
| mtreinish | especially since it ends up being a core piece of the testing | 18:51 |
| mtreinish | ok it's up on pypi now: https://pypi.org/project/testtools/2.5.0/ | 18:51 |
| gmann | mtreinish: +1 thanks | 18:56 |
| gmann | dansmith: instead of moving this test to admin folder, I think we can add it as scenario test as it involve keystone limit + glance verification - https://review.opendev.org/c/openstack/tempest/+/788346/12 | 19:08 |
| gmann | what you say? | 19:08 |
| dansmith | oh sorry I forgot to do that | 19:08 |
| dansmith | gmann: whichever you want, I just forgot to move it | 19:09 |
| gmann | dansmith: scenario test will be good. | 19:09 |
| dansmith | okay I have to get devstack up so I can test the change, as it looks like the test base for those is different | 19:12 |
| dansmith | oh, maybe it inherits from base so I'm okay | 19:13 |
| opendevreview | Dan Smith proposed openstack/tempest master: Add tests for glance unified quotas https://review.opendev.org/c/openstack/tempest/+/788346 | 19:18 |
| gmann | dansmith: but scenario manager base class does not have create_image(). either you can directly call it no image_client but need to add cleanup for each image | 19:21 |
| gmann | or have a create_image in scenario manager class | 19:23 |
| dansmith | okay I better stack for this | 19:33 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: testing https://review.opendev.org/c/openstack/tempest/+/793632 | 19:48 |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: testing https://review.opendev.org/c/openstack/tempest/+/793632 | 20:52 |
| *** dviroel is now known as dviroel|out | 22:01 | |
| opendevreview | Ghanshyam proposed openstack/tempest master: DNM: testing https://review.opendev.org/c/openstack/tempest/+/793632 | 23:02 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!