| *** nguyenhai_ has quit IRC | 02:18 | |
| *** nehaalhat_ has joined #openstack-blazar | 02:51 | |
| *** tetsuro has joined #openstack-blazar | 02:52 | |
| *** nehaalhat_ has quit IRC | 02:52 | |
| *** tetsuro has quit IRC | 03:58 | |
| *** tetsuro has joined #openstack-blazar | 04:54 | |
| *** nehaalhat has quit IRC | 05:02 | |
| *** ShilpaSD has quit IRC | 05:04 | |
| *** ShilpaSD has joined #openstack-blazar | 05:06 | |
| *** masahito has joined #openstack-blazar | 05:09 | |
| *** tetsuro has quit IRC | 05:34 | |
| *** nehaalhat has joined #openstack-blazar | 05:38 | |
| nehaalhat | masahito: Hi | 05:42 |
|---|---|---|
| nehaalhat | masahito: I want to know that which CI job will run tempest test cases return in openstack/blazar-tempest-plugin against any changes made in openstack/blazar project | 05:43 |
| nehaalhat | masahito:tempest test cases written* | 05:43 |
| masahito | nehaalhat: hi | 06:01 |
| masahito | The host reservation scenario tests and API tests are executed to the all patches. | 06:02 |
| masahito | The instance reservation scenario is now under implementing. https://review.openstack.org/#/q/status:open+project:openstack/blazar-tempest-plugin+branch:master+topic:bug/1714438 | 06:02 |
| *** tetsuro has joined #openstack-blazar | 06:32 | |
| nehaalhat | masahito: ok.. I want to know how these tempest test cases will get run against the patches submitted under openstack/blazar? Is there a CI job? | 06:54 |
| masahito | nehaalhat: Yes, the CI is running under the zuulv3. Blazar uses general OpenStack CI mechanism, not Blazar specific one. | 07:13 |
| nehaalhat | masahito: Is this the CI job: blazar-devstack-dsvm? | 07:16 |
| masahito | yes | 07:16 |
| nehaalhat | which will run tempest test cases | 07:16 |
| nehaalhat | ok | 07:16 |
| *** tetsuro has quit IRC | 07:39 | |
| *** priteau has joined #openstack-blazar | 07:48 | |
| *** priteau has quit IRC | 07:50 | |
| *** tetsuro has joined #openstack-blazar | 08:26 | |
| *** priteau has joined #openstack-blazar | 08:45 | |
| priteau | masahito: So you think the Tempest run would still succeed without the `self.reservation_client.delete_lease` call at the end, it's just an extra precaution? | 10:03 |
| masahito | right. | 10:03 |
| priteau | Would you be OK with these patches without the lease delete? I think it's better to exercise the code as much as possible. If the presence of terminated leases triggers an issue, it's likely a bug and we would have to fix it | 10:06 |
| masahito | IMO, both perspectives are important for scenario tests. First one is yours, the next test should work even though the previous lease still exists. Second one is mine, the scenario should be tested in clean environment. | 10:07 |
| masahito | Sure. Ok, I'll remove the codes in next patch set. | 10:08 |
| priteau | If you really want all tests to be done in a clean environment, I would prefer if tearDown did the cleaning of all existing leases, as you suggested in the meeting | 10:11 |
| priteau | But I think I prefer the no-cleaning approach, to potentially catch more bugs | 10:18 |
| masahito | okay. | 10:18 |
| *** masahito has quit IRC | 10:30 | |
| *** tetsuro has quit IRC | 10:35 | |
| *** tetsuro has joined #openstack-blazar | 10:40 | |
| *** tetsuro has quit IRC | 12:07 | |
| openstackgerrit | Pierre Riteau proposed openstack/blazar master: Allow terminating a lease in ERROR state https://review.openstack.org/604159 | 12:22 |
| openstackgerrit | Merged openstack/blazar master: Handle requirements set to empty strings https://review.openstack.org/602173 | 18:40 |
| *** priteau has quit IRC | 21:15 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!