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

Release v2.2.0-beta #1434

Merged
merged 120 commits into from
Aug 13, 2019
Merged

Release v2.2.0-beta #1434

merged 120 commits into from
Aug 13, 2019

Conversation

ondrej-fabry
Copy link
Member

No description provided.

Milan Lenco and others added 30 commits May 9, 2019 09:47
Signed-off-by: Milan Lenco <[email protected]>
Signed-off-by: AndrejKilvady <[email protected]>
* Add etcd ansible python plugin
* Add example ansible playbook
* Fix indentation
* Update example playbook

Signed-off-by: miroslav.kovac <[email protected]>
* Fix *-ifplugin config names
* Unset default stats publishers used as fallback without config

Signed-off-by: Ondrej Fabry <[email protected]>
Signed-off-by: Milan Lenco <[email protected]>
Signed-off-by: Ondrej Fabry <[email protected]>
* Refactor image builds to use ligato/vpp-base

Signed-off-by: Ondrej Fabry <[email protected]>
Refactoring Rx-placement and Rx-mode
ondrej-fabry and others added 16 commits July 23, 2019 11:32
* Optimize interface state dump for specific interfaces

- define env var for printing debug logs about interface states
- extend interface state info in vppcalls for ifplugin
- return error when loading config fails
- minor improvements for VPP integration testing

Signed-off-by: Ondrej Fabry <[email protected]>

* Remove obsolete function

Signed-off-by: Ondrej Fabry <[email protected]>
* Added CRUD integration tests for IPv4/IPv6 routes in default VRF and VRF=2

Signed-off-by: Stanislav Chlebec <[email protected]>

* improving code

Signed-off-by: Stanislav Chlebec <[email protected]>

* removed comments and parametrized value of VRF (vrfMetaIdx)

* removed redundant initialisation to default value

causes warnings in travis

Signed-off-by: Stanislav Chlebec <[email protected]>
* Minor enhancement for transaction summary log

Signed-off-by: Ondrej Fabry <[email protected]>

* Add make target for remote debugging with delve

Signed-off-by: Ondrej Fabry <[email protected]>

* Minor code cleanup in vpp ifplugin

Signed-off-by: Ondrej Fabry <[email protected]>

* Fix error check when renaming tap and cleanup code

Signed-off-by: Ondrej Fabry <[email protected]>
* added arp crud integration test

Signed-off-by: Stanislav Chlebec <[email protected]>

* comments removed

* added more asserts

- asserts related to number of arp entries in arp dump
- asserting of fields of arp entry
- assert non presence in arp dump
- hard-coded index replaced by generated one stored in the variable ifIdx

Signed-off-by: Stanislav Chlebec <[email protected]>

* simplify assert (omitting assert of Static field)

Signed-off-by: Stanislav Chlebec <[email protected]>
This patch fixes issues reported in #1418.
During recreate (Delete+Create) the node would be completely
removed from the graph by Delete and all the associated flags
(metadata) were therefore lost.
When the subsequent Create fails, the scheduler needs to access
the flags, which in this case are undefined, causing the scheduler
to panic by dereferencing nil pointer.
This patch ensures that the node and its flags are preserved
during re-creation.

An in-progress refactor of the scheduling algorithm
will approach value recreation in a much cleaner
way...

Signed-off-by: Milan Lenco <[email protected]>
* add "EXISTING" interface type

Signed-off-by: Matus Mrekaj <[email protected]>

* update interface descriptor

Signed-off-by: Matus Mrekaj <[email protected]>

* update interface descriptor

Signed-off-by: Matus Mrekaj <[email protected]>
* integration test ACL IP rule CRUD

Signed-off-by: Stanislav Chlebec <[email protected]>

* renamed to 040_acl_test.go

Signed-off-by: Stanislav Chlebec <[email protected]>

* to satisfy lint

Signed-off-by: Stanislav Chlebec <[email protected]>

* Trying to loop over acl dump

* Looping over IP rule

* acl IP rule CRUD almost finished

Signed-off-by: Stanislav Chlebec <[email protected]>

* acl IP rule CRUD

Signed-off-by: Stanislav Chlebec <[email protected]>

* satisfy lint

* added acl integration test for MAC IP rules
fixed comments in interface definition

Signed-off-by: Stanislav Chlebec <[email protected]>

* fix comments - they should start with space

Signed-off-by: Stanislav Chlebec <[email protected]>

* create a helper function that returns ACL rule
- for IP rule
- for IP MAC rule

Signed-off-by: Stanislav Chlebec <[email protected]>

* acl integration test - refactoring
- introducing of helper functions to eliminate declaration of acl_rule variables
- removing needless convert with uint32
- simplifying code with expect functions
- improving logged messages to improve understanding

Signed-off-by: Stanislav Chlebec <[email protected]>

* satisfy lint

Signed-off-by: Stanislav Chlebec <[email protected]>
Signed-off-by: Ondrej Fabry <[email protected]>
Signed-off-by: Ondrej Fabry <[email protected]>

# Conflicts:
#	CHANGELOG.md
#	Gopkg.lock
#	api/models/vpp/vpp.pb.go
#	cmd/vpp-agent/main.go
#	docker/dev/Dockerfile
#	docker/dev/linux-ifplugin.conf
#	docker/prod/Dockerfile
#	docker/prod/linux-ifplugin.conf
#	docker/prod/vpp-ifplugin.conf
#	plugins/configurator/dump.go
rastislavs
rastislavs previously approved these changes Aug 9, 2019
@VladoLavor VladoLavor merged commit 2c193f2 into master Aug 13, 2019
ondrej-fabry added a commit to ondrej-fabry/vpp-agent that referenced this pull request Aug 20, 2019
@ondrej-fabry ondrej-fabry deleted the release/2.2.x branch December 12, 2019 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants