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

Added support for Bosh UAA #9

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

hpejcinovic-pivotal
Copy link
Contributor

No description provided.

@cah-jeremykuhn
Copy link

cah-jeremykuhn commented Nov 9, 2016

Hello,

We attempted to use this branch to deploy Chaos Lemur to our PCF instance, and are getting an error 401 when the application is starting up and attempting to connect to Bosh Director. The error we are getting in PCF is below:

2016-11-09T07:47:00.268-05:00 [APP/0] [OUT] org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'standardDirectorUtils' defined in file [/home/vcap/app/io/pivotal/strepsirrhini/chaoslemur/infrastructure/StandardDirectorUtils.class]: Bean instantiation via constructor failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [io.pivotal.strepsirrhini.chaoslemur.infrastructure.StandardDirectorUtils]: Constructor threw exception; nested exception is org.springframework.web.client.HttpClientErrorException: 401 Unauthorized

@hpejcinovic-pivotal
Copy link
Contributor Author

hpejcinovic-pivotal commented Nov 9, 2016

Hi,
Did you create the Bosh client account first as per Deployment section in https://github.com/hpejcinovic-pivotal/chaos-lemur/blob/1360449f3a8afdd608e379197760de268b5ea79c/README.md

You use the this user's username password and pass it via config variables in manifest.yml

If it still doesn't work let me know.

@allomov
Copy link

allomov commented Jan 17, 2017

Hey!

Sorry for being improvident, but I implemented the same functionality here: #10. Just noticed it. It so strange that your PR have not been approved yet.

Here is a difference that my PR has: I expect that somebody is going to use old fashioned basic auth with BOSH, I also made some refactoring to the way properties are retrieved (everything is stored in InfrastructureConfiguration class now, so some custom validation could be added later).

@nebhale
Copy link
Contributor

nebhale commented Jan 18, 2017

I'll take a look at this in the next couple of days and updates will be centralized in #5.

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.

4 participants