On 16.03.2016 14:59, Oleg Fayans wrote:
But as I wrote in ticket #5736, this ticket should be closed, because
issue is caused by ticket which is not finished yet, so we should
continue just with original ticket.
On 03/16/2016 02:39 PM, Martin Basti wrote:
On 16.03.2016 10:59, Oleg Fayans wrote:
With this patch applied integration tests pass and in-tree tests are
@mkubik, It is not possible to put the decorator to util.py as per our
discussion, because it uses tasks, so tasks must be imported. But tasks
already import util, which leads to circular imports. So I've put it to
Use right ticket in commit message (#5723)
But (#5736) is exactly the issue that is being addressed. Probably note
both tickets in the commit message?
Link to ticket should be last in the commit message
wrong doc string, it removes *only* replicas not clients
No, in fact it removes both:
Both tasks have raiseonerr set to False, which means that even if
replica was not installed but the client was - it will also be removed
I see just
for host in args.replicas
I don't see any
for host in args.clients
Also uninstall_client should not be there. ipa-server-install
--uninstall removes client too. The extra call of uninstall client is
IMO there just because an ancient bug that is already fixed.
can we rename it to something different? (replicas_cleanup,
replicas_cleanup, or even topo_cleanup sounds OK to me.
Please fix commit message
- Wile trated correctly
- rewrote -> rewrite
+ def wrapped(*args):
+ for host in args.replicas:
Shouldn't be there try-finally around func() call, or something?
No, the wrapped function is a test_* method: if it fails we need to see
the original failure
but if something raise an exception in func(), cleanup will not be executed.
You can do
In : try:
...: raise ValueError('Hello')
...: raise ValueError('Cleanup')
...: except Exception:
ValueError Traceback (most recent call last)
<ipython-input-4-affb927f7603> in <module>()
----> 2 raise ValueError('Hello')
5 raise ValueError('Cleanup')
Are you sure that there is no need to return result of func()?
The same applies here: we never return results from test_* methods
*) Please create additional patch that will add licence there
Will do :)
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code