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

add log drain to an addon #575

Merged
merged 5 commits into from Jul 1, 2020
Merged

Conversation

curzolapierre
Copy link
Contributor

@curzolapierre curzolapierre commented Jun 25, 2020

User will provide addon <addon_uuid> to the log-drains command.
Example:

$ scalingo --app my-app --addon ad-1234567-1123 log-drains-add \
--type papertrail --host logs.papertrailapp.com --port 10303

Or add a log drains to all addons linked to the app:

$ scalingo --app my-app --with-addon log-drains-add \
--type papertrail --host logs.papertrailapp.com --port 10303

Fix #571

@curzolapierre curzolapierre self-assigned this Jun 25, 2020
@curzolapierre curzolapierre requested a review from EtienneM Jun 30, 2020
Gopkg.toml Outdated Show resolved Hide resolved
cmd/log_drains.go Outdated Show resolved Hide resolved
cmd/log_drains.go Outdated Show resolved Hide resolved
cmd/log_drains.go Outdated Show resolved Hide resolved
cmd/log_drains.go Outdated Show resolved Hide resolved
log_drains/add.go Outdated Show resolved Hide resolved
log_drains/add.go Show resolved Hide resolved
@curzolapierre curzolapierre requested review from EtienneM and removed request for EtienneM Jun 30, 2020
log_drains/add.go Outdated Show resolved Hide resolved
@curzolapierre curzolapierre force-pushed the feature/571/add_log_drain_to_DB branch from 88bb6bd to 5a2d1ea Compare Jul 1, 2020
@curzolapierre curzolapierre requested a review from EtienneM Jul 1, 2020
@EtienneM EtienneM merged commit 13562e9 into master Jul 1, 2020
1 check passed
@EtienneM EtienneM deleted the feature/571/add_log_drain_to_DB branch Jul 1, 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.

2 participants