Skip to content

Commit 21ff2d6

Browse files
author
Power Cloud Robot
authored
Merge pull request #589 from varad-ahirwadkar/rename_service_instance
Renaming instance-id and instance-name
2 parents 3b388b0 + a1a8e5f commit 21ff2d6

File tree

30 files changed

+144
-109
lines changed

30 files changed

+144
-109
lines changed

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ The typical image workflow comprises of the following steps:
3636
- How to convert RHEL qcow2 to ova image format - [guide](docs/RHEL%20Qcow2%20to%20OVA.md)
3737
- How to convert RHCOS(Red Hat CoreOS) qcow2 to ova image format - [guide](docs/RHCOS%20Qcow2%20to%20OVA.md)
3838
- Advanced scenarios for Qcow2 to ova image conversion - [guide](docs/Advanced%20Scenarios%20for%20Qcow2%20to%20OVA.md)
39-
- How to import image to PowerVS instance from COS - [guide](docs/How%20to%20Import%20Image%20to%20PowerVS%20Instance.md)
39+
- How to import image to PowerVS workspace from COS - [guide](docs/How%20to%20Import%20Image%20to%20PowerVS%20Instance.md)
4040
- How to upload image to COS bucket using pvsadm - [guide](docs/How%20to%20Upload%20Image%20to%20COS.md)
4141
- How to build DHCP supported centos image - [guide](docs/Build%20DHCP%20enabled%20Centos%20Images.md)
4242

‎cmd/create/create.go‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ var Cmd = &cobra.Command{
3030

3131
func init() {
3232
Cmd.AddCommand(port.Cmd)
33-
Cmd.PersistentFlags().StringVarP(&pkg.Options.InstanceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
34-
_ = Cmd.MarkPersistentFlagRequired("instance-id")
33+
Cmd.PersistentFlags().StringVarP(&pkg.Options.WorkspaceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
34+
Cmd.PersistentFlags().MarkDeprecated("instance-id", "instance-id is deprecated, workspace-id should be used")
35+
Cmd.PersistentFlags().StringVarP(&pkg.Options.WorkspaceID, "workspace-id", "", "", "Workspace ID of the PowerVS workspace")
3536
}

‎cmd/create/port/port.go‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,12 @@ var Cmd = &cobra.Command{
3535
Use: "port",
3636
Short: "Create PowerVS network port",
3737
Long: `Create PowerVS network port`,
38+
PreRunE: func(cmd *cobra.Command, args []string) error {
39+
if pkg.Options.WorkspaceID == "" {
40+
return fmt.Errorf("--workspace-id required")
41+
}
42+
return nil
43+
},
3844
RunE: func(cmd *cobra.Command, args []string) error {
3945
opt := pkg.Options
4046

@@ -44,7 +50,7 @@ var Cmd = &cobra.Command{
4450
return err
4551
}
4652

47-
pvmclient, err := client.NewPVMClientWithEnv(c, opt.InstanceID, opt.InstanceName, opt.Environment)
53+
pvmclient, err := client.NewPVMClientWithEnv(c, opt.WorkspaceID, opt.WorkspaceName, opt.Environment)
4854
if err != nil {
4955
return err
5056
}

‎cmd/delete/delete.go‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ var Cmd = &cobra.Command{
3030

3131
func init() {
3232
Cmd.AddCommand(port.Cmd)
33-
Cmd.PersistentFlags().StringVarP(&pkg.Options.InstanceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
34-
_ = Cmd.MarkPersistentFlagRequired("instance-id")
33+
Cmd.PersistentFlags().StringVarP(&pkg.Options.WorkspaceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
34+
Cmd.PersistentFlags().MarkDeprecated("instance-id", "instance-id is deprecated, workspace-id should be used")
35+
Cmd.PersistentFlags().StringVarP(&pkg.Options.WorkspaceID, "workspace-id", "", "", "Workspace ID of the PowerVS instance")
3536
}

‎cmd/delete/port/port.go‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,12 @@ var Cmd = &cobra.Command{
3434
Use: "port",
3535
Short: "Delete PowerVS network port",
3636
Long: `Delete PowerVS network port`,
37+
PreRunE: func(cmd *cobra.Command, args []string) error {
38+
if pkg.Options.WorkspaceID == "" {
39+
return fmt.Errorf("--workspace-id required")
40+
}
41+
return nil
42+
},
3743
RunE: func(cmd *cobra.Command, args []string) error {
3844
opt := pkg.Options
3945

@@ -43,7 +49,7 @@ var Cmd = &cobra.Command{
4349
return err
4450
}
4551

46-
pvmclient, err := client.NewPVMClientWithEnv(c, opt.InstanceID, opt.InstanceName, opt.Environment)
52+
pvmclient, err := client.NewPVMClientWithEnv(c, opt.WorkspaceID, opt.WorkspaceName, opt.Environment)
4753
if err != nil {
4854
return err
4955
}

‎cmd/dhcp-sync/dhcp-sync.go‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ func syncDHCPD() {
8787
klog.Fatalf("failed to create a session with IBM cloud: %v", err)
8888
}
8989

90-
pvmclient, err := client.NewPVMClientWithEnv(c, pkg.Options.InstanceID, "", "prod")
90+
pvmclient, err := client.NewPVMClientWithEnv(c, pkg.Options.WorkspaceID, "", "prod")
9191
if err != nil {
9292
klog.Fatalf("failed to create a PVM client: %v", err)
9393
}
@@ -159,8 +159,8 @@ var Cmd = &cobra.Command{
159159
Long: `dhcp-sync tool is a tool populating the dhcpd.conf file from the PowerVS network and restart the dhcpd service.`,
160160
GroupID: "dhcp",
161161
PreRunE: func(cmd *cobra.Command, args []string) error {
162-
if pkg.Options.InstanceID == "" {
163-
return fmt.Errorf("--instance-id is required")
162+
if pkg.Options.WorkspaceID == "" {
163+
return fmt.Errorf("--workspace-id is required")
164164
}
165165
if pkg.Options.APIKey == "" {
166166
return fmt.Errorf("api-key can't be empty, pass the token via --api-key or set IBMCLOUD_API_KEY environment variable")
@@ -211,7 +211,9 @@ var Cmd = &cobra.Command{
211211
}
212212

213213
func init() {
214-
Cmd.Flags().StringVarP(&pkg.Options.InstanceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
214+
Cmd.Flags().StringVarP(&pkg.Options.WorkspaceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
215+
Cmd.Flags().MarkDeprecated("instance-id", "instance-id is deprecated, workspace-id should be used")
216+
Cmd.Flags().StringVarP(&pkg.Options.WorkspaceID, "workspace-id", "w", "", "Workspace ID of the PowerVS instance")
215217
Cmd.Flags().StringVar(&networkID, "network-id", "", "Network ID to be monitored")
216218
Cmd.Flags().StringVar(&file, "file", "/etc/dhcp/dhcpd.conf", "DHCP conf file")
217219
Cmd.Flags().StringVar(&gateway, "gateway", "", "Override the gateway value with")

‎cmd/dhcpserver/cmd.go‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ var Cmd = &cobra.Command{
3030
if pkg.Options.APIKey == "" {
3131
return fmt.Errorf("api-key can't be empty, pass the token via --api-key or set IBMCLOUD_API_KEY environment variable")
3232
}
33+
if pkg.Options.WorkspaceID == "" {
34+
return fmt.Errorf("--workspace-id required")
35+
}
3336
return nil
3437
},
3538
RunE: func(cmd *cobra.Command, args []string) error {
@@ -43,6 +46,7 @@ func init() {
4346
Cmd.AddCommand(getCmd)
4447
Cmd.AddCommand(deleteCmd)
4548

46-
Cmd.PersistentFlags().StringVarP(&pkg.Options.InstanceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
47-
_ = Cmd.MarkPersistentFlagRequired("instance-id")
49+
Cmd.PersistentFlags().StringVarP(&pkg.Options.WorkspaceID, "instance-id", "i", "", "Instance ID of the PowerVS instance")
50+
Cmd.PersistentFlags().MarkDeprecated("instance-id", "instance-id is deprecated, workspace-id should be used")
51+
Cmd.PersistentFlags().StringVarP(&pkg.Options.WorkspaceID, "workspace-id", "", "", "Workspace ID of the PowerVS instance")
4852
}

‎cmd/dhcpserver/create.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ var createCmd = &cobra.Command{
4545
return err
4646
}
4747

48-
pvmclient, err := client.NewPVMClientWithEnv(c, opt.InstanceID, opt.InstanceName, opt.Environment)
48+
pvmclient, err := client.NewPVMClientWithEnv(c, opt.WorkspaceID, opt.WorkspaceName, opt.Environment)
4949
if err != nil {
5050
return err
5151
}

‎cmd/dhcpserver/delete.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ var deleteCmd = &cobra.Command{
3838
return err
3939
}
4040

41-
pvmclient, err := client.NewPVMClientWithEnv(c, opt.InstanceID, opt.InstanceName, opt.Environment)
41+
pvmclient, err := client.NewPVMClientWithEnv(c, opt.WorkspaceID, opt.WorkspaceName, opt.Environment)
4242
if err != nil {
4343
return err
4444
}

‎cmd/dhcpserver/get.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ var getCmd = &cobra.Command{
4242
return err
4343
}
4444

45-
pvmclient, err := client.NewPVMClientWithEnv(c, opt.InstanceID, opt.InstanceName, opt.Environment)
45+
pvmclient, err := client.NewPVMClientWithEnv(c, opt.WorkspaceID, opt.WorkspaceName, opt.Environment)
4646
if err != nil {
4747
return err
4848
}

0 commit comments

Comments
 (0)