Skip to content

Commit

Permalink
Merge pull request #854 from flaper87/actuator-clouds
Browse files Browse the repository at this point in the history
Only copy credentials for the cloud used to deploy OCP
  • Loading branch information
openshift-merge-robot authored Dec 13, 2018
2 parents c309052 + 7d7cdce commit 23aa436
Show file tree
Hide file tree
Showing 19 changed files with 9,838 additions and 2 deletions.
9 changes: 9 additions & 0 deletions Gopkg.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions Gopkg.toml
Original file line number Diff line number Diff line change
Expand Up @@ -91,3 +91,7 @@ ignored = [
[[constraint]]
name = "github.com/openshift/client-go"
revision = "1fa528d3be060e4c7178eb69e76d37cf7e699e3c"

[[constraint]]
name = "github.com/go-yaml/yaml"
version = "2.2.2"
16 changes: 14 additions & 2 deletions pkg/asset/manifests/openshift.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,14 @@ import (
"path/filepath"

"github.com/aws/aws-sdk-go/aws/session"
"github.com/ghodss/yaml"

// TODO(flaper87): Migrate to ghodss asap
// This yaml is currently used only by the OpenStack
// clouds serialization. We're working on migrating
// clientconfig out of go-yaml. We'll use it here
// until that happens.
// https://github.com/openshift/installer/pull/854
"gopkg.in/yaml.v2"

"github.com/gophercloud/utils/openstack/clientconfig"
"github.com/openshift/installer/pkg/asset"
Expand Down Expand Up @@ -76,10 +83,15 @@ func (o *Openshift) Generate(dependencies asset.Parents) error {
},
}
case "openstack":
clouds, err := clientconfig.LoadCloudsYAML()
opts := new(clientconfig.ClientOpts)
cloud, err := clientconfig.GetCloudFromYAML(opts)
if err != nil {
return err
}
clouds := make(map[string]map[string]*clientconfig.Cloud)
clouds["clouds"] = map[string]*clientconfig.Cloud{
"openstack": cloud,
}

marshalled, err := yaml.Marshal(clouds)
if err != nil {
Expand Down
201 changes: 201 additions & 0 deletions vendor/github.com/go-yaml/yaml/LICENSE

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

31 changes: 31 additions & 0 deletions vendor/github.com/go-yaml/yaml/LICENSE.libyaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 13 additions & 0 deletions vendor/github.com/go-yaml/yaml/NOTICE

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 23aa436

Please sign in to comment.