-
Notifications
You must be signed in to change notification settings - Fork 7
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
MRPHS-5166 : GCP cloud creation via c3ntry <DO NOT MERGE> #106
base: master
Are you sure you want to change the base?
Conversation
ClientEmail string `json:"client_email"` | ||
ClientId string `json:"client_id"` | ||
// AccountFile represents the structure of the account file JSON file. | ||
type AccountFile struct { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there any specific reason to expose the struct ?
@@ -524,6 +562,7 @@ func (vm *VM) GetZoneList() ([]Zone, error) { | |||
Region: regionName, | |||
Status: zone.Status, | |||
}) | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete extra line.
} | ||
|
||
// Packer visor image config for GCP cloud | ||
type VisorImageConfig struct { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PackerConfig ?
@@ -225,11 +227,47 @@ type Project struct { | |||
CreateTime string `json:"create_time,omitempty"` | |||
} | |||
|
|||
// Packer visor image config for GCP cloud | |||
type GcpConfig struct { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GcpPackerConfig ?
// GetName returns the name of the virtual machine. | ||
func (vm *VM) GetName() string { | ||
return vm.Name | ||
} | ||
|
||
// Creates a visor image config required for packer | ||
func (vm *VM) CreateVisorImageConfig(accountFile, packerConf string) ([]byte, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we return only the gcp builder here and merge where we are dumping this in a file (c3ntry) ?
Jira-id
https://apporbit.atlassian.net/browse/MRPHS-5166
Problem
We need to support GCP cloud creation via c3ntry. In the new version of cloud creation keeper installation will be done via packer.
Resolution
Testing
In progress