-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
support Field Management (managedFields) #1880
Comments
Hi @alexec , Can you please review this issue? |
Hi @alexec , I can't work with |
hi @dorsany, I'm afraid I don't know much about this managed fields - is this something you'd be able to look into and suggest a solution please? |
Is there a particular reason the depth is set to 10 max ? This appears quite limiting. ---
metadata:
name: test
namespace: argo
labels:
example: 'true'
spec:
entrypoint: basic-service
templates:
- name: basic-service
ressources:
action: create
manifest:
apiVersion: v1
kind: Service
metadata:
name: basic-service
spec:
selector:
app: test
ports:
- protocol: TCP
port: 8080
targetPort: 8080 |
I could see that #1553 had also a proposal for URLs as input for manifest. Could it help if such feature was available ? |
https://kubernetes.io/docs/reference/using-api/server-side-apply/#clearing-managedfields |
Summary
In my k8s cluster I have the Field Management turned on for audit purposes.
The Field Management will add another
key
calledmanagedFields
undermetadata.managedFields
and thismanagedFields
can be more than depth (10) which is the limit inargo workflow
forWorkflowTemplate
example how the
WorkflowTemplate
it looks:and I'm getting the error:
What change you think needs making:
just ignore this key:
metadata.managedFields
Motivation
audit purposes.
Proposal
How do you think this should be implemented?
just ignore the key
metadata.managedFields
if this key exist, the question is where (context.go?validate.go?)or
increase max depth default value to be more than 10
more on: https://kubernetes.io/docs/reference/using-api/api-concepts/#field-management
The text was updated successfully, but these errors were encountered: