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

Fixes unsolicited changes to the Schedule custom resource from globals #297

Merged
merged 1 commit into from
Jan 18, 2021

Conversation

cimnine
Copy link
Contributor

@cimnine cimnine commented Jan 15, 2021

Summary

When global defaults for resource limits and requests were defined,
the controller would add them to the user-provided Schedule instance.
This has been remedied.

Fixes #276

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking,
    as they show up in the changelog
  • Update the documentation.
  • Update tests.
  • Link this PR to related issues.

When global defaults for resource limits and requests were defined,
the controller would add them to the user-provided Schedule instance.
This has been remedied.
@cimnine cimnine added the bug Something isn't working label Jan 15, 2021
@cimnine cimnine marked this pull request as ready for review January 15, 2021 16:30
@cimnine cimnine requested a review from ccremer January 15, 2021 16:30
@cimnine cimnine changed the title Fixes the unsolicited changes to the Schedule custom resource Fixes unsolicited changes to the Schedule custom resource from globals Jan 15, 2021
@cimnine cimnine self-assigned this Jan 18, 2021
@ccremer ccremer added this to the v1.0.0-rc4 milestone Jan 18, 2021
Copy link
Contributor

@ccremer ccremer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested and approved!

@cimnine cimnine merged commit 013bcb1 into master Jan 18, 2021
@cimnine cimnine deleted the 276CantTouchThis branch January 18, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merged resources are stored in the wrong CR object
2 participants