Skip to content

Conversation

estroz
Copy link
Member

@estroz estroz commented May 6, 2021

This PR changes the SetDefaults function used in pkg/validation to the SetObjectDefaults function, which sets the entire object's defaults, ex. conversion webhook service port to 443.

/kind bug

Signed-off-by: Eric Stroczynski ericstroczynski@gmail.com

…ction

Signed-off-by: Eric Stroczynski <ericstroczynski@gmail.com>
@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label May 6, 2021
@estroz
Copy link
Member Author

estroz commented May 6, 2021

/cc @varshaprasad96

@estroz
Copy link
Member Author

estroz commented May 7, 2021

@varshaprasad96 you need to /lgtm

Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 7, 2021
@varshaprasad96
Copy link
Member

not sure if my lgtm will be accepted, I'm not in the reviewers list of the repo

@varshaprasad96
Copy link
Member

/approve

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: estroz, varshaprasad96
To complete the pull request process, please assign njhale after the PR has been reviewed.
You can assign the PR to them by writing /assign @njhale in a comment when ready.

The full list of commands accepted by this bot can be found 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

@estroz estroz merged commit 1c3cb2e into operator-framework:master May 10, 2021
@estroz estroz deleted the bugfix/crd-set-defaults branch May 10, 2021 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants