Skip to content

Commit

Permalink
this fixes two issues made manifest in testing the upgrade-e2e case s…
Browse files Browse the repository at this point in the history
…ubsequent to cutting the new RC.

1. [here](2f96c57#diff-b104edd120983248771684630cd3f66af28359c829df801a5eea77845d459aa0R33) an accidental label change in the v1 api promotion
2. [here](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]>
  • Loading branch information
grokspawn committed Nov 12, 2024
1 parent 2f96c57 commit 5ea03f5
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/upgrade/unpack_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ var _ = Describe("ClusterCatalog Unpacking", func() {
ctx := context.Background()

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

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

Expand Down

0 comments on commit 5ea03f5

Please sign in to comment.