Skip to content

feat: add higress base image - #423

Merged
johnlanni merged 1 commit into
higress-group:mainfrom
fengshunli:base
Jul 25, 2023
Merged

feat: add higress base image#423
johnlanni merged 1 commit into
higress-group:mainfrom
fengshunli:base

Conversation

@fengshunli

Copy link
Copy Markdown
Contributor

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@codecov-commenter

codecov-commenter commented Jul 14, 2023

Copy link
Copy Markdown

Codecov Report

Merging #423 (c19e301) into main (28892cf) will decrease coverage by 0.33%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #423      +/-   ##
==========================================
- Coverage   40.30%   39.98%   -0.33%     
==========================================
  Files          45       45              
  Lines        6178     6178              
==========================================
- Hits         2490     2470      -20     
- Misses       3515     3535      +20     
  Partials      173      173              

see 1 file with indirect coverage changes

@johnlanni

Copy link
Copy Markdown
Collaborator

@fengshunli Please fix the e2e test.

@fengshunli
fengshunli force-pushed the base branch 2 times, most recently from 711d6b3 to 96cc8d0 Compare July 19, 2023 11:33
@fengshunli

fengshunli commented Jul 19, 2023

Copy link
Copy Markdown
Contributor Author

@fengshunli Please fix the e2e test.

@johnlanni approval

@fengshunli
fengshunli force-pushed the base branch 6 times, most recently from c19e301 to d3a5e8a Compare July 20, 2023 10:45
Comment thread Makefile.core.mk


docker-build-base:
docker buildx build --no-cache --platform linux/amd64,linux/arm64 -t ${HUB}/base:${BASE_VERSION} -f docker/Dockerfile.base . --push

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

使用当前的tag来build一个新的base镜像是不是不太合理啊?

@fengshunli
fengshunli force-pushed the base branch 2 times, most recently from 8a83403 to 0983b5a Compare July 25, 2023 03:41
Signed-off-by: fengshunli <1171313930@qq.com>

@johnlanni johnlanni left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@johnlanni
johnlanni merged commit 0620346 into higress-group:main Jul 25, 2023
VinceCui pushed a commit to VinceCui/higress that referenced this pull request May 21, 2025
Signed-off-by: fengshunli <1171313930@qq.com>
ink-hz pushed a commit to ink-hz/higress-ai-capability-auth that referenced this pull request Nov 5, 2025
Signed-off-by: fengshunli <1171313930@qq.com>
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.

4 participants