Clean up the lib389 log messages around commands so that it's a cleaner user experience.
attachment 0001-Ticket-48750-Clean-up-logging-to-improve-command-exp.patch
Hi William,
I think it is great that you've decided to clean up our logs. But, unfortunately, a lot of lib389 tests fail with your patch. I think the main reason is that you've deleted "CONF_PRODUCT_NAME" from lib389/_constants.py.
Can you please fix tests too and include changes to your patch? Just be sure that all of them are still pass. :)
Thanks, Simon
Reinstate product name 0001-Ticket-48750-Clean-up-logging-to-improve-command-exp.2.patch
Hi,
I didn't find anywhere that used it, which is why I removed it. I've put it back now.
Sorry about the confusion.
Thanks, all tests PASS. Looks good to me. Ack.
commit d33624108669bafa12acd46b4544fee329933fda To ssh://git.fedorahosted.org/git/389/lib389.git 543f868..d336241 master -> master
Metadata Update from @spichugi: - Issue assigned to firstyear - Issue set to the milestone: lib389 1.0.2
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/1810
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.