Use OpenShift console timestamp component

[KevinFCormier]

Makes the <Timestamp /> component from OpenShift console available and simplifies the current mechanism for using console dynamic plugin SDK hooks and components while maintaining the ability to run in standalone mode.

The <Timestamp /> component gives us an easy way to have consistent display of timestamps that are localized.

Placeholder for standalone dev mode:

Demonstrating translated date/time string (French) and tooltip, with icon removed:

Relative time display (once the current bug is implemented):

TODOs:

• Implement a simplified version of the component for standalone development use
• Implement a wrapper component to handle removing the icon (which we will not want to use in a table context)
• Contribute a fix for the broken relative string support (see draft PR Fix Timestamp relative display openshift/console#14555)

[vishsanghishetty]

@KevinFCormier, thanks for making the changes, these look good to me

[vishsanghishetty]

@KevinFCormier, please let me know what you think of the to-dos implementation

[vishsanghishetty]

@KevinFCormier I just updated with a date formatting function.

[KevinFCormier]

Thanks @vishsanghishetty
I made a couple small changes like using en-US instead of en-GB and making "09:30 AM" display as "9:30 AM" without the leading zero.

[KevinFCormier]

/cc @Randy424

[Randy424]

/lgtm

[Randy424]

/hold

Holding for development train.

[vishsanghishetty]

@KevinFCormier and @Randy424 please review, reverted the changes in ManagedClusters component

[KevinFCormier]

/lgtm

[openshift-ci]

@KevinFCormier: you cannot LGTM your own PR.

In response to this:

/lgtm

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-sigs/prow repository.

[sonarqubecloud]

Quality Gate failed

Failed conditions
17.2% Coverage on New Code (required ≥ 70%)

See analysis details on SonarQube Cloud

[openshift-ci]

@KevinFCormier: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/unit-tests-sonarcloud	2087106	link	true	/test unit-tests-sonarcloud

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

[Randy424]

Will re-review once this goes green.
Assuming we're ok with the test coverage, I think this will still need to be merged with an override. @KevinFCormier

[Randy424]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: KevinFCormier, Randy424

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:

• OWNERS [KevinFCormier,Randy424]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Randy424]

/unhold