Skip to content

[supervisor] fix open access control not working when git permission … #16987

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

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

mustard-mh
Copy link
Contributor

@mustard-mh mustard-mh commented Mar 23, 2023

Description

[supervisor] fix open access control not working when git permission missed

  • Update runAsGitpodUser to append correct env
  • Fix .gitignore make whole supervisor component unsearchable

Related Issue(s)

Fixes internal chat

How to test

Preview env https://hw-fix-acc7a9a838b10.preview.gitpod-dev.com/workspaces

  • Open preview env and access /user/integrations to remove all optional permission of git
  • Open a workspace
  • Exec git push
  • It should show Open Access Control notification and click should go to right place (unless you blocked pop-up)

Release Notes

Fix `Open Access Control` not working when git permission missing

Documentation

Build Options:

  • /werft with-werft
    Run the build with werft instead of GHA
  • leeway-no-cache
  • /werft no-test
    Run Leeway with --dont-test
Publish Options
  • /werft publish-to-npm
  • /werft publish-to-jb-marketplace
Installer Options
  • with-ee-license
  • with-dedicated-emulation
  • with-ws-manager-mk2
  • workspace-feature-flags
    Add desired feature flags to the end of the line above, space separated

Preview Environment Options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-gce-vm
    If enabled this will create the environment on GCE infra
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh

@mustard-mh mustard-mh requested a review from a team March 23, 2023 06:30
@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-hw-fix-access-control.1 because the annotations in the pull request description changed
(with .werft/ from main)

Copy link
Member

@akosyakov akosyakov left a comment

Choose a reason for hiding this comment

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

code looks good, i did not try

/hold

unhold when you are ready to merge

…missed

- Update `runAsGitpodUser` to append correct env
- Fix .gitignore make whole supervisor component unsearchable
@mustard-mh mustard-mh force-pushed the hw/fix-access-control branch from b195590 to 1d92c50 Compare March 24, 2023 11:17
@mustard-mh
Copy link
Contributor Author

All good

/unhold

@roboquat roboquat merged commit 260497b into main Mar 24, 2023
@roboquat roboquat deleted the hw/fix-access-control branch March 24, 2023 13:48
@roboquat roboquat added deployed: IDE IDE change is running in production deployed Change is completely running in production labels Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: IDE IDE change is running in production deployed Change is completely running in production release-note size/L team: IDE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants