Skip to content

Scope config lookup to org #19216

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

Closed
wants to merge 1 commit into from
Closed

Conversation

filiptronicek
Copy link
Member

@filiptronicek filiptronicek commented Dec 7, 2023

Description

When creating a workspace, we check if by any chance there aren't multiple configurations for this repository, because we cannot just guess it by the context URL you provided. This lookup was previously done on a global level, so if anyone in the world configured a repo which you also had configured, you would start getting a Multiple projects found for clone URL. error.

This change tweaks the behavior so that even if you had a repo configured across two organizations, only the one for the currently active one will apply.

Summary generated by Copilot

🤖[deprecated] Generated by Copilot at 203f6a8

Added an organizationId option to the findProjectsByCloneUrl method in various classes to support filtering projects by clone URL and organization. This improves the accuracy and reliability of workspace context creation.

Related Issue(s)

Fixes EXP-962

How to test

Preview status

Gitpod was successfully deployed to your preview environment.

Build Options

Build
  • /werft with-werft
    Run the build with werft instead of GHA
  • leeway-no-cache
  • /werft no-test
    Run Leeway with --dont-test
Publish
  • /werft publish-to-npm
  • /werft publish-to-jb-marketplace
Installer
  • analytics=segment
  • with-dedicated-emulation
  • workspace-feature-flags
    Add desired feature flags to the end of the line above, space separated
Preview Environment / Integration Tests
  • /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 preemptible
    Saves cost. Untick this only if you're really sure you need a non-preemtible machine.
  • with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh. If enabled, with-preview and with-large-vm will be enabled.
  • with-monitoring

@filiptronicek filiptronicek force-pushed the ft/org-scoped-config-lookup branch from 4590baa to 508715d Compare December 7, 2023 19:09
@filiptronicek
Copy link
Member Author

filiptronicek commented Dec 7, 2023

/gh run recreate-vm=true

Comment triggered a workflow run

Started workflow run: 7133375257

  • recreate_vm: true

@filiptronicek filiptronicek self-assigned this Dec 7, 2023
@filiptronicek
Copy link
Member Author

Superseeded by #19217

@filiptronicek filiptronicek deleted the ft/org-scoped-config-lookup branch January 24, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants