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

fix(core): do not send a cloud provider on v2 search calls #7142

Merged
merged 2 commits into from
Jun 26, 2019
Merged

fix(core): do not send a cloud provider on v2 search calls #7142

merged 2 commits into from
Jun 26, 2019

Conversation

anotherchrisberry
Copy link
Contributor

I am not sure the backstory to this, but if there are multiple default cloud providers configured, we will only get search results for the first one in the list, which can't be right, right?

At Netflix, this manifests in us not seeing search results when running deck locally for cloud provider resources, such as server groups, instances, etc., because we're getting search results for appengine

Copy link
Contributor

@christopherthielen christopherthielen left a comment

Choose a reason for hiding this comment

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

I assume not sending cloudProvider at all is OK?

@anotherchrisberry
Copy link
Contributor Author

@christopherthielen it seems to be fine when I run it locally, I guess we'll find out?!

@anotherchrisberry anotherchrisberry merged commit fb58d70 into spinnaker:master Jun 26, 2019
@anotherchrisberry anotherchrisberry deleted the search-v2-cp branch June 26, 2019 04:13
anotherchrisberry added a commit that referenced this pull request Jun 26, 2019
…94 (#7150)

* chore(core): Bump version to 0.0.377

d5425b4 refactor(core): virtualize execution rendering (#7140)
342087c feat(core): allow users to override pipeline graph positions (#7141)
fb58d70 fix(core): do not send a cloud provider on v2 search calls (#7142)
5d6d9aa chore(core): clarify clone stage help text (#7131)
745f0a1 fix(core): Display latest template in pipeline template list (#7145)
456172b fix(webhooks): addresses issue 3450 - introduce a delay before polling webhook (#7144)
7238d1d feat(core): Enable new artifacts workflow in bakeManifest (#7138)
45e5aa3 feat(artifacts): find multiple artifacts from single execution (#7139)
dae73da fix(core): provide key for repeating param JSX elements (#7136)
bbc1d06 fix(core): filter falsy error messages from errors object on tasks (#7135)
4782808 refactor(core): Reactify overrideTimeout (#7126)
0a3bd68 feat(core/presentation): Always call onBlur in Checklist to "mark as touched" (#7134)
cdd6f23 chore(package): Just Update Prettier™
7e464a9 fix(amazon): Support SpEL in advanced capacity (#7124)
08e9506 chore(deck): Update to Typescript 3.4
fa515dc fix(core): do not stretch provider logos in selection modal (#7128)

* chore(docker): Bump version to 0.0.42

8358010 fix(docker): Allow auto-switch to manual entry when refreshing images (#7120)

* chore(amazon): Bump version to 0.0.194

cdd6f23 chore(package): Just Update Prettier™
7e464a9 fix(amazon): Support SpEL in advanced capacity (#7124)
08e9506 chore(deck): Update to Typescript 3.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants