Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Show a link to the MAAS server. #620

Merged
merged 1 commit into from Oct 15, 2014
Merged

Conversation

frankban
Copy link
Member

When the provider is MAAS, display a link
to the MAAS management interface in the
header.

Also implemented the EnvironmentGet call
in sandbox mode.

When the provider is MAAS, display a link
to the MAAS management interface in the
header.

Also implemented the EnvironmentGet call
in sandbox mode.
@frankban
Copy link
Member Author

QA:

  • juju quickstart a local or ec2 env;
  • switch to this branch:
    juju set juju-gui juju-gui-source="https://github.com/frankban/juju-gui.git maas-server-link";
  • wait for the GUI to be ready;
  • visit the GUI page, no MAAS link should be present in the header.

To see the link, visit the GUI on the juju-gui MAAS: I already
switched to this branch there, and you should see a link on the header.
Clinking the link the MAAS management interface should be open in
another tab.

@jujugui
Copy link
Contributor

jujugui commented Oct 15, 2014

Test FAILed.
Refer to this link for build results: http://ci.jujugui.org:8080/job/juju-gui/2095/

@mitechie
Copy link
Contributor

👍 QA ok on guimaas

@kadams54
Copy link
Contributor

👍 Looks good!

@frankban
Copy link
Member Author

Thanks for the reviews!
:shipit:

@jujugui
Copy link
Contributor

jujugui commented Oct 15, 2014

Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge

jujugui added a commit that referenced this pull request Oct 15, 2014
Show a link to the MAAS server.

When the provider is MAAS, display a link
to the MAAS management interface in the
header.

Also implemented the EnvironmentGet call
in sandbox mode.
@jujugui jujugui merged commit d8a76ac into juju:develop Oct 15, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants