Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP - display error in output when server failed to start #128

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented Jul 24, 2023

SUMMARY

Currently the server start errors are not returned to the end user making hard to debug running issue.
This pull requests helps debugging by capturing shell command logs.

Note: This piece of code will display log information for socket creation, there is no risk to display session credentials or any secured data as this happens later on the process.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

turbo daemon

@softwarefactory-project-zuul
Copy link

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/68514806193e4deea4fb12235b32a8cc

ansible-test-sanity-docker-devel FAILURE in 11m 18s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 10m 34s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 14m 45s
✔️ ansible-test-sanity-docker-stable-2.10 SUCCESS in 12m 00s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 13m 51s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 09s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 14s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 41s
✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 13m 51s
✔️ build-ansible-collection SUCCESS in 10m 38s
✔️ ansible-test-splitter SUCCESS in 4m 41s
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 16m 42s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 30m 26s
✔️ integration-kubernetes.core-with-turbo-3 SUCCESS in 31m 51s
✔️ ansible-tox-linters SUCCESS in 10m 07s
✔️ ansible-galaxy-importer SUCCESS in 4m 17s

@softwarefactory-project-zuul
Copy link

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/9af78d0cce22450faa2c600e3b74d0fd

ansible-test-sanity-docker-devel FAILURE in 9m 50s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 10m 37s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 15m 23s
✔️ ansible-test-sanity-docker-stable-2.10 SUCCESS in 10m 59s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 57s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 12s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 12s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 7m 05s
✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 12m 52s
✔️ build-ansible-collection SUCCESS in 10m 31s
✔️ ansible-test-splitter SUCCESS in 4m 45s
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 15m 32s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 29m 30s
✔️ integration-kubernetes.core-with-turbo-3 SUCCESS in 30m 52s
✔️ ansible-tox-linters SUCCESS in 10m 06s
✔️ ansible-galaxy-importer SUCCESS in 3m 16s

@abikouo abikouo force-pushed the start_server_log branch from 92c118c to fbe3768 Compare July 24, 2023 16:06
@softwarefactory-project-zuul
Copy link

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/331c1cf86b184df8988e4c9012efb461

ansible-test-cloud-integration-vmware-rest NODE_FAILURE Node request 200-0006224426 failed in 0s
✔️ build-ansible-collection SUCCESS in 10m 39s
✔️ ansible-test-splitter SUCCESS in 4m 42s
✔️ integration-kubernetes.core-with-turbo-1 SUCCESS in 15m 39s
✔️ integration-kubernetes.core-with-turbo-2 SUCCESS in 29m 46s
✔️ integration-kubernetes.core-with-turbo-3 SUCCESS in 34m 43s
✔️ ansible-tox-linters SUCCESS in 10m 19s
✔️ ansible-galaxy-importer SUCCESS in 4m 59s

@komaldesai13
Copy link

recheck

@softwarefactory-project-zuul
Copy link

@komaldesai13
Copy link

recheck

@softwarefactory-project-zuul
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants