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 linearizability nightlies for release 3.4/3.5 #15106

Merged
merged 1 commit into from
Jan 16, 2023

Conversation

serathius
Copy link
Member

@serathius serathius force-pushed the linearizability-nightly-release branch 10 times, most recently from ad96b83 to 52dbc69 Compare January 14, 2023 10:33
@codecov-commenter
Copy link

Codecov Report

Merging #15106 (52dbc69) into main (d37f9b2) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #15106      +/-   ##
==========================================
- Coverage   74.68%   74.54%   -0.15%     
==========================================
  Files         415      415              
  Lines       34341    34341              
==========================================
- Hits        25649    25599      -50     
- Misses       7054     7101      +47     
- Partials     1638     1641       +3     
Flag Coverage Δ
all 74.54% <ø> (-0.15%) ⬇️

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

Impacted Files Coverage Δ
client/v3/namespace/watch.go 87.87% <0.00%> (-6.07%) ⬇️
server/storage/mvcc/watchable_store.go 84.42% <0.00%> (-5.08%) ⬇️
server/etcdserver/api/v3rpc/lease.go 77.21% <0.00%> (-5.07%) ⬇️
client/v3/leasing/cache.go 87.22% <0.00%> (-4.45%) ⬇️
server/etcdserver/api/v3rpc/util.go 70.96% <0.00%> (-3.23%) ⬇️
server/proxy/grpcproxy/watch.go 93.64% <0.00%> (-2.90%) ⬇️
server/etcdserver/api/v3rpc/watch.go 82.53% <0.00%> (-2.86%) ⬇️
server/etcdserver/api/v3rpc/interceptor.go 74.47% <0.00%> (-2.09%) ⬇️
server/lease/leasehttp/http.go 64.23% <0.00%> (-1.46%) ⬇️
pkg/adt/interval_tree.go 85.96% <0.00%> (-1.26%) ⬇️
... and 11 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment on lines +37 to +41
*)
make gofail-enable
make build
;;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
*)
make gofail-enable
make build
;;
main)
make gofail-enable
make build
./bin/etcd --version
;;
*)
echo "Invalid GITHUB_REF: ${GITHUB_REF}"
exit 1
;;

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's not correct because on PR we get GITHUB_REF=refs/pull/<pr_number>/merge

Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a minor comment, please also resolve the conflict.

This CL refactors the tests to reuse a single workflow that has
parameters. This is then reused for PRs/pushes and the nightlies.

Signed-off-by: Thomas Jungblut <tjungblu@redhat.com>
Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
@serathius serathius force-pushed the linearizability-nightly-release branch from 52dbc69 to ab3c530 Compare January 16, 2023 10:00
@serathius serathius merged commit 677e528 into etcd-io:main Jan 16, 2023
@serathius serathius deleted the linearizability-nightly-release branch June 15, 2023 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants