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

live_manifest_incluster and yaml_incluster dont return YAML #18

Closed
techdragon opened this issue May 9, 2020 · 1 comment
Closed

live_manifest_incluster and yaml_incluster dont return YAML #18

techdragon opened this issue May 9, 2020 · 1 comment

Comments

@techdragon
Copy link

These two attributes seem to return some kind of concatenated string version of an internal map representation, which looks like fieldName:accessModes[0],fieldValue:ReadWriteOncefieldName:apiVersion,fieldValue:kubedb.com/v1alpha1fieldName:kind,fieldValue:Redis

They are either:

  • broken and not doing what they were meant to do.
  • documented wrong as the documentation says at least one of these should contain YAML explicitly and the other implicitly since you say it will contain the manifest (which is normally presented in YAML formal)

From what I could make out in the code, you seem to handle them differently from the data you use later to generate the yaml_body representation.

@gavinbunney
Copy link
Owner

These fields are used for drift detection, however I've recently pushed a complete rewrite of that (in v1.7.0) to fix up some issues, notably around duplicate keys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants