Skip to content

[workspacekit] Fix empty JSON deserialization error#13192

Merged
roboquat merged 1 commit into
mainfrom
aledbf/fixrlimit
Sep 21, 2022
Merged

[workspacekit] Fix empty JSON deserialization error#13192
roboquat merged 1 commit into
mainfrom
aledbf/fixrlimit

Conversation

@aledbf

@aledbf aledbf commented Sep 21, 2022

Copy link
Copy Markdown
Contributor

Description

If GITPOD_RLIMIT_CORE env variable it's empty we should not attempt to deserialize an empty string

Related Issue(s)

Fixes https://cloudlogging.app.goo.gl/QYQWPUhwnjeTQiXD7

How to test

  • Open a workspace
  • Check there are no cannot deserialize GITPOD_RLIMIT_CORE errors.

Release Notes

NONE

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide

[workspacekit] Fix empty JSON deserialization error
@aledbf
aledbf requested a review from a team September 21, 2022 23:27
@github-actions github-actions Bot added the team: workspace Issue belongs to the Workspace team label Sep 21, 2022
@roboquat
roboquat merged commit 6bf56dc into main Sep 21, 2022
@roboquat
roboquat deleted the aledbf/fixrlimit branch September 21, 2022 23:34
@roboquat roboquat added deployed: workspace Workspace team change is running in production deployed Change is completely running in production labels Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployed: workspace Workspace team change is running in production deployed Change is completely running in production release-note-none size/S team: workspace Issue belongs to the Workspace team

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants