Workaround or missing redirect_info in JaaS. #113

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Collaborator

petevg commented Apr 26, 2017

JaaS deployments were throwing a JujuAPIError because the JaaS
controller does not appear to have implemented redirect_info.

Dropped in a change to catch the error and ignore it in that case.

@tvansteenburgh @johnsca I'm not sure how to test JaaS stuff effectively. To validate that my code works, you can run examples/add_model.py, after editing the file to pass cloud_name="aws" into the call to .add_model.

(Without this change, add_model will fail due to a JujuAPIErrror.)

Workaround or missing redirect_info in JaaS.
JaaS deployments were throwing a JujuAPIError because the JaaS
controller does not appear to have implemented redirect_info.

Dropped in a change to catch the error and ignore it in that case.

@petevg petevg referenced this pull request in juju-solutions/matrix Apr 26, 2017

Merged

Feature/jaas support v2 #127

Member

johnsca commented Apr 26, 2017

I created a more comprehensive refactor in #116 which I'd like to close this PR in favor of if it looks ok to you.

Collaborator

petevg commented Apr 26, 2017

Agree that @johnsca's refactor is more comprehensive. Closing this out ...

@petevg petevg closed this Apr 26, 2017

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