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

Add e2e for augerctl #122

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

wzshiming
Copy link
Member

@wzshiming wzshiming commented Sep 17, 2024

This e2e will be tested by make test

Run make test
Vetting
go vet ./...
Testing
go test ./...
?   	github.com/etcd-io/auger	[no test files]
?   	github.com/etcd-io/auger/augerctl	[no test files]
?   	github.com/etcd-io/auger/augerctl/command	[no test files]
?   	github.com/etcd-io/auger/hack/tools	[no test files]
ok  	github.com/etcd-io/auger/cmd	0.021s
ok  	github.com/etcd-io/auger/pkg/client	0.008s
ok  	github.com/etcd-io/auger/pkg/data	0.016s
?   	github.com/etcd-io/auger/pkg/scheme	[no test files]
ok  	github.com/etcd-io/auger/pkg/encoding	0.013s
ok  	github.com/etcd-io/auger/tests/e2e	79.733s

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: wzshiming
Once this PR has been reviewed and has the lgtm label, please assign siyuanfoundation for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@wzshiming wzshiming force-pushed the e2e-framework branch 3 times, most recently from a5bfdd8 to 15de23c Compare September 17, 2024 06:48
@wzshiming
Copy link
Member Author

/cc @jmhbnz
/cc @siyuanfoundation

Signed-off-by: Shiming Zhang <wzshiming@hotmail.com>
@wzshiming
Copy link
Member Author

wzshiming commented Nov 11, 2024

This e2e also needs to do what, please take a look.

There seems to be something wrong with k8s.io/apimachinery@v0.31.2. It renames a field but does not rename the corresponding usage.

../../../go/pkg/mod/k8s.io/apimachinery@v0.31.2/pkg/util/managedfields/internal/structuredmerge.go:56:4: unknown field IgnoredFields in struct literal of type merge.Updater

fixed by kubernetes/kubernetes@2bc17d1#diff-1966d58e97bf87f3cd69e4957a2c948a72ca97c36df3ea099ac0ea6592e42dbfR57

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

Successfully merging this pull request may close these issues.

2 participants