Add new field migrate_from to handle inputs migration

code/go/pkg/validator/validator_test.go

@@ -480,6 +480,11 @@ func TestValidateFile(t *testing.T) {
 		"deprecated_integration_policy_input": {},
 		"deprecated_integration_policy":       {},
 		"deprecated_integration_stream_var":   {},
+		"good_migrate_from":                   {},
+		"bad_migrate_from": {
+			"manifest.yml",
+			[]string{`field policy_templates.0.inputs.0: Additional property migrate_from is not allowed`},
+		},
 		"bad_deprecation_description": {
 			"manifest.yml",
 			[]string{"field deprecated.description: Invalid type. Expected: string, given: null"},

spec/changelog.yml

@@ -51,6 +51,10 @@
     - description: Add deployer option to system benchmark configuration.
       type: enhancement
       link: https://github.com/elastic/package-spec/pull/1099
+    # Pending on https://github.com/elastic/kibana/pull/242934
+    - description: Add support for migrating input types at integrations policy templates and data streams.
+      type: enhancement
+      link: https://github.com/elastic/package-spec/pull/1021
 - version: 3.5.7
   changes:
     - description: Allow _dev directory for content-only packages; use _dev/shared for development files (e.g. dashboard YML sources).

spec/integration/data_stream/manifest.spec.yml

@@ -606,6 +606,8 @@ spec:
             $ref: "../../integration/manifest.spec.yml#/definitions/var_groups"
           dynamic_signal_types:
             $ref: "../../integration/manifest.spec.yml#/definitions/dynamic_signal_types"
+          migrate_from:
+            $ref: "../../integration/manifest.spec.yml#/definitions/migrate_from"
           enabled:
             description: Is stream enabled?
             type: boolean
@@ -691,6 +693,9 @@ versions:
         path: "/definitions/vars/items/properties/deprecated"
       - op: remove
         path: "/properties/streams/items/properties/var_groups"
+      # Input migration.
+      - op: remove
+        path: "/properties/streams/items/properties/migrate_from"
   - before: 3.5.0
     patch:
       # Require >=3.5.0 to use the duration variable type.

spec/integration/manifest.spec.yml

@@ -124,7 +124,7 @@ spec:
           - notification
           # Metrics, logs, and traces collection
           - observability
-          # OpenTelemetry 
+          # OpenTelemetry
           - opentelemetry
           # Operating system monitoring
           - os_system
@@ -532,7 +532,7 @@ spec:
               type: string
             data_stream:
               description: Name of the data stream that replaces the deprecated one.
-              type: string         
+              type: string
       required:
         - since
         - description
@@ -681,6 +681,15 @@ spec:
       description: "When enabled, decides the transforms and index templates that need to be created depending on the pipelines specified in the configuration. This field is only allowed when the input type is 'otelcol'."
       type: boolean
       default: false
+    migrate_from:
+      description: >
+        Previous input type to migrate configuration from.
+        This allows Fleet to automatically migrate the policy configuration when
+        replacing one input implementation with an equivalent one.
+        This field should only be used for inputs that don't need agent's state persistence.
+      type: string
+      examples:
+        - httpjson
   properties:
     format_version:
       description: The version of the package specification format used by this package.
@@ -815,6 +824,8 @@ spec:
                     - ["default"]
                     - ["agentless"]
                     - ["default", "agentless"]
+                migrate_from:
+                  $ref: "#/definitions/migrate_from"
                 required_vars:
                   $ref: "./data_stream/manifest.spec.yml#/definitions/required_vars"
                 vars:
@@ -965,6 +976,11 @@ versions:
         path: "/properties/var_groups"
       - op: remove
         path: "/properties/policy_templates/items/properties/inputs/items/properties/hide_in_var_group_options"
+      # Input migration.
+      - op: remove
+        path: "/properties/policy_templates/items/properties/inputs/items/properties/migrate_from"
+      - op: remove
+        path: "/definitions/migrate_from"
   - before: 3.3.2
     patch:
       - op: remove

test/packages/bad_migrate_from/LICENSE.txt

@@ -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.

test/packages/bad_migrate_from/_dev/build/docs/README.md

@@ -0,0 +1,101 @@
+{{- generatedHeader }}
+{{/*
+This template can be used as a starting point for writing documentation for your new integration. For each section, fill in the details
+described in the comments.
+
+Find more detailed documentation guidelines in https://www.elastic.co/docs/extend/integrations/documentation-guidelines
+*/}}
+# New Package Integration for Elastic
+
+## Overview
+{{/* Complete this section with a short summary of what data this integration collects and what use cases it enables */}}
+The New Package integration for Elastic enables collection of ...
+This integration facilitates ...
+
+### Compatibility
+{{/* Complete this section with information on what 3rd party software or hardware versions this integration is compatible with */}}
+This integration is compatible with ...
+
+### How it works
+{{/* Add a high level overview on how this integration works. For example, does it collect data from API calls or recieving data from a network or file.*/}}
+
+## What data does this integration collect?
+{{/* Complete this section with information on what types of data the integration collects, and link to reference documentation if available */}}
+The New Package integration collects log messages of the following types:
+* ...
+
+### Supported use cases
+{{/* Add details on the use cases that can be enabled by using this integration. Explain why a user would want to install and use this integration. */}}
+
+## What do I need to use this integration?
+{{/* List any vendor-specific prerequisites needed before starting to install the integration. */}}
+
+## How do I deploy this integration?
+
+### Agent-based deployment
+
+Elastic Agent must be installed. For more details, check the Elastic Agent [installation instructions](docs-content://reference/fleet/install-elastic-agents.md). You can install only one Elastic Agent per host.
+
+Elastic Agent is required to stream data from the syslog or log file receiver and ship the data to Elastic, where the events will then be processed via the integration's ingest pipelines.
+
+{{/* If agentless is available for this integration, we'll want to include that here as well.
+### Agentless deployment
+
+Agentless deployments are only supported in Elastic Serverless and Elastic Cloud environments. Agentless deployments provide a means to ingest data while avoiding the orchestration, management, and maintenance needs associated with standard ingest infrastructure. Using an agentless deployment makes manual agent deployment unnecessary, allowing you to focus on your data instead of the agent that collects it.
+
+For more information, refer to [Agentless integrations](https://www.elastic.co/guide/en/serverless/current/security-agentless-integrations.html) and [Agentless integrations FAQ](https://www.elastic.co/guide/en/serverless/current/agentless-integration-troubleshooting.html) 
+*/}}
+
+### Onboard / configure
+{{/* List the steps that will need to be followed in order to completely set up a working integration.
+For integrations that support multiple input types, be sure to add steps for all inputs.
+*/}}
+
+### Validation
+{{/* How can the user test whether the integration is working? Including example commands or test files if applicable */}}
+
+## Troubleshooting
+
+For help with Elastic ingest tools, check [Common problems](https://www.elastic.co/docs/troubleshoot/ingest/fleet/common-problems).
+{{/*
+Add any vendor specific troubleshooting here.
+
+Are there common issues or “gotchas” for deploying this integration? If so, how can they be resolved?
+If applicable, links to the third-party software’s troubleshooting documentation.
+*/}}
+
+## Scaling
+
+For more information on architectures that can be used for scaling this integration, check the [Ingest Architectures](https://www.elastic.co/docs/manage-data/ingest/ingest-reference-architectures) documentation.
+{{/* Add any vendor specific scaling information here */}}
+
+## Reference
+{{/* Repeat for each data stream of the current type
+### {Data stream name}
+
+The `{data stream name}` data stream provides events from {source} of the following types: {list types}.
+
+For each data_stream_name, include an optional summary of the datastream, the exported fields reference table and the sample event.
+
+The fields template function will be replaced by a generated list of all fields from the `fields/` directory of the data stream when building the integration.
+
+#### {data stream name} fields
+
+To include a generated list of fields from the `fields/` directory, uncomment and use:
+{{ fields "data_stream_name" }}
+
+The event template function will be replace by a sample event, taken from `sample_event.json`, when building this integration.
+
+To include a sample event from `sample_event.json`, uncomment and use:
+{{ event "data_stream_name" }}
+
+*/}}
+
+### Inputs used
+{{/* All inputs used by this package will be automatically listed here. */}}
+{{ inputDocs }}
+
+### API usage
+{{/* For integrations that use APIs to collect data, document all the APIs that are used, and link to relevent information */}}
+These APIs are used with this integration:
+* ...

test/packages/bad_migrate_from/changelog.yml

@@ -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

test/packages/bad_migrate_from/data_stream/logs/agent/stream/filestream.yml.hbs

@@ -0,0 +1,4 @@
+paths:
+{{#each paths as |path|}}
+  - {{path}}
+{{/each}}

test/packages/bad_migrate_from/data_stream/logs/elasticsearch/ingest_pipeline/default.yml

@@ -0,0 +1,18 @@
+---
+description: Pipeline for processing sample logs
+processors:
+  - set:
+      tag: set_dataset
+      field: data_stream.dataset
+      value: bad_migrate_from.logs
+on_failure:
+  - set:
+      field: event.kind
+      value: pipeline_error
+  - set:
+      field: error.message
+      value: >-
+        Processor '{{{ _ingest.on_failure_processor_type }}}'
+        with tag '{{{ _ingest.on_failure_processor_tag }}}'
+        in pipeline '{{{ _ingest.pipeline }}}'
+        failed with message '{{{ _ingest.on_failure_message }}}'

test/packages/bad_migrate_from/data_stream/logs/fields/base-fields.yml

@@ -0,0 +1,12 @@
+- name: data_stream.type
+  type: constant_keyword
+  description: Data stream type.
+- name: data_stream.dataset
+  type: constant_keyword
+  description: Data stream dataset.
+- name: data_stream.namespace
+  type: constant_keyword
+  description: Data stream namespace.
+- name: '@timestamp'
+  type: date
+  description: Event timestamp.

test/packages/bad_migrate_from/data_stream/logs/manifest.yml

@@ -0,0 +1,16 @@
+title: "Sample Logs"
+type: logs
+streams:
+  - input: filestream
+    title: "Collect logs via filestream"
+    description: "Collect logs using the filestream input"
+    template_path: filestream.yml.hbs
+    vars:
+      - name: paths
+        type: text
+        title: "Paths"
+        multi: true
+        required: true
+        show_user: true
+        default:
+          - /var/log/*.log