Skip to content

Clean used resources#183

Merged
mtojek merged 1 commit intoelastic:masterfrom
mtojek:13-clean-used-resources
Nov 30, 2020
Merged

Clean used resources#183
mtojek merged 1 commit intoelastic:masterfrom
mtojek:13-clean-used-resources

Conversation

@mtojek
Copy link
Copy Markdown
Contributor

@mtojek mtojek commented Nov 24, 2020

Resolves #13

Sample output:

$ elastic-package clean -v
2020/11/24 13:54:32 DEBUG Enable verbose logging
Clean used resources
2020/11/24 13:54:32 DEBUG Clean build resources
2020/11/24 13:54:32 DEBUG Build directory for integration: /Users/marcin.tojek/go/src/github.com/elastic/elastic-package/build/integrations/nginx
2020/11/24 13:54:32 DEBUG Remove directory (path: /Users/Foobar/go/src/github.com/elastic/elastic-package/build/integrations/nginx)
Build resources removed: /Users/Foobar/go/src/github.com/elastic/elastic-package/build/integrations/nginx
2020/11/24 13:54:32 DEBUG Clean built packages from the development stack
2020/11/24 13:54:32 DEBUG Stack package is not part of the development stack (missing path: /Users/Foobar/.elastic-package/stack/development/nginx)
2020/11/24 13:54:32 DEBUG Clean all service logs
2020/11/24 13:54:32 DEBUG Remove folder content (path: /Users/Foobar/.elastic-package/tmp/service_logs)
Temporary service logs removed: /Users/Foobar/.elastic-package/tmp/service_logs
Done

@mtojek mtojek requested a review from ycombinator November 24, 2020 12:55
@mtojek mtojek self-assigned this Nov 24, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented Nov 24, 2020

💚 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

Expand to view the summary

Build stats

  • Build Cause: Branch indexing

  • Start Time: 2020-11-25T22:28:05.360+0000

  • Duration: 8 min 24 sec

Test stats 🧪

Test Results
Failed 0
Passed 9
Skipped 0
Total 9

Copy link
Copy Markdown
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Love the output messages showing exactly what locations got cleaned!

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Nov 30, 2020

jenkins run the tests please

2 similar comments
@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Nov 30, 2020

jenkins run the tests please

@mtojek
Copy link
Copy Markdown
Contributor Author

mtojek commented Nov 30, 2020

jenkins run the tests please

@mtojek mtojek merged commit 9647620 into elastic:master Nov 30, 2020
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.

Command: clean up used resources

3 participants