Skip to content

remove supervisor_live_git_status #18921

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
Oct 16, 2023
Merged

remove supervisor_live_git_status #18921

merged 1 commit into from
Oct 16, 2023

Conversation

akosyakov
Copy link
Member

@akosyakov akosyakov commented Oct 13, 2023

Description

We deployed live git status in all installations for some time already. This PR removes supervisor_live_git_status and deprecated instance.status.repo.

This PR does not remove though status capture by ws-manager on stop.

Summary generated by Copilot

🤖 Generated by Copilot at 027403c

This pull request removes the deprecated and unused code related to the repo property and the supervisor_live_git_status feature flag from various components and files. It also simplifies the usage of the gitStatus property to reflect the live git status of the workspace instances.

Related Issue(s)

Fixes #

How to test

  • Start a workspace, change branch, do some commits, go to the dashboard check that changes are reflected.
  • Stop the workspace, check that changes are still reflected.
  • Restart the workspace, check that changes are still reflected. Do more changes.

Documentation

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
  • 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

/hold

@geropl
Copy link
Member

geropl commented Oct 13, 2023

Code LGTM, waiting for the preview.

and deprecated instance.status.repo
@akosyakov akosyakov force-pushed the ak/always_live_git_status branch from 6de97c5 to 5ecc217 Compare October 13, 2023 15:34
@akosyakov
Copy link
Member Author

akosyakov commented Oct 16, 2023

/gh run recreate-vm=true

Comment triggered a workflow run

Started workflow run: 6530360668

  • recreate_vm: true

@akosyakov akosyakov requested a review from geropl October 16, 2023 08:57
@akosyakov
Copy link
Member Author

@geropl I tested it.

@geropl
Copy link
Member

geropl commented Oct 16, 2023

Testing now...

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

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

Changes LGTM, tested and works ✔️

Copy link
Contributor

@mads-hartmann mads-hartmann left a comment

Choose a reason for hiding this comment

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

LGTM

@akosyakov
Copy link
Member Author

/unhold

@roboquat roboquat merged commit cf7d0e9 into main Oct 16, 2023
@roboquat roboquat deleted the ak/always_live_git_status branch October 16, 2023 14:20
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.

4 participants