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

Flexible pipeline kind test #23

Closed
wants to merge 17 commits into from
Closed

Conversation

hongliangl
Copy link
Owner

No description provided.

Jexf and others added 17 commits December 6, 2021 10:35
…#3083)

There was an obvious bug where all NetworkPolicy resources
(NetworkPolicies, AddressGroups, AppliedToGroups) were dumped to the
same file, and the file itself ("agentinfo") was subsequently
overwritten with a dump of the appropriate AgentInfo CRD as expected.

This is fixed by dumping these resources to their respective appropriate
files: networkpolicies, addressgroups, appliedtogroups.

In addition, we:
* switch from JSON to YAML for these resources (along with the AgentInfo
  CRD) for consistency with the files included in the Controller's
  supportbundle.
* define a custom MarshalYAML function for the IPAddress type. IPAddress
  is typedef'ed to a byte slice, which means that by default it is
  marshalled as an array of individual byte values, making the files
  difficult to read.

Fixes antrea-io#3082

Signed-off-by: Antonin Bas <[email protected]>
Add throttle setting property in project definition will be
overwritten by template, so add a new template for integration jobs
so we can define throttle concurrency for them.

Signed-off-by: Lan Luo <[email protected]>
Introduce Antrea interface type to map the OVS port to Antrea created
interfaces, and use OVS port external_ids field to save the
configuration. This is helpful for Antrea to define more interface
types.

Signed-off-by: wenyingd <[email protected]>
Decouple ExternalIPPool controller and MemberList controller from Egress controller.

Signed-off-by: Xu Liu <[email protected]>
The protocol must be written in lowercase, otherwise the validation
would fail.

Signed-off-by: Quan Tian <[email protected]>
Signed-off-by: Hongliang Liu <[email protected]>
Signed-off-by: Hongliang Liu <[email protected]>
Signed-off-by: Hongliang Liu <[email protected]>
Signed-off-by: Hongliang Liu <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #23 (dd223bd) into main (0a60d73) will decrease coverage by 37.03%.
The diff coverage is 38.52%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #23       +/-   ##
===========================================
- Coverage   60.79%   23.76%   -37.04%     
===========================================
  Files         292      292               
  Lines       24766    24411      -355     
===========================================
- Hits        15057     5801     -9256     
- Misses       8063    17866     +9803     
+ Partials     1646      744      -902     
Flag Coverage Δ
kind-e2e-tests 23.76% <38.52%> (-24.26%) ⬇️
unit-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/agent/agent_linux.go 3.62% <0.00%> (-0.06%) ⬇️
pkg/agent/apiserver/handlers/ovsflows/handler.go 0.00% <ø> (-75.00%) ⬇️
pkg/agent/controller/egress/egress_controller.go 0.00% <0.00%> (-38.33%) ⬇️
pkg/agent/controller/traceflow/packetin.go 0.00% <0.00%> (-59.26%) ⬇️
pkg/agent/flowexporter/exporter/exporter.go 0.00% <0.00%> (-81.30%) ⬇️
pkg/agent/interfacestore/types.go 59.09% <ø> (-13.64%) ⬇️
pkg/agent/ipassigner/ip_assigner_linux.go 0.00% <ø> (ø)
pkg/agent/ipassigner/local_ip_detector_linux.go 0.00% <ø> (ø)
pkg/agent/memberlist/cluster.go 0.00% <0.00%> (-72.25%) ⬇️
pkg/agent/openflow/client_other.go 20.00% <0.00%> (-20.00%) ⬇️
... and 192 more

@hongliangl hongliangl closed this Apr 27, 2022
@hongliangl hongliangl deleted the flexible-pipeline-kind-test branch April 28, 2022 11:59
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.

9 participants