#7186 testing: get back command outputs when running tests
Closed: fixed 6 years ago Opened 6 years ago by mreznik.

Previously when running the integration tests we got also command`s stdout:

E.g. ipa-server-install

[ipa.ipatests.pytest_plugins.integration.host.Host.master.ParamikoTransport] RUN ['ipa-server-install', '-a', 'xxx', '-p', 'xxx', '--external-cert-file', '/root/ipatests/testdb/ipa.crt', '--external-cert-file', '/root/ipatests/testdb/ca.crt']
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] RUN ['ipa-server-install', '-a', 'xxx', '-p', 'xxx', '--external-cert-file', '/root/ipatests/testdb/ipa.crt', '--external-cert-file', '/root/ipatests/testdb/ca.crt']
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] 
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] The log file for this installation can be found in /var/log/ipaserver-install.log
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] ==============================================================================
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] This program will set up the FreeIPA Server.
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] 
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] This includes:
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Configure a stand-alone CA (dogtag) for certificate management
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Configure the Network Time Daemon (ntpd)
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Create and configure an instance of Directory Server
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Create and configure a Kerberos Key Distribution Center (KDC)
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Configure Apache (httpd)
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Configure DNS (bind)
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10]   * Configure the KDC to enable PKINIT
[ipa.ipatests.pytest_plugins.integration.host.Host.master.cmd10] 

However now we are missing those:

[ipatests.pytest_plugins.integration.host.Host.master.ParamikoTransport] RUN ['ipa-server-install', '-U', '-a', 'xxx', '-p', 'xxx', '--setup-dns', '--no-forwarders', '-n', 'IPA.TEST', '-r', 'IPA.TEST', '--domain-level=1', '--external-ca']
[ipatests.pytest_plugins.integration.host.Host.master.ParamikoTransport] WRITE /root/ipatests/root_ca.crt
[ipatests.pytest_plugins.integration.host.Host.master.ParamikoTransport] WRITE /root/ipatests/ipa_ca.crt

Should be related to the following changes:

07229c8ff66669ba87b7d6599c3ec0d362ef2be4 logging: do not use `ipa_log_manager` to create module-level loggers
7a482b7c7286f738f10e43ca70c94c35029398bc logging: do not log into the root logger
ab9d1e75fc69830f9ee79f62501701f3a7a82c52 logging: do not reference loggers in arguments and attributes
ffadcb0414e31117921660ca1b2526323afac56f logging: remove object-specific loggers
9d19654cbd99c2133bfb86253c63da33621ea110 logging: use the actual root logger as the root logger
f62a0fdb904d2a4bb1961847e240dbb6df3b0b67 logging: port to standard Python logging
464516489ff029a39446138b16e858e96d157841 logging: do not configure any handlers by default

As in 4.5 and after reverting those it is working as expected.


This issue affects the output of PR CI runner.log. It'd be quite useful to get this functionality back.

Metadata Update from @tkrizek:
- Issue priority set to: critical
- Issue set to the milestone: FreeIPA 4.6.2

6 years ago

Metadata Update from @fbarreto:
- Issue assigned to fbarreto

6 years ago

Metadata Update from @fbarreto:
- Custom field on_review adjusted to https://github.com/freeipa/freeipa/pull/1234

6 years ago

master:

  • 16da7c6 Fix log capture when running pytests_multihosts commands

ipa-4-6:

  • d9737c0 Fix log capture when running pytests_multihosts commands

Metadata Update from @stlaz:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

6 years ago

Login to comment on this ticket.

Metadata