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

[RFE] Remove unused fields #817

Closed
git-hyagi opened this issue Dec 1, 2022 · 0 comments · Fixed by #1061
Closed

[RFE] Remove unused fields #817

git-hyagi opened this issue Dec 1, 2022 · 0 comments · Fixed by #1061
Labels
enhancement New feature or request go-alpha Go-based operator issues

Comments

@git-hyagi
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
We have some fields that are not being used.
Some examples are:

StorageType string `json:"storage_type,omitempty"`

InstanceName string `json:"instance_name"`

Describe the solution you'd like
We could double-check if all the fields defined in CRD are in use and remove the ones that aren't.

Additional context
Before removing the fields we should consider reviewing them in ansible version to get a better understanding of why ansible oeprator had them and if we are missing a feature/functionality.

@git-hyagi git-hyagi added enhancement New feature or request go-alpha Go-based operator issues labels Dec 1, 2022
git-hyagi added a commit to git-hyagi/pulp-operator that referenced this issue Aug 24, 2023
git-hyagi added a commit that referenced this issue Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request go-alpha Go-based operator issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant