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

test bitte-ci #1106

Open
wants to merge 1 commit into
base: mig-bitte-ci
Choose a base branch
from
Open

test bitte-ci #1106

wants to merge 1 commit into from

Conversation

blaggacao
Copy link

  • Remove .buildkite
  • test

passed=$(grep -oP 'Total Tests Run: \d+' "retesteth-$1-log.txt")
failed=$(grep -oP 'TOTAL ERRORS DETECTED: \d+' "retesteth-$1-log.txt")

cat <<EOF | buildkite-agent annotate --context "retesteth-$1" --style "$style"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there any Bitte CI equivalent of this? Since the ETS step currently fails (was marked "softFail" in Buildkite) it would be convenient to be able to see the actual success / failure status.

Copy link
Author

Choose a reason for hiding this comment

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

Unfortunately not (yet). We are trying to get a PoC running for now. You'll be able to use buildkite in parallel for a while until we can make sure Bitte CI is in a usable shape.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, that sounds good 👍

Can you make a bitte / ci-specific version of this file (again)? I'm also making one soon for doing the same thing but locally and it would be good to distinguish these use cases.

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes exactly.

@blaggacao blaggacao marked this pull request as draft August 19, 2021 19:45
@blaggacao blaggacao marked this pull request as ready for review August 23, 2021 19:41
@blaggacao blaggacao marked this pull request as draft August 26, 2021 18:21
@blaggacao blaggacao marked this pull request as ready for review August 26, 2021 18:25
@blaggacao blaggacao marked this pull request as draft September 2, 2021 22:45
@blaggacao blaggacao marked this pull request as ready for review September 2, 2021 22:45
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