Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

[WIP] Add Calico e2e Test #2758

Closed
wants to merge 189 commits into from
Closed

[WIP] Add Calico e2e Test #2758

wants to merge 189 commits into from

Conversation

dtzar
Copy link
Contributor

@dtzar dtzar commented Apr 24, 2018

What this PR does / why we need it:
Adds an end-to-end test to validate kubernetes network policy is working when calico network policy is specified.

Add calico network policy e2e test

@ghost ghost assigned dtzar Apr 24, 2018
@ghost ghost added the in progress label Apr 24, 2018
weinong and others added 26 commits April 24, 2018 14:03
* defaults rbac to false when omitted

* fix UT
* initial commit

* master resource

* vmss name and vnet dependency

* revert master change

* added parameters for vmType and primaryScaleSetName

* instanceMetadata

* update vmss upgrade policy

* update vmss agent

* add osdisk and datadisk storage accounts

* add validation

* update example

* add managed identity

* windows vmss node

* update example

* windows update

* agent custom image

* update windows setup script

* disable overprovision

* update validation

* setting hostname dynamically on windows nodes

* add tests

* add validation for instanceMetadata

* update docs

* update tests
* use addon-resizer 1.8.1 for k8s 1.9 and 1.10

* make 1.9 and greater heapster config the default
* Add OpenShift on RHEL published image

* Add etcd disk only on k8s deployments

* Hook up plan in VMs

* Bindata updates

* Add CentOS image

* Set plan based on imagereference values and gate for openshift
* add node labels arg to windows setup script

* recommitting changes for visibility

* abstracting getAgentKubernetesLabels, adding value to parametersMap

* removing comment

* removing accidental AzProfile reference, and .Name property on agentProfile

* adding kubeletNodeLabels to base and params template files

* adding kubeletNodeLabels kubernetesmastervars.t

* fixing syntax issue with #, trying to reproduce template error in build

* node labels are working in generator, agentPoolProfile was in the ps1 context

* undoing changes to examples/kubernetes.json
* normalizeazureregion added in scale, upgrade, and deploy

* fixing linting errors

* go formatting files

* defining normalizeazurelocation

* getting rid of duplicate code

* refactoring helper function

* importing helpers in upgrade

* Added unit test for normalize azure region

* fixing linting errors

* fixing linting errors

* fixed linting errors

* fixed printf statement

* fixed printf to %s
* Add separate nsg for the router

* IsKubernetes -> not IsOpenShift
* enable cleanup script to continue despite errors
* Validate bindata drift

* Validate only openshift bindata.go

* Bindata updates
* add os type

* fix wording
* Add node selector for linux

* fix formating
* etcd encryption key

* add unit test
…ters (masters and agents) (#2782)

* working version with blank acsengineVersion in upgrader.go

* acsengine version changes for upgrade path

* modified unit tests to pass acsengineVersion

* incorporated initial review comments

* Added 'Checking VM tags' section to deploy document
axier and others added 21 commits May 30, 2018 23:01
* hardcode ve-ci.com search domain for testing

* change customdata template to update searchdomain and register dns registries on windows server dns

* add required types

* add functionality in acs-engine

* add params and variables in templates

* remove not required lines

* add custom search domain examples

* update clusterdefinition

* change sd prefix to searchDomain

* should be customSearchDomain in json

* move implementation to CSE

* always include searchDomain vars

* this is cleaner in a cloud-init-provided file

* cse func cruft and check for all vals
* add codecov yaml

* change threshold to 0.5
* acs-engine deploy: model validation and required fields in --help

* Forced apimodel validation to avoid incongruences (fixes #2025)

* Added more details about required fields while deploying (fixes #2491)

* WIP - Adding windows agent pool scaling

* Revert "WIP - Adding windows agent pool scaling" - wrong branch

This reverts commit 0eb7ece.

* WIP - Windows Scaling on Availability Sets and VMSS

* WIP - Naming format modified for vmss windows scaling

* Merge remote-tracking branch 'Azure/patch-release-v0.16.2' into windowsscale

* Fixing tests

* feat(*): Bumps client-go to v7.0.0

This updates client-go to v7.0.0 and adds the needed dependencies on
`k8s.io/api`. Also fixes a small issue with conflicting `uuid` library
versions

* testing re-vendor health

make build-vendor passed!

* removing the vnet from scale templates (#2994)

This allows a cluster to scale even if the user has added a subnet to the vnet

* Fix missing DefaultKubernetesClusterSubnet

* dos2unix

* Clean and go fmt

* Clean and go fmt

* Clean and go fmt

* Fix util_test.go

* Using StorageProfile.ImageReference.Publisher to understand if Windows Agent Pool in Scaling scenario

* lint
* Version artifacts gathering in openshift e2e

* Add orchestrator version in openshift e2e
* return err for autofill

* replace log fatal with return error

* check for err in unit tests

* fix typo

* lint

* log -> t
@acs-bot
Copy link

acs-bot commented Jun 1, 2018

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: jackfrancis

Assign the PR to them by writing /assign @jackfrancis in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@codecov
Copy link

codecov bot commented Jun 1, 2018

Codecov Report

Merging #2758 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2758   +/-   ##
=======================================
  Coverage   47.03%   47.03%           
=======================================
  Files          86       86           
  Lines       12715    12715           
=======================================
  Hits         5980     5980           
  Misses       6182     6182           
  Partials      553      553

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b37a65c...52ad7ac. Read the comment docs.

@ghost ghost assigned jackfrancis Jun 1, 2018
@jackfrancis
Copy link
Member

got into rebase hell, see #3146

@jackfrancis jackfrancis closed this Jun 1, 2018
@ghost ghost removed the in progress label Jun 1, 2018
@jackfrancis jackfrancis deleted the calico-e2e branch June 1, 2018 23:56
@dtzar
Copy link
Contributor Author

dtzar commented Jun 2, 2018

Understandable, bummed I haven't had a chance to pick this back up. Hope we can get it in the pipeline soon 👍

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

Successfully merging this pull request may close these issues.