Skip to content

BZ2067190: HPP error re: too-long FQDN #44069

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 1, 2022
Merged

BZ2067190: HPP error re: too-long FQDN #44069

merged 1 commit into from
Apr 1, 2022

Conversation

@ousleyp
Copy link
Member

@ousleyp ousleyp commented Mar 31, 2022 •
edited by openshift-ci bot
Loading

@ousleyp ousleyp added CNV Label for all CNV PRs branch/enterprise-4.10 labels Mar 31, 2022
@ousleyp ousleyp added this to the Next Release milestone Mar 31, 2022
@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 31, 2022
@netlify
Copy link

netlify bot commented Mar 31, 2022 •
edited
Loading

✅ Deploy Preview for osdocs ready!

Name Link
🔨 Latest commit f998f58
🔍 Latest deploy log https://app.netlify.com/sites/osdocs/deploys/6246fa7face9fd00088967f9
😎 Deploy Preview https://deploy-preview-44069--osdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@ousleyp ousleyp requested a review from duyanyan March 31, 2022 21:53
@duyanyan
Copy link

duyanyan commented Apr 1, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2022
@duyanyan duyanyan removed their assignment Apr 1, 2022
awels
awels reviewed Apr 1, 2022
View reviewed changes
sjhala-ccs
sjhala-ccs reviewed Apr 1, 2022
View reviewed changes
Copy link
Contributor

@sjhala-ccs sjhala-ccs left a comment

Choose a reason for hiding this comment

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

LGTM

@sjhala-ccs sjhala-ccs added the peer-review-done Signifies that the peer review team has reviewed this PR label Apr 1, 2022
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 1, 2022
@openshift-ci
Copy link

openshift-ci bot commented Apr 1, 2022

New changes are detected. LGTM label has been removed.

@ousleyp ousleyp merged commit cb8f072 into openshift:enterprise-4.10 Apr 1, 2022
@ousleyp ousleyp deleted the bz2067190 branch April 1, 2022 13:17
@awels
Copy link

awels commented Apr 1, 2022

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sjhala-ccs sjhala-ccs sjhala-ccs left review comments

@duyanyan duyanyan Awaiting requested review from duyanyan

+1 more reviewer

@awels awels awels left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

branch/enterprise-4.10 CNV Label for all CNV PRs peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

Next Release

Development

Successfully merging this pull request may close these issues.

4 participants

@ousleyp @duyanyan @awels @sjhala-ccs