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

Implement sanity check regarding versions for depending components (Mailer, Chatbot,...) #38

Closed
DonMartin76 opened this issue Dec 5, 2016 · 0 comments
Assignees
Milestone

Comments

@DonMartin76
Copy link
Member

For transition scenarios where the wicked core components are updated one by one (which will be the case for Kubernetes and alike), the wicked components should do a standard version check on the portal API when calling it. It should pass an "expected version" e.g. as a header to the upstream API, and if present, this version is checked by the API and in case it does not match the call is rejected.

This will be optional behavior which will be implemented by the standard components, but which will not be enforced by the portal API in case the additional header is not present (this will be a similar behavior as described in #36).

Containers which receive such a faulty version, and/or have a version mismatch, will fail early and fast with a descriptive error message.

As soon as the version of the container has been updated, the operations would return to normal again.

@DonMartin76 DonMartin76 added this to the 0.11.0 milestone Dec 5, 2016
DonMartin76 added a commit to apim-haufe-io/wicked.api that referenced this issue Dec 8, 2016
- Haufe-Lexware/wicked.haufe.io#36 and
- Haufe-Lexware/wicked.haufe.io#38

If you pass X-Config-Hash, the API will reject the call if the config hash is different than stored.
DonMartin76 added a commit to apim-haufe-io/wicked.ui that referenced this issue Dec 8, 2016
DonMartin76 added a commit to apim-haufe-io/wicked.kong-adapter that referenced this issue Dec 8, 2016
DonMartin76 added a commit to apim-haufe-io/wicked.mailer that referenced this issue Dec 8, 2016
DonMartin76 added a commit to apim-haufe-io/wicked.chatbot that referenced this issue Dec 8, 2016
@DonMartin76 DonMartin76 self-assigned this Dec 9, 2016
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

No branches or pull requests

1 participant