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

[kots]: add workaround for #8529 #8657

Merged
merged 1 commit into from
Mar 8, 2022
Merged

[kots]: add workaround for #8529 #8657

merged 1 commit into from
Mar 8, 2022

Conversation

mrsimonemms
Copy link
Contributor

Description

Workaround for #8529, using the advice of #8532

The Go library has a misconfiguration which adds in replicas and availableReplicas to the OpenVSX Proxy StatefulSet. This can cause validation issues dependent upon the version of Kubernetes that's being used.

This removes the block from the rendered Kubernetes objects which is the expected behaviour.

How to test

Run via KOTS

Release Notes

NONE

Documentation

@mrsimonemms mrsimonemms requested a review from a team March 8, 2022 10:58
The Go library has a misconfiguration which adds in "replicas"
and "availableReplicas" to the OpenVSX Proxy StatefulSet. This
can cause validation issues dependent upon the version of Kubernetes
that's being used.

This removes the block from the rendered Kubernetes objects which
is the expected behaviour.
@roboquat roboquat added the size/S label Mar 8, 2022
@github-actions github-actions bot added the team: delivery Issue belongs to the self-hosted team label Mar 8, 2022
@mrsimonemms mrsimonemms force-pushed the sje/kots-statefulset branch from e6b56e7 to fdf1358 Compare March 8, 2022 10:58
@mrsimonemms mrsimonemms requested a review from ArthurSens March 8, 2022 11:03
@mrsimonemms mrsimonemms marked this pull request as draft March 8, 2022 11:03
@mrsimonemms mrsimonemms marked this pull request as ready for review March 8, 2022 11:06
@codecov
Copy link

codecov bot commented Mar 8, 2022

Codecov Report

Merging #8657 (fdf1358) into main (f3a51dc) will decrease coverage by 3.38%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #8657      +/-   ##
==========================================
- Coverage   14.56%   11.17%   -3.39%     
==========================================
  Files          53       18      -35     
  Lines        5067      993    -4074     
==========================================
- Hits          738      111     -627     
+ Misses       4257      880    -3377     
+ Partials       72        2      -70     
Flag Coverage Δ
components-gitpod-cli-app 11.17% <ø> (ø)
components-local-app-app-darwin-amd64 ?
components-local-app-app-darwin-arm64 ?
components-local-app-app-linux-amd64 ?
components-local-app-app-linux-arm64 ?
components-local-app-app-windows-386 ?
components-local-app-app-windows-amd64 ?
components-local-app-app-windows-arm64 ?
components-ws-daemon-app ?
components-ws-daemon-lib ?
install-installer-raw-app ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
install/installer/pkg/common/storage.go
components/ws-daemon/pkg/content/initializer.go
install/installer/pkg/common/common.go
...installer/pkg/components/ws-manager/rolebinding.go
components/ws-daemon/pkg/content/service.go
components/ws-daemon/pkg/cpulimit/cfs_v2.go
components/ws-daemon/pkg/cpulimit/dispatch.go
install/installer/pkg/common/display.go
...onents/ws-daemon/pkg/internal/session/workspace.go
components/ws-daemon/pkg/daemon/cache_reclaim.go
... and 25 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3a51dc...fdf1358. Read the comment docs.

@roboquat roboquat merged commit 65cd121 into main Mar 8, 2022
@roboquat roboquat deleted the sje/kots-statefulset branch March 8, 2022 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note-none size/S team: delivery Issue belongs to the self-hosted team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants