Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Commit

Permalink
v1.9 clusters have 443-enabled dashboards (#1988)
Browse files Browse the repository at this point in the history
  • Loading branch information
jackfrancis committed Jan 4, 2018
1 parent 4fefc83 commit fc9ffbe
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 1 deletion.
12 changes: 12 additions & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -325,6 +325,12 @@ workflows:
filters:
branches:
ignore: master
- k8s-1.9-release-e2e:
requires:
- pr-e2e-hold
filters:
branches:
ignore: master
- swarm-e2e:
requires:
- pr-e2e-hold
Expand Down Expand Up @@ -385,6 +391,12 @@ workflows:
filters:
branches:
only: master
- k8s-1.9-release-e2e:
requires:
- test
filters:
branches:
only: master
- swarm-e2e:
requires:
- test
Expand Down
6 changes: 5 additions & 1 deletion test/e2e/kubernetes/kubernetes_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,11 @@ var _ = Describe("Azure Container Cluster using the Kubernetes Orchestrator", fu

s, err := service.Get("kubernetes-dashboard", "kube-system")
Expect(err).NotTo(HaveOccurred())
port := s.GetNodePort(80)
dashboardPort := 80
if eng.ClusterDefinition.Properties.OrchestratorProfile.OrchestratorRelease == "1.9" {
dashboardPort = 443
}
port := s.GetNodePort(dashboardPort)

master := fmt.Sprintf("azureuser@%s", kubeConfig.GetServerName())
nodeList, err := node.Get()
Expand Down

0 comments on commit fc9ffbe

Please sign in to comment.