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 1693180 - MachineSet and Machine pages need Event page for resources page consistency #2018
Conversation
I would skip this cherrypick on this one. We can fix in 4.2. You can still get to the events from Home -> Events. |
@jhadvig Can you confirm all of these resources really do generate events? |
@spadgett checked the |
@spadgett was talking to @runcom and @ingvagabund regarding |
@jhadvig sounds good to me |
Looks like you accidentally checked in a |
@@ -143,6 +144,7 @@ export const MachineAutoscalerDetailsPage: React.FC<MachineAutoscalerDetailsPage | |||
pages={[ | |||
navFactory.details(MachineAutoscalerDetails), | |||
navFactory.editYaml(), | |||
navFactory.events(ResourceEventStream), |
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.
Based on our discussion, let's take this out for now.
…ces page consistency
@spadgett PR updated. Will open a separate PR for the |
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
thanks @jhadvig
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dtaylor113, jhadvig, spadgett 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 |
2 similar comments
/retest |
/retest |
/assign @spadgett
The bug is referring only to
Machine
andMachineSet
resources but otherCompute
resource have been missing theEvents
tab so adding as well.I know this is 4.1 bug but lets add it here to the master as well and cherrypick to 4.1 branch.
PTAL