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

Initial support for content packages #2049

Merged
merged 4 commits into from
Sep 10, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cmd/create_package.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ func createPackageCommandAction(cmd *cobra.Command, args []string) error {
Name: "type",
Prompt: &survey.Select{
Message: "Package type:",
Options: []string{"input", "integration"},
Options: []string{"input", "integration", "content"},
Default: "integration",
},
Validate: survey.Required,
Expand Down
6 changes: 6 additions & 0 deletions internal/packages/archetype/_static/package-manifest.yml.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ conditions:
version: "{{.Manifest.Conditions.Kibana.Version}}"
elastic:
subscription: "{{.Manifest.Conditions.Elastic.Subscription}}"
{{- if eq .Manifest.Type "content" }}
discovery:
fields: []
{{- end }}
screenshots:
- src: /img/sample-screenshot.png
title: Sample screenshot
Expand All @@ -26,6 +30,7 @@ icons:
title: Sample logo
size: 32x32
type: image/svg+xml
{{- if (or (eq .Manifest.Type "integration") (eq .Manifest.Type "input")) }}
policy_templates:
{{- if eq .Manifest.Type "integration" }}
- name: sample
Expand Down Expand Up @@ -72,6 +77,7 @@ policy_templates:
- localhost
{{ end }}
{{ end -}}
{{ end -}}
{{ if .Manifest.Elasticsearch }}
elasticsearch:
{{ if .Manifest.Elasticsearch.SourceMode }}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
errors:
exclude_checks:
{{- range $check := .ExcludeChecks }}
- {{ $check }}
{{- end }}
10 changes: 10 additions & 0 deletions internal/packages/archetype/package.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@ type PackageDescriptor struct {
Manifest packages.PackageManifest

InputDataStreamType string

ExcludeChecks []string
}

// CreatePackage function bootstraps the new package based on the provided descriptor.
Expand Down Expand Up @@ -92,6 +94,14 @@ func createPackageInDir(packageDescriptor PackageDescriptor, cwd string) error {

}

if len(packageDescriptor.ExcludeChecks) > 0 {
Copy link
Contributor

Choose a reason for hiding this comment

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

There were no exclude checks when creating a content package. These are the files created:

 $ elastic-package create package 
Create a new package
? Package type: content
? Package name: new_package
? Version: 0.0.1
? License: Elastic-2.0
? Package title: New Package
? Description: This is a new package.
? Categories: custom
? Kibana version constraint: ^8.15.1
? Required Elastic subscription: basic
? Github owner: elastic/integrations
? Owner type: elastic
New package has been created: new_package
Done

 $ cd new_package

 $ ls -l 
total 20
-rw-r--r-- 1 user user  226 sep  9 18:22 changelog.yml
drwxr-xr-x 2 user user 4096 sep  9 18:22 docs
drwxr-xr-x 2 user user 4096 sep  9 18:22 img
-rw-rw-r-- 1 user user 3860 sep  9 18:22 LICENSE.txt
-rw-r--r-- 1 user user  560 sep  9 18:26 manifest.yml

Copy link
Member Author

Choose a reason for hiding this comment

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

That's correct. I added this mainly for testing by now.

logger.Debugf("Write validation file")
err = renderResourceFile(validationBaseTemplate, &packageDescriptor, filepath.Join(baseDir, "validation.yml"))
if err != nil {
return fmt.Errorf("can't render validation file")
}
}

logger.Debugf("Format the entire package")
err = formatter.Format(baseDir, false)
if err != nil {
Expand Down
21 changes: 19 additions & 2 deletions internal/packages/archetype/package_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import (
"path/filepath"
"testing"

"github.com/Masterminds/semver/v3"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"

Expand All @@ -29,6 +30,10 @@ func TestPackage(t *testing.T) {
pd := createPackageDescriptorForTest("input", "^8.9.0")
createAndCheckPackage(t, pd, true)
})
t.Run("content-package", func(t *testing.T) {
pd := createPackageDescriptorForTest("content", "^8.16.0")
createAndCheckPackage(t, pd, true)
})
}

func createAndCheckPackage(t *testing.T, pd PackageDescriptor, valid bool) {
Expand All @@ -52,17 +57,28 @@ func createPackageDescriptorForTest(packageType, kibanaVersion string) PackageDe
},
}
}
version := "1.2.3"
specVersion, err := GetLatestStableSpecVersion()
excludeChecks := []string{}
if err != nil {
panic(err)
}
if packageType == "content" {
minSpecVersion := semver.MustParse("3.4.0")
if !specVersion.LessThan(minSpecVersion) {
panic("this code can be removed")
}
version = "0.1.0"
specVersion = *minSpecVersion
excludeChecks = append(excludeChecks, "PSR00002")
Comment on lines +72 to +73
Copy link
Contributor

Choose a reason for hiding this comment

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

Currently, elastic-package create package it would create a content package with format_version: 3.2.2 (latest spec version published). Should it be set somehow here for content packages at least 3.4.0 ?

Just created a package, and when running elastic-package build, it shows these errors:

Error: building package failed: invalid content found in built zip package: found 6 validation errors:
   1. package with non-stable semantic version and active beta features (enabled in [/home/user/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]) can't be released as stable version. (PSR00002)
   2. item [LICENSE.txt] is not allowed in folder [/home/user/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]
   3. item [changelog.yml] is not allowed in folder [/home/user/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]
   4. item [docs] is not allowed in folder [/home/user/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]
   5. item [img] is not allowed in folder [/home/user/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]
   6. item [manifest.yml] is not allowed in folder [/home/user/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]

When updating the format_version to 3.4.0, it still reports this error:

Error: building package failed: invalid content found in built zip package: found 1 validation error:
   1. package with non-stable semantic version and active beta features (enabled in [/home/mariorodriguez/Coding/work/elastic-package/build/packages/new_package-0.0.1.zip]) can't be released as stable version. (PSR00002)

Copy link
Member Author

Choose a reason for hiding this comment

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

These are handled in the test case, I didn't add the exceptions to the real command to don't make it too easy to use content packages yet, and to avoid us having to remember to remove these exceptions when/if the package is released as GA.

Would you prefer that we add the exceptions in the command?

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah ok! Better to force to add the exceptions manually. No need to change anything then.

}
return PackageDescriptor{
Manifest: packages.PackageManifest{
SpecVersion: specVersion.String(),
Name: "go_unit_test_package",
Title: "Go Unit Test Package",
Type: packageType,
Version: "1.2.3",
Version: version,
Conditions: packages.Conditions{
Kibana: packages.KibanaConditions{
Version: kibanaVersion,
Expand All @@ -79,12 +95,13 @@ func createPackageDescriptorForTest(packageType, kibanaVersion string) PackageDe
Categories: []string{"aws", "custom"},
Elasticsearch: elasticsearch,
},
ExcludeChecks: excludeChecks,
InputDataStreamType: inputDataStreamType,
}
}

func checkPackage(t *testing.T, packageRoot string, valid bool) {
err := validation.ValidateFromPath(packageRoot)
err, _ := validation.ValidateAndFilterFromPath(packageRoot)
if !valid {
assert.Error(t, err)
return
Expand Down
3 changes: 3 additions & 0 deletions internal/packages/archetype/resources.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ var packageDocsReadme string
//go:embed _static/fields-base.yml.tmpl
var fieldsBaseTemplate string

//go:embed _static/package-validation.yml.tmpl
var validationBaseTemplate string

// Images (logo and screenshot)

//go:embed _static/sampleIcon.svg
Expand Down
19 changes: 18 additions & 1 deletion internal/packages/packages.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import (
"io/fs"
"os"
"path/filepath"
"slices"

"github.com/elastic/go-ucfg"
"github.com/elastic/go-ucfg/yaml"
Expand Down Expand Up @@ -96,6 +97,16 @@ type Conditions struct {
Elastic ElasticConditions `config:"elastic" json:"elastic" yaml:"elastic"`
}

// Discovery define indications for the data this package can be useful with.
type Discovery struct {
Fields []DiscoveryField `config:"fields" json:"fields" yaml:"fields"`
}

// DiscoveryField defines a field used for discovery.
type DiscoveryField struct {
Name string `config:"name" json:"name" yaml:"name"`
}

// PolicyTemplate is a configuration of inputs responsible for collecting log or metric data.
type PolicyTemplate struct {
Name string `config:"name" json:"name" yaml:"name"` // Name of policy template.
Expand Down Expand Up @@ -130,6 +141,7 @@ type PackageManifest struct {
Version string `config:"version" json:"version" yaml:"version"`
Source Source `config:"source" json:"source" yaml:"source"`
Conditions Conditions `config:"conditions" json:"conditions" yaml:"conditions"`
Discovery Discovery `config:"discovery" json:"discovery" yaml:"discovery"`
PolicyTemplates []PolicyTemplate `config:"policy_templates" json:"policy_templates" yaml:"policy_templates"`
Vars []Variable `config:"vars" json:"vars" yaml:"vars"`
Owner Owner `config:"owner" json:"owner" yaml:"owner"`
Expand Down Expand Up @@ -450,7 +462,12 @@ func isPackageManifest(path string) (bool, error) {
if err != nil {
return false, fmt.Errorf("reading package manifest failed (path: %s): %w", path, err)
}
return (m.Type == "integration" || m.Type == "input") && m.Version != "", nil
supportedTypes := []string{
"content",
"input",
"integration",
}
return slices.Contains(supportedTypes, m.Type) && m.Version != "", nil
}

func isDataStreamManifest(path string) (bool, error) {
Expand Down
2 changes: 1 addition & 1 deletion internal/testrunner/testrunner.go
Original file line number Diff line number Diff line change
Expand Up @@ -503,7 +503,7 @@ func PackageHasDataStreams(manifest *packages.PackageManifest) (bool, error) {
switch manifest.Type {
case "integration":
return true, nil
case "input":
case "input", "content":
return false, nil
default:
return false, fmt.Errorf("unexpected package type %q", manifest.Type)
Expand Down
93 changes: 93 additions & 0 deletions test/packages/parallel/otel_http_server/LICENSE.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,93 @@
Elastic License 2.0

URL: https://www.elastic.co/licensing/elastic-license

## Acceptance

By using the software, you agree to all of the terms and conditions below.

## Copyright License

The licensor grants you a non-exclusive, royalty-free, worldwide,
non-sublicensable, non-transferable license to use, copy, distribute, make
available, and prepare derivative works of the software, in each case subject to
the limitations and conditions below.

## Limitations

You may not provide the software to third parties as a hosted or managed
service, where the service provides users with access to any substantial set of
the features or functionality of the software.

You may not move, change, disable, or circumvent the license key functionality
in the software, and you may not remove or obscure any functionality in the
software that is protected by the license key.

You may not alter, remove, or obscure any licensing, copyright, or other notices
of the licensor in the software. Any use of the licensor’s trademarks is subject
to applicable law.

## Patents

The licensor grants you a license, under any patent claims the licensor can
license, or becomes able to license, to make, have made, use, sell, offer for
sale, import and have imported the software, in each case subject to the
limitations and conditions in this license. This license does not cover any
patent claims that you cause to be infringed by modifications or additions to
the software. If you or your company make any written claim that the software
infringes or contributes to infringement of any patent, your patent license for
the software granted under these terms ends immediately. If your company makes
such a claim, your patent license ends immediately for work on behalf of your
company.

## Notices

You must ensure that anyone who gets a copy of any part of the software from you
also gets a copy of these terms.

If you modify the software, you must include in any modified copies of the
software prominent notices stating that you have modified the software.

## No Other Rights

These terms do not imply any licenses other than those expressly granted in
these terms.

## Termination

If you use the software in violation of these terms, such use is not licensed,
and your licenses will automatically terminate. If the licensor provides you
with a notice of your violation, and you cease all violation of this license no
later than 30 days after you receive that notice, your licenses will be
reinstated retroactively. However, if you violate these terms after such
reinstatement, any additional violation of these terms will cause your licenses
to terminate automatically and permanently.

## No Liability

*As far as the law allows, the software comes as is, without any warranty or
condition, and the licensor will not be liable to you for any damages arising
out of these terms or the use or nature of the software, under any kind of
legal claim.*

## Definitions

The **licensor** is the entity offering these terms, and the **software** is the
software the licensor makes available under these terms, including any portion
of it.

**you** refers to the individual or entity agreeing to these terms.

**your company** is any legal entity, sole proprietorship, or other kind of
organization that you work for, plus all organizations that have control over,
are under the control of, or are under common control with that
organization. **control** means ownership of substantially all the assets of an
entity, or the power to direct its management and policies by vote, contract, or
otherwise. Control can be direct or indirect.

**your licenses** are all the licenses granted to you for the software under
these terms.

**use** means anything you do with the software requiring one of your licenses.

**trademark** means trademarks, service marks, and similar rights.
6 changes: 6 additions & 0 deletions test/packages/parallel/otel_http_server/changelog.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
# newer versions go on top
- version: "0.0.1"
changes:
- description: Initial draft of the package
type: enhancement
link: https://github.com/elastic/integrations/pull/1 # FIXME Replace with the real PR link
4 changes: 4 additions & 0 deletions test/packages/parallel/otel_http_server/docs/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# HTTP Server following OTEL Semconv

This package includes dashboards for data collected from HTTP Servers following
OTEL Semconv.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
30 changes: 30 additions & 0 deletions test/packages/parallel/otel_http_server/manifest.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
format_version: 3.4.0
name: otel_http_server
title: "HTTP Server following OTEL Semconv"
version: 0.0.1
source:
license: "Elastic-2.0"
description: "This package provides dashboards for HTTP server metrics that follow OTEL Semantic Conventions."
type: content
categories:
- web
conditions:
kibana:
version: "^8.16.0"
elastic:
subscription: "basic"
discovery:
fields: []
screenshots:
- src: /img/sample-screenshot.png
title: Sample screenshot
size: 600x600
type: image/png
icons:
- src: /img/sample-logo.svg
title: Sample logo
size: 32x32
type: image/svg+xml
owner:
github: elastic/ecosystem
type: elastic
3 changes: 3 additions & 0 deletions test/packages/parallel/otel_http_server/validation.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
errors:
exclude_checks:
- PSR00002