Skip to content

action: update specs #251

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 7 commits into from
Feb 8, 2023
Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Feb 6, 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.

Test

#250 was created with this setup

Tasks

  • Create description with the commit.

Create PRs with the description pointing to the original change in the ecs-logging, similarly done in #235

@v1v v1v marked this pull request as ready for review February 7, 2023 13:23
@v1v v1v requested a review from a team February 7, 2023 13:23
@v1v v1v self-assigned this Feb 7, 2023
@v1v v1v added automation Team:Automation Label for the Observability productivity team labels Feb 7, 2023
@v1v v1v requested a review from Mpdreamz February 7, 2023 13:24
automerge: false
labels:
- dependencies
title: 'synchronize ecs-logging spec'
Copy link
Member

Choose a reason for hiding this comment

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

Is this actually working for you? 😲

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm using dry-run in my local, let me try with my fork

Copy link
Member Author

Choose a reason for hiding this comment

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

v1v#1 nop!

But by changing the target name:

 targets:
   spec.json-update:
-    name: Update spec.json
+    name: 'synchronize ecs-logging spec'
     kind: file
     sourceid: spec.json
     scmid: githubConfig

Copy link
Member

Choose a reason for hiding this comment

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

Interesting. Thanks for checking.

Copy link
Member

Choose a reason for hiding this comment

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

Interesting. Thanks for checking.

Copy link
Member

Choose a reason for hiding this comment

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

Interesting. Thanks for checking.

@v1v v1v merged commit e9365fc into elastic:main Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants