Skip to content

action: update specs #43

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

Merged
merged 5 commits into from
Feb 8, 2023
Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Feb 8, 2023

What

Use GitHub actions in addition to the updatecli to bump the json schemas created in ecs-logging.

It runs on a daily basis and create a Pull Request if new changes exist.

Create PRs with the description pointing to the original change in the ecs-logging, similarly done in elastic/ecs-logging-ruby#30

Why

Jenkins ecosystem at Elastic is deprecated, so let's use GH actions with updatecli, so consumers can control what to do and when.

Further details

The automation relied on Jenkins and projects were onboarded by having a configuration entry in https://github.com/elastic/ecs-logging/blob/main/.ci/.jenkins-loggers.yml

@v1v v1v requested review from a team February 8, 2023 15:52
@v1v v1v self-assigned this Feb 8, 2023
@v1v
Copy link
Member Author

v1v commented Feb 8, 2023

I created the specs.json empty file so we can validate the automation works as expected when this PR is merged

@apmmachine
Copy link
Contributor

apmmachine commented Feb 8, 2023

💔 Build Failed

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-02-08T17:28:04.119+0000

  • Duration: 6 min 27 sec

Test stats 🧪

Test Results
Failed 0
Passed 6
Skipped 0
Total 6

Steps errors 5

Expand to view the steps failures

Run tests
  • Took 0 min 1 sec . View more details here
  • Description: VERSION=7.2-alpine composer test
Run tests
  • Took 0 min 1 sec . View more details here
  • Description: VERSION=7.3-alpine composer test
Run tests
  • Took 0 min 1 sec . View more details here
  • Description: VERSION=7.4-alpine composer test
Run tests
  • Took 0 min 1 sec . View more details here
  • Description: VERSION=8.0-alpine composer test
Error signal
  • Took 0 min 0 sec . View more details here
  • Description: hudson.AbortException: script returned exit code 2

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Co-authored-by: Sergey Kleyman <SergeyKleyman@users.noreply.github.com>
@v1v v1v merged commit a0eaf60 into elastic:main Feb 8, 2023
@reakaleek reakaleek mentioned this pull request Feb 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.

4 participants