Skip to content

[chore](workflow) Fix the BE UT (Clang) workflow#16847

Merged
BiteTheDDDDt merged 1 commit into
apache:masterfrom
adonis0147:workflow
Feb 17, 2023
Merged

[chore](workflow) Fix the BE UT (Clang) workflow#16847
BiteTheDDDDt merged 1 commit into
apache:masterfrom
adonis0147:workflow

Conversation

@adonis0147
Copy link
Copy Markdown
Contributor

@adonis0147 adonis0147 commented Feb 16, 2023

Proposed changes

Remove the command apt upgrade in the BE UT (Clang) workflow.

Problem summary

Recently, the runner-image was updated frequently and some bugs were introduced which made our workflow BE UT (Clang) fail. See https://github.com/apache/doris/actions/runs/4195816421.

Actually, in BE UT (Clang) workflow, we don't need to execute the command apt upgrade. We can remove it and fix the issues.

Checklist(Required)

  • Does it affect the original behavior
  • Has unit tests been added
  • Has document been added or modified
  • Does it need to update dependencies
  • Is this PR support rollback (If NO, please explain WHY)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@adonis0147
Copy link
Copy Markdown
Contributor Author

p0

Copy link
Copy Markdown
Contributor

@BiteTheDDDDt BiteTheDDDDt left a comment

Choose a reason for hiding this comment

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

LGTM

@BiteTheDDDDt BiteTheDDDDt merged commit 630865a into apache:master Feb 17, 2023
@adonis0147 adonis0147 deleted the workflow branch February 17, 2023 02:35
morningman pushed a commit that referenced this pull request Feb 18, 2023
yagagagaga pushed a commit to yagagagaga/doris that referenced this pull request Mar 9, 2023
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.

2 participants