Skip to content

Commit 4d6327f

Browse files
authored
this fixes two issues made manifest in testing the upgrade-e2e case subsequent to cutting the new RC. (#457)
1. [here](operator-framework/catalogd@2f96c57#diff-b104edd120983248771684630cd3f66af28359c829df801a5eea77845d459aa0R33) an accidental label change in the v1 api promotion 2. [here](operator-framework/catalogd@2f96c57#diff-b104edd120983248771684630cd3f66af28359c829df801a5eea77845d459aa0R69) a location which I failed to update for the new Progressing==True logic for happy path clustercatalog unpacking. This was masked due to the fact that we knew we were making breaking changes, and the upgrade-e2e test will fail until we have cut a new release. 😞 Signed-off-by: Jordan Keister <[email protected]>
1 parent 2f96c57 commit 4d6327f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/upgrade/unpack_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ var _ = Describe("ClusterCatalog Unpacking", func() {
3030
ctx := context.Background()
3131

3232
var managerDeployment appsv1.Deployment
33-
managerLabelSelector := labels.Set{"control-plane": "catalogdv1-controller-manager"}
33+
managerLabelSelector := labels.Set{"control-plane": "catalogd-controller-manager"}
3434
By("Checking that the controller-manager deployment is updated")
3535
Eventually(func(g Gomega) {
3636
var managerDeployments appsv1.DeploymentList
@@ -66,13 +66,13 @@ var _ = Describe("ClusterCatalog Unpacking", func() {
6666
Expect(found).To(BeTrue())
6767

6868
catalog := &catalogdv1.ClusterCatalog{}
69-
By("Ensuring ClusterCatalog has Status.Condition of Progressing with a status == False, reason == Succeeded")
69+
By("Ensuring ClusterCatalog has Status.Condition of Progressing with a status == True, reason == Succeeded")
7070
Eventually(func(g Gomega) {
7171
err := c.Get(ctx, types.NamespacedName{Name: testClusterCatalogName}, catalog)
7272
g.Expect(err).ToNot(HaveOccurred())
7373
cond := meta.FindStatusCondition(catalog.Status.Conditions, catalogdv1.TypeProgressing)
7474
g.Expect(cond).ToNot(BeNil())
75-
g.Expect(cond.Status).To(Equal(metav1.ConditionFalse))
75+
g.Expect(cond.Status).To(Equal(metav1.ConditionTrue))
7676
g.Expect(cond.Reason).To(Equal(catalogdv1.ReasonSucceeded))
7777
}).Should(Succeed())
7878

0 commit comments

Comments
 (0)