Jesse Hu

Thanks. Sound good! imgpkg will need to pull the two images, mount the data volume to local dir and run `diff` to get the result. Another format could be `imgpkg...

Got your pionts :D

Thanks @cppforlife. My point is Package.Spec.Template.Spec.SyncPeriod (if not nil, defined by the package auther) should be used when PackageInstall.Spec.SyncPeriod is nil. ``` desiredApp.Name = pkgInstall.Name desiredApp.Namespace = pkgInstall.Namespace desiredApp.Spec =...

I understand your point. Shall we add a hint on https://carvel.dev/kapp-controller/docs/v0.36.1/packaging/#package to explicitly describe only fetch/template/deploy fields from App CR should be specified in Package.Spec.Template.Spec? That doc currently says: ```...

just recall a possible use case that the package author might want to set Package.Spec.Template.Spec.SyncPeriod of package A to a shorter time as this package A depends on other packages,...

Thanks Joe. This feature request is not high priority for us. We have a work around. I will see if we can make the contribution.

A quick solution is adding the following command into cloud-init before the kubeadm command: ``` // Clear guestinfo.userdata after it's read by cloud-init to prevent security leaks. // There are...

Thanks @akutz for the notice. Make sense. We will need to find out the earliest time when CAPV controller can safely delete userdata, because the sooner the safer.

Thanks @akutz. I added some minor comments for https://github.com/vmware/cloud-init-vmware-guestinfo/pull/25, please take a look.

Hi @xcoulon could you share the sample code of your workaround please ? I hit the same issue.