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
feat: add open created pod details #4499
Conversation
Can we have unit tests ? |
They are in progress |
@jeffmaury added the unit test for the handling "Open Pod" button |
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
f60635a
to
ff13415
Compare
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.
Re-ran failing E2E test and it passed. LGTM.
Signed-off-by: Vladyslav Zhukovskyi <vzhukovs@redhat.com>
@jeffmaury can we proceed to merge? |
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 for me but you should contact @benoitf is this should land in 1.7.0 |
What does this PR do?
Adds "Open Pod" button after deploying generated pod to the Kubernetes.
Screenshot/screencast of this PR
Additional button in the bottom right corner:
After clicking on "Open Pod" it navigates to the pod details:
What issues does this PR fix or reference?
fix #1470
How to test this PR?
Deploy existing container to the Kubernetes from the dropdown menu item "Deploy to Kubernetes", then when deployment is finished, an additional button should appear "Open Pod", which navigates to the newly deployed pod.