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

change of backend config #1125

Closed
wants to merge 1 commit into from
Closed

Conversation

TobiasKampmann
Copy link
Contributor

@TobiasKampmann TobiasKampmann added backend related to the loculus backend component config Configuration related issues, i.e. helm processing labels Feb 26, 2024
@TobiasKampmann TobiasKampmann added this to the MVP (closed beta) milestone Feb 26, 2024
Comment on lines +3 to +4
<option name="ALTERNATIVE_JRE_PATH" value="21" />
<option name="ALTERNATIVE_JRE_PATH_ENABLED" value="true" />
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this part does not work:
grafik

Copy link
Contributor Author

Choose a reason for hiding this comment

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

mmmhh... I cannot run without that. But maybe thats a local issue.

When this change is not necessary for anybody, I would close this PR. I can handle it locally then for this week.

@corneliusroemer
Copy link
Contributor

I'll convert this to draft PR as it's apparently not supposed to be merged per comments (makes it easier to spot review-ready PRs by looking for green)

@corneliusroemer corneliusroemer marked this pull request as draft February 26, 2024 18:12
@TobiasKampmann
Copy link
Contributor Author

As this seems only relevant for me, I will manage locally for the remaining days.

@chaoran-chen chaoran-chen deleted the change-in-backend-ide-config branch August 14, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend related to the loculus backend component config Configuration related issues, i.e. helm processing
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants