Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Throw errors from ECS/ACI/Kube backends on unsupported flags#2005

Closed
ulyssessouza wants to merge 7 commits intodocker-archive:mainfrom
ulyssessouza:error-on-unsupported-flags
Closed

Throw errors from ECS/ACI/Kube backends on unsupported flags#2005
ulyssessouza wants to merge 7 commits intodocker-archive:mainfrom
ulyssessouza:error-on-unsupported-flags

Conversation

@ulyssessouza
Copy link
Copy Markdown
Contributor

@ulyssessouza ulyssessouza commented Aug 9, 2021

What I did
Add checks to ECS, ACI, and Kubernetes backends.

/test-kube
/test-aci
/test-ecs
/test-windows

@ulyssessouza ulyssessouza requested a review from ndeloof August 9, 2021 13:05
@github-actions github-actions Bot added aci cli cli ecs kube Kubernetes backend labels Aug 9, 2021
@ulyssessouza ulyssessouza force-pushed the error-on-unsupported-flags branch 2 times, most recently from e67488b to 4186282 Compare August 9, 2021 14:08
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
@ulyssessouza ulyssessouza force-pushed the error-on-unsupported-flags branch from 4186282 to efb2f37 Compare August 10, 2021 03:46
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
@ulyssessouza ulyssessouza force-pushed the error-on-unsupported-flags branch 13 times, most recently from 8ea258f to 25976be Compare August 12, 2021 05:13
Signed-off-by: Ulysses Souza <ulyssessouza@gmail.com>
@ulyssessouza ulyssessouza force-pushed the error-on-unsupported-flags branch from 25976be to 55fd893 Compare August 12, 2021 05:19
@ulyssessouza
Copy link
Copy Markdown
Contributor Author

Closing in favor of #2039

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

aci cli cli ecs kube Kubernetes backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant