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
Resource owner now shown in every pane #7489
base: master
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: marcosdiez The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@marcosdiez CI doesn't pass. What do you mean by duplicate? |
68e9ebb
to
808864b
Compare
808864b
to
2d138e0
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #7489 +/- ##
==========================================
- Coverage 42.10% 41.94% -0.16%
==========================================
Files 217 163 -54
Lines 12084 10845 -1239
Branches 179 0 -179
==========================================
- Hits 5088 4549 -539
+ Misses 6711 6049 -662
+ Partials 285 247 -38 |
I'll get all the tests passed soon. Sorry for the inconvenient. |
2d138e0
to
95a06ec
Compare
@floreks @maciaszczykm do you guys know why the tests are failing ? The issue is caused by this line:
The part that the above line has an issue with is this one: but I am having trouble understanding why. Suggestions are welcome. Thank you! |
@@ -23,6 +26,8 @@ export class ObjectMetaComponent { | |||
@Input() initialized = false; | |||
|
|||
private objectMeta_: ObjectMeta; | |||
private readonly kdState_: KdStateService = GlobalServicesModule.injector.get(KdStateService); |
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.
Why not using constructor for that?
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.
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.
@maciaszczykm I tried
private readonly kdState_: KdStateService;
constructor() {
this.kdState_ = GlobalServicesModule.injector.get(KdStateService);
}
and it fails just like without the constructor.
mdiez@batman:~/code/dashboard/modules/web$ yarn test src/resource/discovery/ingress/detail/component.spec.ts
FAIL src/resource/discovery/ingress/detail/component.spec.ts
● Test suite failed to run
src/common/services/pluginloader/clientloader.service.ts:21:25 - error TS2339: Property 'System' does not exist on type 'Window & typeof globalThis'.
21 const systemJS = window.System;
~~~~~~
src/common/services/pluginloader/clientloader.service.ts:31:14 - error TS2339: Property 'define' does not exist on type 'Window & typeof globalThis'.
31 window.define(externalKey, [], () => {
~~~~~~
src/common/services/pluginloader/clientloader.service.ts:51:52 - error TS7006: Parameter 'm' implicitly has an 'any' type.
51 return systemJS.import(dependency.path).then(m => {
~
src/common/services/pluginloader/clientloader.service.ts:52:16 - error TS7015: Element implicitly has an 'any' type because index expression is not of type 'number'.
52 window['define'](dep, [], () => m.default);
~~~~~~~~
src/common/services/pluginloader/clientloader.service.ts:57:48 - error TS7006: Parameter 'module' implicitly has an 'any' type.
57 return systemJS.import(plugin.path).then(module => module.default.default);
~~~~~~
Test Suites: 1 failed, 1 total
I am out of ideas. I'll try again next week.
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.
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.
@maciaszczykm it did not work either :(
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.
@maciaszczykm I still need help here ....
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.
@marcosdiez Sorry, I do not have time this and perhaps next month to work on the project. Once I will find some I will take a look on it.
/remove-lifecycle stale |
@@ -930,6 +930,10 @@ | |||
<source>unset</source> | |||
<target>non défini</target> | |||
</trans-unit> | |||
<trans-unit id="3715596725146409911" datatype="html"> | |||
<source>Owner</source> | |||
<target state="new">Owner</target> |
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.
<target state="new">Owner</target> | |
<target state="new">Propriétaire</target> |
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
/remove-lifecycle stale |
@maciaszczykm hi. could we merge my PR sometime this year ? |
PR needs rebase. 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 is a duplicate of #6237
I am not sure if this is useful anywhere but in a
ReplicaSet
and in aJob
started by aCronJob
Also, we have the "problem" of having the info shown twice in Pods.@floreks , my team and I use this every day. It's quite useful. Consider merging, please!
This PR can be tested using the following containers: