Skip to content

Conversation

ccbayer
Copy link

@ccbayer ccbayer commented Aug 18, 2020

No description provided.

abhiraut and others added 30 commits July 16, 2020 10:58
Add Phase as part of `kubectl get` output. In addition to that, add source-pod, destination-pod and destination-ip as
wide option output for `kubectl get -o`.
* Enhance testing for prometheus metrics at agent

Added testing for following metrics in integration tests:
antrea_agent_ovs_total_flow_count
antrea_agent_ovs_flow_count
antrea_agent_local_pod_count

Fixes#799

* Changes to Makefile to run integration test on MacOS

When running integration test on MacOS, hit an error because of an old
docker image for antrea/openvswitch:2.13.0. Pulling that docker image
explicitly which is used as base image for test container image.

* Addressed latest comments
Go, kind, integration CI can be triggered when there's a PR or push on ipv6 branch.
SubscribePacketIn requires a receiver to pop data in ch timely, otherwise
it will block all inbound messages from OVS.
Support handle LoadBalancer Service traffic:
- Install loadbalancing OpenFlow entires for Pod to LoadBalancer
  Service traffic. The traffic handling logic is same with the
  existing logic for ClusterService.
- Install OpenFlow entires for LoadBalancer Service traffic from
  outside. The traffic is forwarded antrea gateway, and then
  kube-proxy will handle the traffic.
- Add new OpenFlow table "UplinkTable" to place OpenFlow entries
  which are used to handle packtes received from uplink port.

Signed-off-by: Rui Cao <[email protected]>
pgrep is provided in Ubuntu and Debian by default. pidof is provided by
the sysvinit-utils package, which is not available in Debian. Use of
pgrep will allow downstream to repackage Antrea agent into various
distributions and still re-using the E2E test.
- Update the links to cloud CI with the latest ones using scripts managed by Antrea
- Bump up GKE cluster version
- Number of OVS flow operations, partitioned by operations(add, modify and delete)
- Number of OVS flow operation errors, partitioned by operations(add, modify and delete)
- The latency of OVS flow operations, partitioned by operations(add, modify and delete)

Signed-off-by: Yuki Tsuboi <[email protected]>
…a-io#959)

When initialize the agent network on Windows node, the IP
configuration of uplink interface will be moved to OVS bridge
local interface. Which will cause the routes configured on
the uplink interface are lost.

This patch saves the routes configured on the uplink interface
before the IP configuration change and adds the saved routes on
OVS bridge local interface after that.

Signed-off-by: Rui Cao <[email protected]>
Support for batch OVS flow updates

This is an optimization for NetworkPolicy OVS flows to be collectively installed in agent restart case, so that those
flows will rolled out at once and avoid potential priority re-assignments.
In the future, mechanisms can also be added so that once this operation is done, NetworkPolicy flows from the
previous round are deleted immediately after.
* IP as destination, UDP, intra node
* IP as destination, ICMP, inter nodes
Ever since goccy/go-graphviz was added as a dependency, running "make
golangci" (and related targets) has been broken on macOS. This is
because goccy/go-graphviz requires cgo to be enabled and go disables it
when cross-compiling (we set GOOS to linux in "make golangci" to ensure
that files with a linux build tag are verified by the linters).

We now run "make golangci" from a macOS worker in CI to avoid breaking
this again in the future.

In the future, we should also ensure that the linters are run on
Windows-specific files.

This commit also adds a few "missing" doc.go files.
The test was incorrect (traffic is dropped by an ingress policy
rule). However, this only became apparent after antrea-io#981 was merged and the
DMAC of the Traceflow packet started being set correctly.
OVS supports to match DSCP field in OpenFlow, and the field is shown
as "nw_tos" in OVS command line. Besides, the TOS value is shifting
the DSCP value left 2 bits.
* add support for ovs hardware offload

* support ovs hw-offload config in antrea image

* add ovs-offload doc

Fixes antrea-io#630
We enable the staticcheck and gosec linters in golangci-lint.
The most important one here is gosec, as it will be useful when
answering questions for the CII Best Practices self-certification
questionnaire (see antrea-io#989).

Running these linters did not reveal any noticeable issue in the Antrea
codebase.
Add document to guide users how to
use Traceflow via Antrea UI and Kubectl.

Related: antrea-io#901
Azure CNI operates in L2 (bridge) mode when no network policy
is specified. With a help of antrea node init, we toggle the
mode from bridge to transparent (L3).

Verified reboot and auto scale of nodes.

Co-authored-by: Rahul Jain <[email protected]>
- Make e2e prometheus test consistent by querying target metadata API

Signed-off-by: Yuki Tsuboi <[email protected]>
goimports makes imports deterministic, putting them in 3 sections:
builtin, 3rd-party, local packages. It saves reviewers's effort to
comment the order of imports.

It doesn't allow imports named "v1" unless the alias name is specified
as "v1", this patch adds the parent packages of v1 packages as the
prefix of the import names.
McCodeman and others added 30 commits August 18, 2020 15:49
* Fixed ordered list formatting in `docs/manual-install.md`
* Used markdown image placement instead of `<img/>` to ensure
relative paths processed correctly in jekyll
* updated all code blocks (specified formatters, ensured correct
spacing and indenting)
* updated additional TAG version number references
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
* updated all links to be relative for compatibility
  with jekyll markdown to html converter
* fixed docs navigation to allow for external links
* Updated docs ToC for kind to "Deploy to kind"
* Fixed issue reference in kind.md
* Updated index page to advertise release v0.5.1
* Fixed ordered lists in manual installation
* Fixed ordered list formatting in `docs/manual-install.md`
* Used markdown image placement instead of `<img/>` to ensure
relative paths processed correctly in jekyll
* updated all code blocks (specified formatters, ensured correct
spacing and indenting)
* updated additional TAG version number references
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
Signed-off-by: Colin Bayer <[email protected]>
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.