Skip to content

Tychon modular#8400

Closed
joeperuzzi wants to merge 84 commits intoelastic:mainfrom
joeperuzzi:tychon_modular
Closed

Tychon modular#8400
joeperuzzi wants to merge 84 commits intoelastic:mainfrom
joeperuzzi:tychon_modular

Conversation

@joeperuzzi
Copy link

@joeperuzzi joeperuzzi commented Nov 3, 2023

Proposed commit message

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

joeperuzzi and others added 30 commits June 26, 2023 16:47
Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com>
Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com>
Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com>
Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com>
….hbs

Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com>
Changed {{#each paths as |path i|}} to {{#each paths as |path|}} per efd6 request
Added new line to end per efd6 request.
Updated ecs.version per efd6 request
Added new line to end per efd6 request
Added descriptions to names.
Removed Asset Identification and updated exported field for tychon_cve
@elasticmachine
Copy link

❕ Build Aborted

The PR is not allowed to run in the CI yet

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

Expand to view the summary

Build stats

  • Start Time: 2023-11-03T17:20:20.260+0000

  • Duration: 41 min 48 sec

Steps errors 2

Expand to view the steps failures

Load a resource file from a library
  • Took 0 min 0 sec . View more details here
  • Description: approval-list/elastic/integrations.yml
Error signal
  • Took 0 min 0 sec . View more details here
  • Description: githubApiCall: The REST API call https://api.github.com/orgs/elastic/members/joeperuzzi return the message : java.lang.Exception: httpRequest: Failure connecting to the service https://api.github.com/orgs/elastic/members/joeperuzzi : httpRequest: Failure connecting to the service https://api.github.com/orgs/elastic/members/joeperuzzi : Code: 404Error: {"message":"User does not exist or is not a member of the organization","documentation_url":"https://docs.github.com/rest/orgs/members#check-organization-membership-for-a-user"}

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

@botelastic
Copy link

botelastic bot commented Dec 6, 2023

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Dec 6, 2023
@joeperuzzi
Copy link
Author

We have finally got the project building, however, resubmission is delayed a bit more. We will update the build asap.

@botelastic botelastic bot removed the Stalled label Dec 6, 2023
@mrodm
Copy link
Collaborator

mrodm commented Dec 21, 2023

Hi @joeperuzzi, please update your branch with the latest contents from main branch. There was an important PR merged updating the CI pipelines. Thanks!

@andrewkroh
Copy link
Member

One thing I expected was that all of the dashboards would be added last in a separate PR since it's difficult to know which data stream the dashboard is associated with and we don't have all data streams yet.

@botelastic
Copy link

botelastic bot commented Feb 7, 2024

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Feb 7, 2024
@botelastic
Copy link

botelastic bot commented Mar 8, 2024

Hi! This PR has been stale for a while and we're going to close it as part of our cleanup procedure. We appreciate your contribution and would like to apologize if we have not been able to review it, due to the current heavy load of the team. Feel free to re-open this PR if you think it should stay open and is worth rebasing. Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants