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 1801295: fix pod ring text while scaling up from 0 #4090
Conversation
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.
@nemesis09 This method has become a bit complicated and is in need of unit tests.
@nemesis09 I can still see the wrong text on pods |
/kind bug |
There is no test in place for this util. Can we capture the tests for the complete util in a separate automation story? |
fixed. PTAL |
@nemesis09 you don't have to right unit-tests for the whole file |
@nemesis09 original issue above seems to be fixed but we should log another issue regarding the spacing here: cc @openshift/team-devconsole-ux |
@nemesis09: This pull request references Bugzilla bug 1801295, which is invalid:
Comment 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. |
/bugzilla refresh |
@nemesis09: This pull request references Bugzilla bug 1801295, which is valid. The bug has been moved to the POST 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. |
Out of the gate here, your tests fail @nemesis09 |
@nemesis09 Did you address these issues? I can still reproduce the first screenshot just by putting something on Topology. /retitle [WIP] Bug 1801295: fix pod ring text while scaling up from 0 This looks like it's still under work, putting a WIP flag on it. |
this issue is being tracked in a separate ticket. https://issues.redhat.com/browse/ODC-3036 |
@nemesis09 ah indeed... fair enough. You just have a test to fix before we can give approvals. |
@nemesis09: This pull request references Bugzilla bug 1801295, which is valid. 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. |
2 similar comments
@nemesis09: This pull request references Bugzilla bug 1801295, which is valid. 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. |
@nemesis09: This pull request references Bugzilla bug 1801295, which is valid. 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. |
fixed failing tests. |
/test images |
@nemesis09 I dont think this issue is fixed. I can still see the double text on DC. |
I can't seem to reproduce this. I rebased with master but still failed to reproduce. Not sure where it is breaking. |
Yeah, This is fixed. I guess PR was not pulled properly |
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
verified changes locally.
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: christianvogt, nemesis09, sahil143 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@nemesis09: All pull requests linked via external trackers have merged. Bugzilla bug 1801295 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. |
This PR fixes https://issues.redhat.com/browse/ODC-2566
Analysis / Root cause:
Pod ring text shows two phrases when scaling from 0 to 1. Title is set to "scaled to 0" and Subtitle is set to "scaling to 1".
Solution description
checks status and spec for the workload are equal to set proper title for the case of scaling up from 0 to 1
Browser conformance:
Screens: