Skip to content
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

OCPBUGS-3779: support only image or version in ClusterVersion when doing ocp precache #399

Conversation

pixelsoccupied
Copy link
Contributor

@pixelsoccupied pixelsoccupied commented Dec 7, 2022

  • support clusterVersion CR with only Image or only version or both.
  • reduce unnecessary calls by making:
    - single external only after CR validation
    - no calls if user provides the Spec.DesiredUpdate.image
  • unit tests

/cc @jc-rh @fontivan

@openshift-ci openshift-ci bot requested review from fontivan and jc-rh December 7, 2022 17:13
@openshift-ci-robot
Copy link

@pixelsoccupied: This pull request references Jira Issue OCPBUGS-3779, which is invalid:

  • expected the bug to target the "4.13.0" version, but no target version was set

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

/cc @jc-rh @fontivan

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.

@pixelsoccupied pixelsoccupied changed the title OCPBUGS-3779: support only image or version in ClusterVersion when doing ocp precache [wip] OCPBUGS-3779: support only image or version in ClusterVersion when doing ocp precache Dec 7, 2022
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 7, 2022
@openshift-ci-robot
Copy link

@pixelsoccupied: This pull request references Jira Issue OCPBUGS-3779, which is invalid:

  • expected the bug to target the "4.13.0" version, but no target version was set

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

  • support clusterVersion CR with only Image or only version or both.
  • reduce unnecessary calls by making:
    - single external only after CR validation
    - no calls if user provides the Spec.DesiredUpdate.image
  • unit tests

/cc @jc-rh @fontivan

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.

@pixelsoccupied
Copy link
Contributor Author

/retest

@pixelsoccupied pixelsoccupied force-pushed the precache-support-only-image-or-version-ocp branch from cbfe858 to 2b00e52 Compare December 12, 2022 18:12
@pixelsoccupied pixelsoccupied changed the title [wip] OCPBUGS-3779: support only image or version in ClusterVersion when doing ocp precache OCPBUGS-3779: support only image or version in ClusterVersion when doing ocp precache Dec 12, 2022
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 12, 2022
@pixelsoccupied
Copy link
Contributor Author

/jira refresh

@openshift-ci-robot
Copy link

@pixelsoccupied: This pull request references Jira Issue OCPBUGS-3779, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.13.0) matches configured target version for branch (4.13.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @yliu127

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

/jira refresh

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.

@openshift-ci
Copy link

openshift-ci bot commented Dec 13, 2022

@openshift-ci-robot: GitHub didn't allow me to request PR reviews from the following users: yliu127.

Note that only openshift-kni members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

@pixelsoccupied: This pull request references Jira Issue OCPBUGS-3779, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.13.0) matches configured target version for branch (4.13.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @yliu127

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

/jira refresh

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.

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.

@jc-rh
Copy link
Member

jc-rh commented Dec 13, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 13, 2022
@pixelsoccupied pixelsoccupied force-pushed the precache-support-only-image-or-version-ocp branch from 2b00e52 to 7f09a0c Compare December 13, 2022 22:20
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Dec 13, 2022
…ing ocp precache

refactor code to support cases and add unit tests

add http mock reply when trying to make external calls

remove the extra check

update logic to handle when CV cr is not present

improve check for spec and add additional test make sure ClusterVersion is present

add const to where policies are changed, add another test make sure that empty strings are ok and make polcies minimal

make the upstearm test url more streamlined and obvious that no external calls are being made to test
@pixelsoccupied pixelsoccupied force-pushed the precache-support-only-image-or-version-ocp branch from 7f09a0c to bdfb0e1 Compare December 13, 2022 22:29
@pixelsoccupied pixelsoccupied requested review from fontivan and removed request for jc-rh December 13, 2022 22:31
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 14, 2022
@openshift-ci
Copy link

openshift-ci bot commented Dec 14, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jc-rh

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 14, 2022
@openshift-merge-robot openshift-merge-robot merged commit 07c0a39 into openshift-kni:main Dec 14, 2022
@openshift-ci-robot
Copy link

@pixelsoccupied: All pull requests linked via external trackers have merged:

Jira Issue OCPBUGS-3779 has been moved to the MODIFIED state.

In response to this:

  • support clusterVersion CR with only Image or only version or both.
  • reduce unnecessary calls by making:
    - single external only after CR validation
    - no calls if user provides the Spec.DesiredUpdate.image
  • unit tests

/cc @jc-rh @fontivan

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.

@pixelsoccupied
Copy link
Contributor Author

/cherrypick release-4.12

@openshift-cherrypick-robot

@pixelsoccupied: new pull request created: #400

In response to this:

/cherrypick release-4.12

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants