Skip to content

[MINOR] chore(ci): Fix K8S CI pipeline - #1227

Merged
roryqi merged 1 commit into
apache:masterfrom
roryqi:ci
Oct 9, 2023
Merged

[MINOR] chore(ci): Fix K8S CI pipeline#1227
roryqi merged 1 commit into
apache:masterfrom
roryqi:ci

Conversation

@roryqi

@roryqi roryqi commented Oct 9, 2023

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Use a fixed version instead of latest version.

Why are the changes needed?

Fix the ci error of master branch

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI passed.

@roryqi roryqi changed the title [MINOR] chore(ci): Fix kubernetest CI pipeline [MINOR] chore(ci): Fix K8S CI pipeline Oct 9, 2023
@codecov-commenter

codecov-commenter commented Oct 9, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1227 (45dcbca) into master (98b0d12) will decrease coverage by 1.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1227      +/-   ##
============================================
- Coverage     54.55%   53.48%   -1.08%     
  Complexity     2613     2613              
============================================
  Files           375      395      +20     
  Lines         20224    22584    +2360     
  Branches       1930     1930              
============================================
+ Hits          11033    12078    +1045     
- Misses         8526     9771    +1245     
- Partials        665      735      +70     

see 21 files with indirect coverage changes

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

@roryqi
roryqi merged commit 6bfbbc2 into apache:master Oct 9, 2023
roryqi added a commit that referenced this pull request Oct 17, 2023
### What changes were proposed in this pull request?
Use a fixed version instead of latest version.

### Why are the changes needed?
Fix the ci error of master branch

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
CI passed.
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.

3 participants