Enable to ignore manifests' fields from drift detection

[khanhtc1202]

What would you like to be added:

Why is this needed:

Some autogenerated fields in k8s manifests changed on each rerun, thus users want to ignore them from the drift detection feature. Would be best if we can make it available via application config.

[golemiso]

How about definining a new annotation pipecd.dev/ignore-drift-detection-on?

spec:
  template:
    metadata:
      annotations:
        deployedAt: {{ now | unixEpoch | quote }} # regenerate every run
        pipecd.dev/ignore-drift-detection-on: |
          ["spec.template.metadata.annotations.deployedAt"]

[khanhtc1202]

Thanks for the idea, but we think that this feature should be portable to other application kinds which pipecd currently supports, so maybe it's better to make this configuration a part of the pipecd application configuration (the app.pipecd.yaml file). wdyt? 🤔

[golemiso]

It could be more complicated when you specify fields out of the resource. You have to specify the resource name, kind and apiVersion since you can have several resources on one deployment.

[funera1]

TODO

• Support Kubernetes
• Support Cloud Run
• Support Lambda
• Support ECS
• Documentation

I don't think terraform's drift detection is necessary for this issue because it doesn't detect it in the manifest diff, so I won't support it for now.

[funera1]

ECS and Lambda have not yet implemented drift detection, so I will separate them into separate Issues.

[funera1]

By operating Kubernetes' ignoreManifests, and if it looks OK, I will implement CloudRun's.

[github-actions]

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This issue was closed because it has been stalled for 7 days with no activity. Feel free to reopen if still applicable.