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.

Sign up for GitHub

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

Jump to bottom

fix(misc): newer version of fiat api client requires url to wire up c… #900

Merged

cfieber merged 1 commit into spinnaker:master from srekapalli:fiat_config_fix

May 14, 2020

Contributor

srekapalli commented May 14, 2020

Newer version of fiat api client requires url and performs non null check and this change is to satisfy that constraint by supplying the necessary config while building the application contexts in unit tests. There is a failing build of autobump PR due to this.


          fix(misc): newer version of fiat api client requires url to wire up c…

fdf64ac

…lient

srekapalli requested review from robzienert, cfieber, ajordens and jonsie

May 14, 2020 07:46

cfieber approved these changes

View reviewed changes

cfieber merged commit 0d3379e into spinnaker:master

spinnakerbot added the target-release/1.21 label

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

Labels

target-release/1.21