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

Initial titan integration #8

Merged
merged 24 commits into from
Sep 22, 2015
Merged

Initial titan integration #8

merged 24 commits into from
Sep 22, 2015

Conversation

sthadeshwar
Copy link
Contributor

This is an initial draft. This module can be disabled by setting the titan.enabled property to false.


TitanClient getTitanClient(String account, String region) {
TitanEndpoint titanEndpoint = new TitanEndpoint(account, region)
TitanClientFactory.newInstance(titanEndpoint)
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd expect to see something like this done in a DI friendly manner

@cfieber
Copy link
Contributor

cfieber commented Jul 1, 2015

I'd like to see this cleaned up quite a bit more before it hits master - first pass through it looks very copied and pasted from the AWS implementation and I'd like to make sure stuff isn't coming along for the ride from that where it doesn't make sense.

Also will need some tests in here...

@sthadeshwar
Copy link
Contributor Author

  • Removed the ‘cats’ prefix for the oort providers
  • Removed the TitanClient.newInstance call and made the titan client implementation be injected via a bean
  • Removed the OnDemandCacheUpdater as it has been genericised and moved to clouddriver-core
  • Implemented the TitanHealthIndicator (basically, completed the TODO)
  • Replicated @dependsOn('netflixAmazonCredentials') in AwsProviderConfig for TitanConfig
  • Removed the default env in TitanCredentialsConfig
  • Enbaled provision for creation of separate accounts for vpc (for example: titan-test-internal (vpc0), titan-test-external (ec2))

sthadeshwar added a commit that referenced this pull request Sep 22, 2015
@sthadeshwar sthadeshwar merged commit 3fc7648 into master Sep 22, 2015
@cfieber cfieber deleted the titan-integration branch October 29, 2015 16:15
duftler pushed a commit that referenced this pull request Nov 13, 2015
update unixScript.text template file
tomaslin pushed a commit to tomaslin/clouddriver that referenced this pull request Sep 11, 2018
…essor

Introduce ApplicationSpecificIamRoleDescriptionPreProcessor
tomaslin pushed a commit to tomaslin/clouddriver that referenced this pull request Sep 11, 2018
…-iam-role-tagging to master

* commit '70dae3ffca5a416dfe28d9edeb201d152ef11a1e':
  Support for restricting the set of users that can tag an iam role
luba239 pushed a commit to vasiliy-briginets/clouddriver that referenced this pull request Mar 5, 2020
ttomsu pushed a commit to ttomsu/clouddriver that referenced this pull request Mar 11, 2020
Move config from halyard-web to halyard-config
ttomsu pushed a commit to ttomsu/clouddriver that referenced this pull request Mar 11, 2020
* Adding openstack bakery (#3)

Adding Openstack Bakery commands to halyard


h/t @shazy792 @dylanbernhardt @MatthewKohner @msilpala

* fix openstack bakery commands (spinnaker#4)

* fix formatting (spinnaker#5)

* Remove unused imports and some extra spaces. (spinnaker#6)

* fix virtualization settings for openstack base image (spinnaker#7)

* fix formatting

* fix virtualization settings for openstack base image

* Bakery Options finishing touches (spinnaker#8)

* remove extraneous if

* fix intellij formatting

* Make bakery insecure parameter nullable

* remove commented out line

* remove OpenstackBakeryDefaults null setting and make insecure required setting so vaildation works
kirangodishala added a commit to kirangodishala/clouddriver that referenced this pull request May 10, 2022
Moved all the classes from groovy to java.
sanopsmx pushed a commit to sanopsmx/clouddriver that referenced this pull request May 1, 2023
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

Successfully merging this pull request may close these issues.

2 participants