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

Migration to buildkite #158

Merged
merged 32 commits into from
Aug 16, 2023
Merged

Migration to buildkite #158

merged 32 commits into from
Aug 16, 2023

Conversation

leo-ri
Copy link
Contributor

@leo-ri leo-ri commented Jul 18, 2023

No description provided.

@elasticmachine
Copy link
Collaborator

elasticmachine commented Jul 18, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-08-16T15:32:01.825+0000

  • Duration: 4 min 26 sec

Test stats 🧪

Test Results
Failed 0
Passed 85
Skipped 0
Total 85

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@leo-ri
Copy link
Contributor Author

leo-ri commented Jul 19, 2023

/test

@leo-ri
Copy link
Contributor Author

leo-ri commented Jul 20, 2023

buildkite test it

1 similar comment
@leo-ri
Copy link
Contributor Author

leo-ri commented Jul 20, 2023

buildkite test it

@leo-ri
Copy link
Contributor Author

leo-ri commented Jul 28, 2023

buildkite test it

env:
ELASTIC_PR_COMMENTS_ENABLED: 'true'
ELASTIC_SLACK_NOTIFICATIONS_ENABLED: 'true'
SLACK_NOTIFICATIONS_CHANNEL: '#elastic-agent-control-plane'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

need to decide which channel to use instead

@leo-ri leo-ri requested a review from alexsapran August 4, 2023 10:13
@leo-ri leo-ri marked this pull request as ready for review August 4, 2023 10:13
with_goreleaser

echo "--- Goreleaser Release"
goreleaser release --snapshot #TODO remove snapshot
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this TODO something that should remove before merging?

awk '{gsub("---", "----"); print }' tests-report.txt

# Create Junit report for junit annotation plugin
go-junit-report > junit-report.xml < tests-report.txt
Copy link
Collaborator

Choose a reason for hiding this comment

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

I was discussing this with @pazone today, I think it would be better to move the go-junit-report into its own step in the pipeline.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it will run in a separate container then.... but i changed it

@leo-ri leo-ri merged commit cd02349 into elastic:main Aug 16, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants