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

Full support for projected volumes #113

Open
ldx opened this issue May 31, 2020 · 0 comments
Open

Full support for projected volumes #113

ldx opened this issue May 31, 2020 · 0 comments
Labels
conformance enhancement New feature or request good first issue Good for newcomers

Comments

@ldx
Copy link
Contributor

ldx commented May 31, 2020

Right now Kip only supports configmap and secret sources:

// Projection that may be projected along with other supported volume types
type VolumeProjection struct {
	// all types below are the supported types for projection into the same volume

	// information about the secret data to project
	// +optional
	Secret *SecretProjection `json:"secret,omitempty"`
	// // information about the downwardAPI data to project
	// // +optional
	// DownwardAPI *DownwardAPIProjection `json:"downwardAPI,omitempty"`
	// information about the configMap data to project
	// +optional
	ConfigMap *ConfigMapProjection `json:"configMap,omitempty"`
	// information about the serviceAccountToken data to project
	// +optional
	//ServiceAccountToken *ServiceAccountTokenProjection `json:"serviceAccountToken,omitempty"`
}

Once service account token rotation is enabled, service accounts will be added to pods via a ServiceAccountToken projected volume source. Example:

- name: kube-api-access-tz9tt
  projected:
    defaultMode: 420
    sources:
    - serviceAccountToken:
        expirationSeconds: 3600
        path: token
    - configMap:
        items:
        - key: ca.crt
          path: ca.crt
        name: kube-root-ca.crt
    - downwardAPI:
        items:
        - fieldRef:
            apiVersion: v1
            fieldPath: metadata.namespace
          path: namespace

So in-cluster API server access configuration will break for pods.

Currently, ServiceAccountTokenVolumeProjection is beta in 1.12 and enabled by passing all of the following flags to the API server:

  • --service-account-issuer
  • --service-account-signing-key-file
  • --service-account-api-audiences

We need to implement DownwardAPIProjection and ServiceAccountTokenVolumeProjection.

@ldx ldx added the enhancement New feature or request label May 31, 2020
@hidalgopl hidalgopl added the good first issue Good for newcomers label Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conformance enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants