New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug 1934637: Honor all subscriptions to a package #2026
Bug 1934637: Honor all subscriptions to a package #2026
Conversation
@benluddy: This pull request references Bugzilla bug 1934637, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: benluddy The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold |
ResolutionFailed events exposed internal terminology (i.e. "mandatory" and "dependency") directly to users. Common constraints applied to subscriptions appeared as "PACKAGE is mandatory" or "PACKAGE has a dependency without any candidates to satisfy it", which in practice meant "a subscription to PACKAGE exists in the namespace" and "no operators found matching the criteria of the subscription to PACKAGE", respectively. The language of these Subscription-related constraint strings now better reflects what they mean to users. Signed-off-by: Ben Luddy <bluddy@redhat.com>
During dependency resolution, invariants are enforced for one-operator-per-package and one-provider-per-api. Their user-facing string representation (shown when they prevent resolution from succeeding) had re-used the same representation as constraints on Subscriptions out of convenience. Recently, the messaging of Subscription-related constraints was changed to be more descriptive which inadvertently made the invariant messaging misleading. This patch improves constraint messaging for all dependency relationships except bundle dependencies. Signed-off-by: Ben Luddy <bluddy@redhat.com>
there are no longer two resolvers, so having a separate shared set of test cases doesn't make sense (and means the resolver tests look different from all of the other unit tests) Signed-off-by: Evan <cordell.evan@gmail.com>
without this, some of the invariant checking was being missed by the unit tests Signed-off-by: Evan <cordell.evan@gmail.com>
this better aligns the on-cluster status with the output of the resolver we may want to prevent this case from happening at a higher layer (i.e. a webhook that prevents two subscriptions to the same package), but for now this behavior ensures that the actions the resolver takes are clearly visible in the subscription status. this also adds test cases for multiple subscriptions to the same package Signed-off-by: Evan <cordell.evan@gmail.com>
e463dc3
to
ed1e00b
Compare
/bugzilla refresh |
@benluddy: This pull request references Bugzilla bug 1934637, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/hold cancel |
/lgtm |
/lgtm |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
6 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
CI failed six times in a row. @benluddy could you take a look? |
Hmmm, aside from the first failure, these seem to be related to a recent CI outage. |
/retest |
2 similar comments
/retest |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@benluddy: All pull requests linked via external trackers have merged: Bugzilla bug 1934637 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Cherry-pick of #2009 to release-4.6.