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

Revert "chore(clouddriver/oracle): Get the Oracle API jar from Maven." #3756

Merged
merged 1 commit into from
Jun 6, 2019

Conversation

robzienert
Copy link
Member

Reverts #3695

spinnaker/spinnaker#4488
spinnaker/spinnaker#4484

This will get Clouddriver as it exists today back to a working runtime state (albeit a painful dev / build experience thanks to Oracle). The @spinnaker/toc will add to the agenda to more clearly define what it takes to:

  1. Requirements for adding a new core cloud provider.
  2. Requirements for continued core cloud provider support.
  3. Process for removal of cloud providers.

It's my opinion that Oracle should be responsible for maintaining their cloud provider and if they're unable to do so, their cloud provider should be deprecated and removed.

cc @andrewbackes @plumpy @shihchang @guoyongzhang

@plumpy
Copy link
Member

plumpy commented Jun 6, 2019

spinnaker/spinnaker#4488 is unrelated (this change isn't on the 1.14.x branch)

@spinnakerbot
Copy link
Contributor

The following commits need their title changed:

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

@robzienert
Copy link
Member Author

Oopsies! I was distracted by dog.

@robzienert robzienert merged commit be87697 into master Jun 6, 2019
@robzienert robzienert deleted the revert-3695-oracle_maven branch August 14, 2019 19:56
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