Skip to content

Why is the dependency on vscode-kubernetes added during release? #3091#3170

Merged
datho7561 merged 1 commit intoredhat-developer:mainfrom
vrubezhny:fix-add-vscode-k8s-tools-as-extension-dependency
Sep 6, 2023
Merged

Why is the dependency on vscode-kubernetes added during release? #3091#3170
datho7561 merged 1 commit intoredhat-developer:mainfrom
vrubezhny:fix-add-vscode-k8s-tools-as-extension-dependency

Conversation

@vrubezhny
Copy link
Copy Markdown
Contributor

Fixes: #3091

@vrubezhny vrubezhny marked this pull request as draft September 1, 2023 20:24
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 1, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.26% 🎉

Comparison is base (943a3c7) 37.03% compared to head (08b3407) 37.30%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3170      +/-   ##
==========================================
+ Coverage   37.03%   37.30%   +0.26%     
==========================================
  Files          75       75              
  Lines        5265     5265              
  Branches     1000     1000              
==========================================
+ Hits         1950     1964      +14     
+ Misses       3315     3301      -14     

see 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Looks good so far, integration tests are passing for me. You will need to remove the part in the Jenkinsfile that adds the Kubernetes extension to the package.json

@vrubezhny
Copy link
Copy Markdown
Contributor Author

vrubezhny commented Sep 1, 2023

Looks good so far, integration tests are passing for me.

Yes, but have you tried running tests from vscode run "Extension Debug Tests" on Debug view?

You will need to remove the part in the Jenkinsfile that adds the Kubernetes extension to the package.json

Sure I'll remove this part

@vrubezhny vrubezhny force-pushed the fix-add-vscode-k8s-tools-as-extension-dependency branch from 0deebe9 to 457125c Compare September 1, 2023 22:26
@datho7561
Copy link
Copy Markdown
Contributor

The tests suites seem debuggable to me, but I needed to add the following:

diff --git a/.vscode/launch.json b/.vscode/launch.json
index 7f70119d..227fe9be 100644
--- a/.vscode/launch.json
+++ b/.vscode/launch.json
@@ -74,7 +74,8 @@
             "sourceMaps": true,
             "preLaunchTask": "instrument",
             "env": {
-                "VSCODE_REDHAT_TELEMETRY_DEBUG":"true"
+                "VSCODE_REDHAT_TELEMETRY_DEBUG": "true",
+                "IS_OPENSHIFT": "true",
             }
         },
         {

for the integration test suite, since I'm using an OpenShift cluster

@vrubezhny vrubezhny force-pushed the fix-add-vscode-k8s-tools-as-extension-dependency branch from 457125c to 08b3407 Compare September 5, 2023 22:00
@vrubezhny
Copy link
Copy Markdown
Contributor Author

vrubezhny commented Sep 5, 2023

The tests suites seem debuggable to me, but I needed to add the following:

diff --git a/.vscode/launch.json b/.vscode/launch.json
...

for the integration test suite, since I'm using an OpenShift cluster

Added "IS_OPENSHIFT": "true" to launch.json

@vrubezhny vrubezhny marked this pull request as ready for review September 5, 2023 22:14
@vrubezhny vrubezhny requested a review from datho7561 September 5, 2023 22:14
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks, Victor!

@datho7561 datho7561 merged commit b72fdc6 into redhat-developer:main Sep 6, 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.

Why is the dependency on vscode-kubernetes added during release?

2 participants