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

Collect unsafe logs #679

Merged
merged 2 commits into from
Feb 2, 2022
Merged

Collect unsafe logs #679

merged 2 commits into from
Feb 2, 2022

Conversation

mtojek
Copy link
Contributor

@mtojek mtojek commented Feb 2, 2022

This PR modifies the Jenkinsfile to collect unsafe artifacts. We plan to use it to dump Terraform service deployer logs, useful to debug #665 and #662.

Logs are saved in this place/bucket: beats-ci-temp-internal/Ingest-manager/elastic-package/PR-679-2

@elasticmachine
Copy link
Collaborator

elasticmachine commented Feb 2, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Reason: null

  • Start Time: 2022-02-02T11:34:37.435+0000

  • Duration: 29 min 45 sec

  • Commit: e6e79d2

Test stats 🧪

Test Results
Failed 0
Passed 512
Skipped 0
Total 512

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@mtojek mtojek self-assigned this Feb 2, 2022
@mtojek mtojek requested review from a team and endorama February 2, 2022 12:07
@mtojek mtojek marked this pull request as ready for review February 2, 2022 12:07
@endorama
Copy link
Member

endorama commented Feb 2, 2022

@mtojek thank you!!

I was wondering if the insecure logs functionality is documented; if not, I'd like to add a doc page detailing this functionality. Would that be ok?

@mtojek
Copy link
Contributor Author

mtojek commented Feb 2, 2022

Sure, but please remember that this feature is available for Elastic members with access to the observability-elastic account. Nobody from outside can access it. Please also consider it low-priority compared to other PRs to elastic-package.

@mtojek mtojek merged commit 0c62930 into elastic:main Feb 2, 2022
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.

3 participants