Move semconv package from pdata to its own module (#5196)

- Move `model/semconv` package to a new `semconv` module
- Add deprecation notices to API from `model/semconv`

.github/dependabot.yml

@@ -30,3 +30,7 @@ updates:
  directory: "/pdata"
  schedule:
  interval: "weekly"
+ - package-ecosystem: "gomod"
+ directory: "/semconv"
+ schedule:
+ interval: "weekly"

CHANGELOG.md

@@ -9,6 +9,8 @@
 
 ### 🚩 Deprecations 🚩
 
+- Deprecate all API in `pdata/semconv`. The package is moved to a new `semcomv` module (#5196)
+
 ### 💡 Enhancements 💡
 
 ### 🧰 Bug fixes 🧰

Makefile

@@ -297,8 +297,8 @@ gensemconv:
  @[ "${SPECPATH}" ] || ( echo ">> env var SPECPATH is not set"; exit 1 )
  @[ "${SPECTAG}" ] || ( echo ">> env var SPECTAG is not set"; exit 1 )
  @echo "Generating semantic convention constants from specification version ${SPECTAG} at ${SPECPATH}"
- semconvgen -o model/semconv/${SPECTAG} -t model/internal/semconv/template.j2 -s ${SPECTAG} -i ${SPECPATH}/semantic_conventions/resource -p conventionType=resource -f generated_resource.go
- semconvgen -o model/semconv/${SPECTAG} -t model/internal/semconv/template.j2 -s ${SPECTAG} -i ${SPECPATH}/semantic_conventions/trace -p conventionType=trace -f generated_trace.go
+ semconvgen -o semconv/${SPECTAG} -t semconv/template.j2 -s ${SPECTAG} -i ${SPECPATH}/semantic_conventions/resource -p conventionType=resource -f generated_resource.go
+ semconvgen -o semconv/${SPECTAG} -t semconv/template.j2 -s ${SPECTAG} -i ${SPECPATH}/semantic_conventions/trace -p conventionType=trace -f generated_trace.go
 
 # Checks that the HEAD of the contrib repo checked out in CONTRIB_PATH compiles
 # against the current version of this repo.
@@ -307,6 +307,7 @@ check-contrib:
  @echo Setting contrib at $(CONTRIB_PATH) to use this core checkout
  @$(MAKE) -C $(CONTRIB_PATH) for-all CMD="$(GOCMD) mod edit -replace go.opentelemetry.io/collector=$(CURDIR)"
  @$(MAKE) -C $(CONTRIB_PATH) for-all CMD="$(GOCMD) mod edit -replace go.opentelemetry.io/collector/pdata=$(CURDIR)/pdata"
+ @$(MAKE) -C $(CONTRIB_PATH) for-all CMD="$(GOCMD) mod edit -replace go.opentelemetry.io/collector/semconv=$(CURDIR)/semconv"
  @$(MAKE) -C $(CONTRIB_PATH) for-all CMD="$(GOCMD) mod edit -replace go.opentelemetry.io/collector/model=$(CURDIR)/model"
  @$(MAKE) -C $(CONTRIB_PATH) -j2 gotidy
  @$(MAKE) -C $(CONTRIB_PATH) test

cmd/otelcorecol/builder-config.yaml

@@ -28,5 +28,5 @@ processors:
 
 replaces:
  - go.opentelemetry.io/collector => ../../
- - go.opentelemetry.io/collector/model => ../../model
+ - go.opentelemetry.io/collector/semconv => ../../semconv
  - go.opentelemetry.io/collector/pdata => ../../pdata

cmd/otelcorecol/go.mod

@@ -52,8 +52,8 @@ require (
  github.com/tklauser/numcpus v0.4.0 // indirect
  github.com/yusufpapurcu/wmi v1.2.2 // indirect
  go.opencensus.io v0.23.0 // indirect
- go.opentelemetry.io/collector/model v0.49.0 // indirect
  go.opentelemetry.io/collector/pdata v0.49.0 // indirect
+ go.opentelemetry.io/collector/semconv v0.0.0-00010101000000-000000000000 // indirect
  go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.31.0 // indirect
  go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.31.0 // indirect
  go.opentelemetry.io/contrib/zpages v0.31.0 // indirect
@@ -77,6 +77,6 @@ require (
 
 replace go.opentelemetry.io/collector => ../../
 
-replace go.opentelemetry.io/collector/model => ../../model
+replace go.opentelemetry.io/collector/semconv => ../../semconv
 
 replace go.opentelemetry.io/collector/pdata => ../../pdata

go.mod

@@ -20,8 +20,8 @@ require (
  github.com/spf13/cobra v1.4.0
  github.com/stretchr/testify v1.7.1
  go.opencensus.io v0.23.0
- go.opentelemetry.io/collector/model v0.49.0
  go.opentelemetry.io/collector/pdata v0.49.0
+ go.opentelemetry.io/collector/semconv v0.0.0-00010101000000-000000000000
  go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.31.0
  go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.31.0
  go.opentelemetry.io/contrib/zpages v0.31.0
@@ -74,7 +74,7 @@ require (
  gopkg.in/yaml.v3 v3.0.0-20210107192922-496545a6307b // indirect
 )
 
-replace go.opentelemetry.io/collector/model => ./model
+replace go.opentelemetry.io/collector/semconv => ./semconv
 
 replace go.opentelemetry.io/collector/pdata => ./pdata