Closed cynepco3hahue closed 2 years ago
@cynepco3hahue: This pull request references Bugzilla bug 2055318, which is invalid:
Comment /bugzilla refresh
to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.
hm it fails on operator installation with the error
- message: 'constraints not satisfiable: no operators found in package nfd in the
catalog referenced by subscription oo-mcbpf, subscription oo-mcbpf exists'
I will try to retest, but I am feeling it is some real problem.
/retest
/bugzilla refresh
@ArangoGutierrez: This pull request references Bugzilla bug 2055318, which is invalid:
Comment /bugzilla refresh
to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.
/retest-required
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/bugzilla refresh
Recalculating validity in case the underlying Bugzilla bug has changed.
@openshift-bot: This pull request references Bugzilla bug 2055318, 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.
Requesting review from QA contact: /cc @wabouhamad
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
/retest-required
Please review the full test history for this PR and help us cut down flakes.
I think I found the issue - https://github.com/openshift/cluster-nfd-operator/pull/243/commits/e0f91cca1127bd5a385eb20726c1417b1f3394b7
@cynepco3hahue: all tests passed!
Full PR test history. Your PR dashboard.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ArangoGutierrez, cynepco3hahue
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@cynepco3hahue: All pull requests linked via external trackers have merged:
Bugzilla bug 2055318 has been moved to the MODIFIED state.
In Telco far edge use case, a cluster might have limited management bandwidth. Upgrading such a cluster is expected to be time-limited (capped by a maintenance window) and will involve container image pre-caching on the node.
Image pre-caching solution for nodes with limited management bandwidth relies on the assumption, that workloads can use locally stored images without contacting a registry. This is possible if the container image pull policy is set to "IfNotPresent". The NFD operator is using Always as the pull policy, we should delete the policy and have the default behavior.
From https://kubernetes.io/docs/concepts/containers/images/#imagepullpolicy-defaulting
When you (or a controller) submit a new Pod to the API server, your cluster sets the image pull policy field when specific conditions are met:
if you omit the imagePullPolicy field, and the tag for the container image is :latest, imagePullPolicy is automatically set to Always
if you omit the imagePullPolicy field, and you don't specify the tag for the container image, imagePullPolicy is automatically set to Always;
if you omit the imagePullPolicy field, and you specify the tag for the container image that isn't :latest, the imagePullPolicy is automatically set to IfNotPresent.
Signed-off-by: Artyom Lukianov alukiano@redhat.com