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 1874817: fixes issue with route creation in case of no ports exposed #6494
Bug 1874817: fixes issue with route creation in case of no ports exposed #6494
Conversation
@invincibleJai: This pull request references Bugzilla bug 1874817, 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. 3 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. |
/assign @divyanshiGupta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/lgtm cancel |
@invincibleJai I just noticed that if we create a go app using the sample from the new samples catalog then the app is not created since now you have added a check for This can be fixed easily by adding |
d48058a
to
f7c0018
Compare
Thanks @divyanshiGupta , have updated as suggested |
Route creation happens successfully now, but while editing the go application I am not able to edit the |
f7c0018
to
1e46174
Compare
1e46174
to
d2290cb
Compare
Verified locally, Editing targetPort values, route creation and builder image detection for on Golang applications works as expected. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: invincibleJai, karthikjeeyar 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 |
@invincibleJai: All pull requests linked via external trackers have merged: Bugzilla bug 1874817 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. |
Fixes:
Analysis / Root cause:
Solution Description:
Browser conformance: