Skip to content

[fix](docker)modify docker base image to apache dockerhub#47837

Merged
zy-kkk merged 2 commits intoapache:masterfrom
catpineapple:modify-docker-base-image
Feb 17, 2025
Merged

[fix](docker)modify docker base image to apache dockerhub#47837
zy-kkk merged 2 commits intoapache:masterfrom
catpineapple:modify-docker-base-image

Conversation

@catpineapple
Copy link
Contributor

@catpineapple catpineapple commented Feb 12, 2025

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@catpineapple
Copy link
Contributor Author

run buildall

intelligentfu8
intelligentfu8 previously approved these changes Feb 12, 2025
Copy link

@intelligentfu8 intelligentfu8 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Feb 12, 2025
@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@github-actions github-actions bot removed the approved Indicates a PR has been approved by one committer. label Feb 14, 2025
@catpineapple
Copy link
Contributor Author

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Feb 17, 2025
@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

Copy link

@intelligentfu8 intelligentfu8 left a comment

Choose a reason for hiding this comment

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

LGTM

@zy-kkk zy-kkk merged commit e745c23 into apache:master Feb 17, 2025
26 of 27 checks passed
lzyy2024 pushed a commit to lzyy2024/doris that referenced this pull request Feb 21, 2025
@catpineapple catpineapple deleted the modify-docker-base-image branch February 28, 2025 02:30
koarz pushed a commit to koarz/doris that referenced this pull request Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants