Bug 2096691

Summary:	[Alibaba 4.11] Specifying ResourceGroup id in install-config.yaml, New pv are still getting created to default ResourceGroups
Product:	OpenShift Container Platform	Reporter:	Rohit Patil <ropatil>
Component:	Storage	Assignee:	Jan Safranek <jsafrane>
Storage sub component:	Storage	QA Contact:	Rohit Patil <ropatil>
Status:	CLOSED ERRATA	Docs Contact:
Severity:	medium
Priority:	unspecified	CC:	bihu, jiewu, jsafrane, ropatil, yqu
Version:	4.11
Target Milestone:	---
Target Release:	4.11.0
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2022-08-10 11:17:47 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Comment 3 Jan Safranek 2022-06-14 11:23:48 UTC

If I understand it correctly, alibaba-disk-csi-driver-operator should provide the group ID in csi-disk StorageClass:

parameters:
  resourceGroupId: <the resource group from install-config.yaml>

Comment 4 Wei Duan 2022-06-14 13:27:25 UTC

It works if we adding sc.parameters.resourceGroupId in the storageclass.

Here I'm not sure if our operator should do this (adding resourceGroupId in preset sc) or CSI Driver should default create volume in the same resource group with cluster.

Comment 5 Jan Safranek 2022-06-14 13:34:48 UTC

... and the resource group is part of infrastructure.status:


apiVersion: config.openshift.io/v1
kind: Infrastructure
metadata:
  name: cluster
spec:
  cloudConfig:
    key: config
    name: cloud-provider-config
  platformSpec:
    type: AlibabaCloud
status:
  [...]
  platform: AlibabaCloud
  platformStatus:
    alibabaCloud:
      region: us-east-1
      resourceGroupID: rg-acfnxrgr6qtm3mq
    type: AlibabaCloud

Comment 6 Jan Safranek 2022-06-14 14:10:30 UTC

> Here I'm not sure if our operator should do this (adding resourceGroupId in preset sc) or CSI Driver should default create volume 
> in the same resource group with cluster.

I think the operator should set the resource group in the default StorageClass, to have all cluster resources together.

Comment 13 Jan Safranek 2022-06-23 09:46:03 UTC

> Here the "resourceGroupID" should be "resourceGroupId" inside default sc.

Good catch! I should not have tested in the default resource group!

Comment 17 errata-xmlrpc 2022-08-10 11:17:47 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: OpenShift Container Platform 4.11.0 bug fix and security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:5069