Skip to content

Add check for OpenShift cluster when opening developer console#2987

Merged
msivasubramaniaan merged 1 commit intoredhat-developer:mainfrom
JessicaJHee:2628-check-os
Jun 28, 2023
Merged

Add check for OpenShift cluster when opening developer console#2987
msivasubramaniaan merged 1 commit intoredhat-developer:mainfrom
JessicaJHee:2628-check-os

Conversation

@JessicaJHee
Copy link
Copy Markdown
Member

Fixes #2628

@JessicaJHee JessicaJHee marked this pull request as draft June 23, 2023 19:50
@JessicaJHee
Copy link
Copy Markdown
Member Author

@mohitsuman I wasn't able to reproduce the error popup since I believe we no longer use oc get project. But I think it still makes sense to not show the option to open the developer console in that case.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 23, 2023

Codecov Report

Patch coverage has no change and project coverage change: -1.76 ⚠️

Comparison is base (48ed462) 37.59% compared to head (43a31c3) 35.83%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2987      +/-   ##
==========================================
- Coverage   37.59%   35.83%   -1.76%     
==========================================
  Files          62       63       +1     
  Lines        3937     4152     +215     
  Branches      755      830      +75     
==========================================
+ Hits         1480     1488       +8     
- Misses       2457     2664     +207     
Impacted Files Coverage Δ
src/explorer.ts 29.82% <0.00%> (-1.37%) ⬇️
src/odo/command.ts 21.25% <0.00%> (-0.35%) ⬇️

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mohitsuman
Copy link
Copy Markdown
Contributor

@mohitsuman I wasn't able to reproduce the error popup since I believe we no longer use oc get project. But I think it still makes sense to not show the option to open the developer console in that case.

Yes we should add a check and only show Developer Console for an OpenShift cluster.

@JessicaJHee JessicaJHee marked this pull request as ready for review June 26, 2023 13:17
Comment thread package.json Outdated
Comment thread src/odo/command.ts
Signed-off-by: Jessica He <jhe@redhat.com>
Copy link
Copy Markdown
Collaborator

@msivasubramaniaan msivasubramaniaan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@msivasubramaniaan msivasubramaniaan merged commit 3ed826e into redhat-developer:main Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Open in Developer console fails for kubernetes applications

3 participants