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

Skiperator creates empty Sidecar-resources #23

Closed
eliihen opened this issue Aug 11, 2022 · 1 comment
Closed

Skiperator creates empty Sidecar-resources #23

eliihen opened this issue Aug 11, 2022 · 1 comment

Comments

@eliihen
Copy link
Contributor

eliihen commented Aug 11, 2022

The sidecar resource gets the following spec:

  spec:
    outboundTrafficPolicy: {}

This is potentially a bug in the enum from the library we use. The enum value is 0 and this may be recieved as a null value somewhere and not properly mapped to REGISTRY_ONLY in the YAML. It is set in the code: https://github.com/kartverket/skiperator/blob/main/controllers/sidecar.go#L53

@hagen93
Copy link
Contributor

hagen93 commented Aug 25, 2022

REGISTRY_ONLY is the default value and thus removed by the library. Looks weird, but works just fine as default is picked up on the other side too. I tested this while developing the feature.

@hagen93 hagen93 closed this as completed Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants