Marcus Zoller

🐛 ClusterVersion CRD gets rejected on k8 v1.20.2

3

The ClusterVersion CRD is using apiVersion v1beta1 which requires properties in x-kubernetes-list-map-keys to be required (validation was introduced with https://github.com/kubernetes/kubernetes/pull/88076/commits/2e187415fd582ea80bc316fb76cb30c3ad1aead4). v1 supports defaults too. A manual rebuild with a newer...

🐛 Unable to create a VirtualCluster on k8 v1.20.2

9

## Problem Virtual cluster does not deploy with k8 v1.20.2. Output from vc-manager: > {"level":"info","ts":1621457612.1333222,"logger":"clusterversion-controller","msg":"reconciling ClusterVersion..."} > {"level":"info","ts":1621457612.1334903,"logger":"clusterversion-controller","msg":"new ClusterVersion event","ClusterVersionName":"cv-sample-np"} > {"level":"info","ts":1621457635.4177175,"logger":"virtualcluster-webhook","msg":"validate create","vc-name":"vc-sample-1"} > {"level":"info","ts":1621457635.4421399,"logger":"virtualcluster-controller","msg":"reconciling VirtualCluster..."} > {"level":"info","ts":1621457635.4824774,"logger":"virtualcluster-webhook","msg":"validate update","vc-name":"vc-sample-1"} >...

I am wondering how snapshot size limits are configured or if this is still unsupported. I can see Usage was implemented with https://github.com/containerd/zfs/pull/38 but if I am not mistaken maxSnapshotSize...

Wrong encoding used for basic Authorization header

1

The header is generated by url encoding the username and password before base64 encoding. This is wrong and may result in authentication errors with special characters within username or credentials....