-
Notifications
You must be signed in to change notification settings - Fork 2
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: added connected clusters to the ocm view #43
Conversation
TomerFi
commented
May 19, 2022
- docs: typo
- feat: added view for local connected k8s clusters
- chore: finalize tree-view for ocm connected clusters
- chore: update views naming convention
Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
…w runners Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
e5b81f0
to
1944e94
Compare
@@ -16,7 +16,7 @@ async function sleep(ms: number): Promise<void> { | |||
// Test cases for the the ocm-vscode-extension.createLocalEnvironment command | |||
suite('Create local environment command', () => { | |||
const fakeBuildSuccessMsg = 'this is a fake message'; | |||
const buildEnvironmentDelayMS = 500; | |||
const buildEnvironmentDelayMS = 1000; |
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.
GH runners are much slower than my station...
https://github.com/ilan-pinto/ocm-vscode-extension/actions/runs/2350333292
refresh(): void { | ||
this.updateK8SConfig(); | ||
this.configRefreshedEmitter.fire(); | ||
} |
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.
fire?????
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.
this fires an event informing vscode on the data being refreshed.
https://code.visualstudio.com/api/references/vscode-api#EventEmitter%3CT%3E
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.
dor demo it looks good.
|
||
let k8sCoreApi = this.kubeConfig.makeApiClient(k8s.CoreV1Api); | ||
let namespacesApiResponse = await k8sCoreApi.listNamespace(); | ||
if (namespacesApiResponse.response.statusCode === 200) { |
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.
what about error handling?
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.
not for the demo, nither does testing.
for now, for error in k8s, an empty list is returned to not f-up the demo.
listResourcesPromises.push( | ||
k8sCustomObjApi.listClusterCustomObject(spec.group, ver.name, spec.names.plural) | ||
.then(crApiResponse => { | ||
if (crApiResponse.response.statusCode === 200) { |
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.
error handling
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.
not for the demo, nither does testing.
for now, for error in k8s, an empty list is returned to not f-up the demo.