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

Cleanup: render, validation, and defaulting #705

Merged
merged 5 commits into from
Jul 16, 2020

Conversation

caseydavenport
Copy link
Member

Description

  • Remove unnecessary intermedaite render config structure.
  • Streamline defaulting to account for a few more scenarios.
  • Remove some duplication in the validation logic.

For PR author

  • Tests for change.
  • If changing pkg/apis/, run make gen-files
  • If changing versions, run make gen-versions

For PR reviewers

A note for code reviewers - all pull requests must have the following:

  • Milestone set according to targeted release.
  • Appropriate labels:
    • kind/bug if this is a bugfix.
    • kind/enhancement if this is a a new feature.
    • enterprise if this PR applies to Calico Enterprise only.

@caseydavenport caseydavenport force-pushed the casey-calico-network-eks branch 3 times, most recently from 2ba0dd7 to 3d80a58 Compare July 16, 2020 17:39
Copy link
Member

@tmjd tmjd left a comment

Choose a reason for hiding this comment

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

LGTM

@caseydavenport caseydavenport merged commit df8d39e into tigera:master Jul 16, 2020
@caseydavenport caseydavenport deleted the casey-calico-network-eks branch July 16, 2020 20:34
@caseydavenport caseydavenport added this to the v1.9.0 milestone Jul 16, 2020
caseydavenport added a commit to caseydavenport/operator that referenced this pull request Jul 27, 2020
* Streamline defaulting and validation

* Remove intermediate netconfig structure

* Remove outdate render config_test.go

* Test fixups

* Generate files
caseydavenport added a commit that referenced this pull request Jul 27, 2020
* Cleanup: render, validation, and defaulting (#705)

* Streamline defaulting and validation

* Remove intermediate netconfig structure

* Remove outdate render config_test.go

* Test fixups

* Generate files

* Allow enabling / disabling BGP (#719)

* Changes for Calico networking on EKS (#743)

* Changes for Calico networking on EKS

* Add tests for Calico Network on EKS

* Fix tests after cherry-pick
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

Successfully merging this pull request may close these issues.

None yet

2 participants