Skip to content

Conversation

@eguzki
Copy link
Member

@eguzki eguzki commented Apr 19, 2022

what

Fix func (apimanager *APIManager) IsSystemMysqlEnabled() bool. System mysql pod template labels were not upgraded

verification steps

Deploy 3scale 2.11 with system mysql. System mysql pod labels are:

$ oc get "$(oc get pods --selector deploymentconfig=system-mysql -o name)" -o jsonpath='{.metadata.labels}' | yq e -P
app: 3scale-api-management
com.redhat.component-name: system-mysql
com.redhat.component-type: application
com.redhat.component-version: "57"
com.redhat.product-name: 3scale
com.redhat.product-version: "2.11"
deployment: system-mysql-1
deploymentConfig: system-mysql
deploymentconfig: system-mysql
threescale_component: system
threescale_component_element: mysql

Run upgrade to 2.12 procedure. For this step, I just removed the operator and make run the version from this PR branch. Check system mysql pod labels:

$ oc get "$(oc get pods --selector deploymentconfig=system-mysql -o name)" -o jsonpath='{.metadata.labels}' | yq e -P
app: 3scale-api-management
com.company: Red_Hat
deployment: system-mysql-2
deploymentConfig: system-mysql
deploymentconfig: system-mysql
rht.comp: 3scale
rht.comp_ver: master
rht.prod_name: Red_Hat_Integration
rht.prod_ver: master
rht.subcomp: system-mysql
rht.subcomp_t: application
threescale_component: system
threescale_component_element: mysql

Labels have been upgraded

@eguzki eguzki requested a review from sergioifg94 April 19, 2022 13:13
@eguzki
Copy link
Member Author

eguzki commented Apr 19, 2022

/retest

@eguzki eguzki force-pushed the fix-mysql-pod-labels branch from ba42b99 to 6bd8b5b Compare April 20, 2022 17:01
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 6bd8b5b and detected 0 issues on this pull request.

View more on Code Climate.

@eguzki
Copy link
Member Author

eguzki commented Apr 20, 2022

/retest

@sergioifg94
Copy link
Contributor

Verified.

Labels before upgrading:

app: 3scale-api-management
com.redhat.component-name: system-mysql
com.redhat.component-type: application
com.redhat.component-version: "57"
com.redhat.product-name: 3scale
com.redhat.product-version: "2.11"
deployment: system-mysql-1
deploymentConfig: system-mysql
deploymentconfig: system-mysql
threescale_component: system
threescale_component_element: mysql

After upgrading

app: 3scale-api-management
com.company: Red_Hat
deployment: system-mysql-3
deploymentConfig: system-mysql
deploymentconfig: system-mysql
rht.comp: 3scale
rht.comp_ver: master
rht.prod_name: Red_Hat_Integration
rht.prod_ver: master
rht.subcomp: system-mysql
rht.subcomp_t: application
threescale_component: system
threescale_component_element: mysql

@eguzki eguzki merged commit fa3d82b into master Apr 21, 2022
@eguzki eguzki deleted the fix-mysql-pod-labels branch April 21, 2022 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants