From ee73f85e4cd52766cc6034d0ecef70dd226c5adf Mon Sep 17 00:00:00 2001
From: Dominik Rosiek <drosiek@sumologic.com>
Date: Tue, 6 Jun 2023 13:08:46 +0200
Subject: [PATCH 1/6] feat: deprecate options which are going to be removed

Signed-off-by: Dominik Rosiek <drosiek@sumologic.com>
---
 .../drosiek-sumoexporter-deprecations.yaml    | 28 +++++++
 exporter/sumologicexporter/README.md          | 73 +++++++++++++++++++
 exporter/sumologicexporter/exporter.go        | 25 +++++++
 3 files changed, 126 insertions(+)
 create mode 100755 .chloggen/drosiek-sumoexporter-deprecations.yaml

diff --git a/.chloggen/drosiek-sumoexporter-deprecations.yaml b/.chloggen/drosiek-sumoexporter-deprecations.yaml
new file mode 100755
index 000000000000..0820c1bd1e2a
--- /dev/null
+++ b/.chloggen/drosiek-sumoexporter-deprecations.yaml
@@ -0,0 +1,28 @@
+# Use this changelog template to create an entry for release notes.
+# If your change doesn't affect end users, such as a test fix or a tooling change,
+# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
+
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: deprecation
+
+# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
+component: sumologicexporter
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: deprecating options which are going to be removed
+
+# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
+issues: [23059]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext: |-
+  The following options are going to be deprecated and removed in the future:
+
+  - `metric_format: {carbon2, graphite}` (leaving only `prometheus`)
+  - `metadata_attributes: [<regex>]`
+  - `graphite_template: <template>`
+  - `source_category: <template>`
+  - `source_name: <template>`
+  - `source_host: <template>`
diff --git a/exporter/sumologicexporter/README.md b/exporter/sumologicexporter/README.md
index f6ae9269241f..3824836dca18 100644
--- a/exporter/sumologicexporter/README.md
+++ b/exporter/sumologicexporter/README.md
@@ -10,6 +10,61 @@
 [contrib]: https://github.com/open-telemetry/opentelemetry-collector-releases/tree/main/distributions/otelcol-contrib
 <!-- end autogenerated section -->
 
+## Migration to new architecture
+
+**This exporter is subject of major changes now.**
+
+For some time we have been developing [new Sumo Logic exporter](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/exporter/sumologicexporter#sumo-logic-exporter) and now we are in progress of moving it into this repository.
+
+The following options are deprecated and they won't exist in new version:
+
+- `metric_format: {carbon2, graphite}`
+- `metadata_attributes: [<regex>]`
+- `graphite_template: <template>`
+- `source_category: <template>`
+- `source_name: <template>`
+- `source_host: <template>`
+
+After the new exporter will be moved to this repository:
+
+- `carbon2` and `graphite` are going to be no longer supported and `prometheus` or `otlp` format should be used
+- all resource level attributes are going to be treated as `metadata_attributes`. You can use [Group by Attributes processor](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/processor/groupbyattrsprocessor/) to move attributes from record level to resource level. For example:
+
+  ```yaml
+  # before switch to new collector
+  exporters:
+    sumologic:
+      metadata_attribute:
+        - my_attribute
+  # after switch to new collector
+  processors:
+    groupbyattrs:
+      keys:
+        - my_attribute
+  ```
+
+- Source templates (`source_category`, `source_name` and `source_host`) are going to be removed from the exporter and sources may be set using `_sourceCategory`, `sourceName` or `_sourceHost` resource attributes. We recommend to use [Transform Processor](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/processor/transformprocessor/). For example:
+
+  ```yaml
+  # before switch to new collector
+  exporters:
+    sumologic:
+      source_category: "%{foo}/constant/%{bar}"
+  # after switch to new collector
+  processors:
+    transformprocessor:
+      log_statements:
+        context: log
+        statements:
+          # set default value to unknown
+          - set(attributes["foo"], "unknown") where attributes["foo"] == nil
+          - set(attributes["bar"], "unknown") where attributes["foo"] == nil
+          # set _sourceCategory as "%{foo}/constant/%{bar}"
+          - set(resource.attributes["_sourceCategory"], Concat([attributes["foo"], "/constant/", attributes["bar"]], ""))
+  ```
+
+## Configuration
+
 This exporter supports sending logs and metrics data to [Sumo Logic](https://www.sumologic.com/).
 Traces are exported using native otlphttp exporter as described
 [here](https://help.sumologic.com/Traces/Getting_Started_with_Transaction_Tracing)
@@ -30,12 +85,18 @@ exporters:
 
     # List of regexes for attributes which should be send as metadata
     # default = []
+    #
+    # This opion is deprecated:
+    # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     metadata_attributes: [<regex>]
 
     # format to use when sending logs to Sumo Logic, default = json,
     log_format: {json, text}
 
     # format to use when sending metrics to Sumo Logic, default = prometheus,
+    #
+    # carbon2 and graphite are deprecated:
+    # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     metric_format: {carbon2, graphite, prometheus}
 
     # Template for Graphite format.
@@ -44,24 +105,36 @@ exporters:
     #
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
+    #
+    # This opion is deprecated:
+    # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     graphite_template: <template>
 
     # Desired source category. Useful if you want to override the source category configured for the source.
     #
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
+    #
+    # This opion is deprecated:
+    # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     source_category: <template>
 
     # Desired source name. Useful if you want to override the source name configured for the source.
     #
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
+    #
+    # This opion is deprecated:
+    # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     source_name: <template>
 
     # Desired source host. Useful if you want to override the source hosy configured for the source.
     #
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
+    #
+    # This opion is deprecated:
+    # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     source_host: <template>
 
     # timeout is the timeout for every attempt to send data to the backend,
diff --git a/exporter/sumologicexporter/exporter.go b/exporter/sumologicexporter/exporter.go
index fdedaae1a5ad..58b3aee00def 100644
--- a/exporter/sumologicexporter/exporter.go
+++ b/exporter/sumologicexporter/exporter.go
@@ -29,6 +29,31 @@ type sumologicexporter struct {
 }
 
 func initExporter(cfg *Config, settings component.TelemetrySettings) (*sumologicexporter, error) {
+
+	if cfg.MetricFormat == GraphiteFormat {
+		settings.Logger.Warn("`metric_format: graphite` nad `graphite_template` are deprecated and are going to be removed in the future. See https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture for more information")
+	}
+
+	if cfg.MetricFormat == Carbon2Format {
+		settings.Logger.Warn("`metric_format: carbon` is deprecated and is going to be removed in the future. See https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture for more information")
+	}
+
+	if len(cfg.MetadataAttributes) > 0 {
+		settings.Logger.Warn("`metadata_attributes: []` is deprecated and is going to be removed in the future. See https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture for more information")
+	}
+
+	if cfg.SourceCategory != "" {
+		settings.Logger.Warn("`source_category: <template>` is deprecated and is going to be removed in the future. See https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture for more information")
+	}
+
+	if cfg.SourceHost != "" {
+		settings.Logger.Warn("`source_host: <template>` is deprecated and is going to be removed in the future. See https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture for more information")
+	}
+
+	if cfg.SourceName != "" {
+		settings.Logger.Warn("`source_name: <template>` is deprecated and is going to be removed in the future. See https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture for more information")
+	}
+
 	sfs := newSourceFormats(cfg)
 
 	f, err := newFilter(cfg.MetadataAttributes)

From 21117dfcea7f3f9f35499508832f879afc6d9d2f Mon Sep 17 00:00:00 2001
From: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com>
Date: Tue, 13 Jun 2023 13:33:54 +0200
Subject: [PATCH 2/6] Apply suggestions from code review

---
 exporter/sumologicexporter/README.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/exporter/sumologicexporter/README.md b/exporter/sumologicexporter/README.md
index 3824836dca18..f39a2f924c89 100644
--- a/exporter/sumologicexporter/README.md
+++ b/exporter/sumologicexporter/README.md
@@ -28,7 +28,7 @@ The following options are deprecated and they won't exist in new version:
 After the new exporter will be moved to this repository:
 
 - `carbon2` and `graphite` are going to be no longer supported and `prometheus` or `otlp` format should be used
-- all resource level attributes are going to be treated as `metadata_attributes`. You can use [Group by Attributes processor](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/processor/groupbyattrsprocessor/) to move attributes from record level to resource level. For example:
+- all resource level attributes are going to be treated as `metadata_attributes`. You can use [Group by Attributes processor](https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/processor/groupbyattrsprocessor) to move attributes from record level to resource level. For example:
 
   ```yaml
   # before switch to new collector
@@ -43,7 +43,7 @@ After the new exporter will be moved to this repository:
         - my_attribute
   ```
 
-- Source templates (`source_category`, `source_name` and `source_host`) are going to be removed from the exporter and sources may be set using `_sourceCategory`, `sourceName` or `_sourceHost` resource attributes. We recommend to use [Transform Processor](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/processor/transformprocessor/). For example:
+- Source templates (`source_category`, `source_name` and `source_host`) are going to be removed from the exporter and sources may be set using `_sourceCategory`, `sourceName` or `_sourceHost` resource attributes. We recommend to use [Transform Processor](https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/processor/transformprocessor/). For example:
 
   ```yaml
   # before switch to new collector

From ce63f19d55b3039500be16a4f279f96df0210235 Mon Sep 17 00:00:00 2001
From: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com>
Date: Wed, 14 Jun 2023 08:25:16 +0200
Subject: [PATCH 3/6] Update exporter/sumologicexporter/README.md
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Mikołaj Świątek <mswiatek@sumologic.com>
---
 exporter/sumologicexporter/README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/exporter/sumologicexporter/README.md b/exporter/sumologicexporter/README.md
index f39a2f924c89..3eca1ffb796f 100644
--- a/exporter/sumologicexporter/README.md
+++ b/exporter/sumologicexporter/README.md
@@ -16,7 +16,7 @@
 
 For some time we have been developing [new Sumo Logic exporter](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/exporter/sumologicexporter#sumo-logic-exporter) and now we are in progress of moving it into this repository.
 
-The following options are deprecated and they won't exist in new version:
+The following options are deprecated and they will not exist in the new version:
 
 - `metric_format: {carbon2, graphite}`
 - `metadata_attributes: [<regex>]`

From a3389c2cc547f0dfbb3f3d1e71766a8a1540e6e7 Mon Sep 17 00:00:00 2001
From: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com>
Date: Wed, 14 Jun 2023 08:25:23 +0200
Subject: [PATCH 4/6] Update exporter/sumologicexporter/README.md
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Mikołaj Świątek <mswiatek@sumologic.com>
---
 exporter/sumologicexporter/README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/exporter/sumologicexporter/README.md b/exporter/sumologicexporter/README.md
index 3eca1ffb796f..8cff0ba48fa4 100644
--- a/exporter/sumologicexporter/README.md
+++ b/exporter/sumologicexporter/README.md
@@ -12,7 +12,7 @@
 
 ## Migration to new architecture
 
-**This exporter is subject of major changes now.**
+**This exporter is undergoing major changes right now.**
 
 For some time we have been developing [new Sumo Logic exporter](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/exporter/sumologicexporter#sumo-logic-exporter) and now we are in progress of moving it into this repository.
 

From 1090f2d485ae77b505f1f7d4195c0453637ebc0b Mon Sep 17 00:00:00 2001
From: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com>
Date: Wed, 14 Jun 2023 08:25:32 +0200
Subject: [PATCH 5/6] Update exporter/sumologicexporter/README.md
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Mikołaj Świątek <mswiatek@sumologic.com>
---
 exporter/sumologicexporter/README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/exporter/sumologicexporter/README.md b/exporter/sumologicexporter/README.md
index 8cff0ba48fa4..8e6a6eeb1d76 100644
--- a/exporter/sumologicexporter/README.md
+++ b/exporter/sumologicexporter/README.md
@@ -14,7 +14,7 @@
 
 **This exporter is undergoing major changes right now.**
 
-For some time we have been developing [new Sumo Logic exporter](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/exporter/sumologicexporter#sumo-logic-exporter) and now we are in progress of moving it into this repository.
+For some time we have been developing the [new Sumo Logic exporter](https://github.com/SumoLogic/sumologic-otel-collector/tree/main/pkg/exporter/sumologicexporter#sumo-logic-exporter) and now we are in the process of moving it into this repository.
 
 The following options are deprecated and they will not exist in the new version:
 

From 0afc7e98967c286c4173768b22d09aaf9bb98cb5 Mon Sep 17 00:00:00 2001
From: Alex Boten <alex@boten.ca>
Date: Wed, 14 Jun 2023 09:40:44 -0700
Subject: [PATCH 6/6] Apply suggestions from code review

---
 exporter/sumologicexporter/README.md | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/exporter/sumologicexporter/README.md b/exporter/sumologicexporter/README.md
index 8e6a6eeb1d76..f82878cbb6f3 100644
--- a/exporter/sumologicexporter/README.md
+++ b/exporter/sumologicexporter/README.md
@@ -86,7 +86,7 @@ exporters:
     # List of regexes for attributes which should be send as metadata
     # default = []
     #
-    # This opion is deprecated:
+    # This option is deprecated:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     metadata_attributes: [<regex>]
 
@@ -106,7 +106,7 @@ exporters:
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
     #
-    # This opion is deprecated:
+    # This option is deprecated:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     graphite_template: <template>
 
@@ -115,7 +115,7 @@ exporters:
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
     #
-    # This opion is deprecated:
+    # This option is deprecated:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     source_category: <template>
 
@@ -124,7 +124,7 @@ exporters:
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
     #
-    # This opion is deprecated:
+    # This option is deprecated:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     source_name: <template>
 
@@ -133,7 +133,7 @@ exporters:
     # Please regfer to Source temmplates for formatting explanation:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#source-templates
     #
-    # This opion is deprecated:
+    # This option is deprecated:
     # https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/sumologicexporter#migration-to-new-architecture
     source_host: <template>