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

Ignore changes to username and password in Diff() #484

Closed
guineveresaenger opened this issue Feb 8, 2023 · 1 comment · Fixed by #498
Closed

Ignore changes to username and password in Diff() #484

guineveresaenger opened this issue Feb 8, 2023 · 1 comment · Fixed by #498
Assignees
Labels
4.x.x kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Milestone

Comments

@guineveresaenger
Copy link
Contributor

As login credentials can change frequently, it should not be necessary to re-build the image every time this happens.

We should ignore these if they are the only changes in Diff().

@guineveresaenger guineveresaenger added 4.x.x kind/enhancement Improvements or new features labels Feb 8, 2023
@kmosher
Copy link

kmosher commented Feb 10, 2023

Tried using the ignoreChanges resource option to paper over this, but then the creds simply never get updated and end up expiring.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x.x kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants