Skip to content
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

Align the git services skip-oauth flag with oauth-1 providers #1020

Merged
merged 3 commits into from
Dec 21, 2023

Conversation

vinokurig
Copy link
Contributor

What does this PR do?

Use includes filter to handle the bitbucket-server oauth 2 provider. The bitbucket server oauth2 provider name is 'bitbucket', but the corresponding 'skip oauth' item is 'bitbucket-server'.

What issues does this PR fix or reference?

eclipse-che/che#22723

Is it tested? How?

  1. Deploy che from the che-server PR image: quay.io/eclipse/che-server:pr-624
  2. Register Bitbucket Server oauth2 provider.
  3. Start a workspace from the Bitbucket-Server repo
  4. Reject the Bitbucket-Server authorisation page.
  5. Go to User Preferences -> Git Services tab, see: the authorisation flag is set to skip authorisation warning.
  6. Click the three dot icon of the Bitbucket item and clear the skip authorisation flag.
  7. Delete the workspace and start the workspace from the Bitbucket-Server again, accept the authorisation page, see: the skip authorisation flag is set to authorised icon.

Release Notes

Docs PR

@che-bot
Copy link
Contributor

che-bot commented Dec 11, 2023

Click here to review and test in web IDE: Contribute

Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1020

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1020", name: che-dashboard}]}}]"

@olexii4
Copy link
Contributor

olexii4 commented Dec 11, 2023

@vinokurig Could you update snapshots?
yarn --cwd packages/dashboard-frontend test -u

Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1020

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1020", name: che-dashboard}]}}]"

Copy link

codecov bot commented Dec 11, 2023

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (d4ebc4a) 87.91% compared to head (346dc25) 87.99%.
Report is 6 commits behind head on main.

Files Patch % Lines
...src/pages/UserPreferences/GitServicesTab/index.tsx 40.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1020      +/-   ##
==========================================
+ Coverage   87.91%   87.99%   +0.07%     
==========================================
  Files         382      383       +1     
  Lines       39261    39321      +60     
  Branches     2593     2610      +17     
==========================================
+ Hits        34517    34599      +82     
+ Misses       4719     4697      -22     
  Partials       25       25              

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

Copy link
Contributor

@akurinnoy akurinnoy left a comment

Choose a reason for hiding this comment

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

LGTM

@artaleks9
Copy link
Contributor

Verified on Eclipse Che with quay.io/eclipse/che-server:pr-624 and quay.io/eclipse/che-dashboard:pr-1020 - works as expected.

Copy link

openshift-ci bot commented Dec 19, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: akurinnoy, artaleks9, olexii4, vinokurig

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot removed the lgtm label Dec 20, 2023
Copy link

openshift-ci bot commented Dec 20, 2023

New changes are detected. LGTM label has been removed.

Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1020

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1020", name: che-dashboard}]}}]"

@vinokurig vinokurig force-pushed the che-22723 branch 2 times, most recently from 6b7c2eb to 55f5626 Compare December 20, 2023 11:49
Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1020

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1020", name: che-dashboard}]}}]"

1 similar comment
Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1020

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1020", name: che-dashboard}]}}]"

Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1020

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1020", name: che-dashboard}]}}]"

@vinokurig vinokurig merged commit c256532 into main Dec 21, 2023
16 of 17 checks passed
@vinokurig vinokurig deleted the che-22723 branch December 21, 2023 12:04
@devstudio-release
Copy link

Build 3.12 :: dashboard_3.x/429: Console, Changes, Git Data

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.12 :: dashboard_3.x/429: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/5737 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.12 :: update-digests_3.x/5317: UNSTABLE

No new images detected: nothing to do!

@devstudio-release
Copy link

Build 3.12 :: dashboard_3.x/430: Console, Changes, Git Data

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.12 :: dashboard_3.x/430: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/5759 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.12 :: update-digests_3.x/5345: UNSTABLE

No new images detected: nothing to do!

@vinokurig
Copy link
Contributor Author

screenshot-eclipse-che apps rosa sqcr5-629wf-29x 6jy4 p3 openshiftapps com-2024 01 11-14_28_14
screenshot-eclipse-che apps rosa sqcr5-629wf-29x 6jy4 p3 openshiftapps com-2024 01 11-14_20_10

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.

6 participants