Skip to content

Commit

Permalink
chore: Remove logging.yaml (#1809)
Browse files Browse the repository at this point in the history
* remove logging.yaml

* remove logging files
  • Loading branch information
dpaasman00 committed Sep 11, 2024
1 parent cb8e9ef commit 551c22e
Show file tree
Hide file tree
Showing 9 changed files with 2 additions and 56 deletions.
17 changes: 0 additions & 17 deletions .goreleaser.yml
Original file line number Diff line number Diff line change
Expand Up @@ -112,9 +112,6 @@ archives:
- src: release_deps/opentelemetry-java-contrib-jmx-metrics.jar
dst: "."
strip_parent: true
- src: release_deps/logging.yaml
dst: "."
strip_parent: true
- src: release_deps/plugins/*
dst: plugins
strip_parent: true
Expand Down Expand Up @@ -160,14 +157,6 @@ nfpms:
mode: 0755
owner: observiq-otel-collector
group: observiq-otel-collector
# TODO: (dakota) evaluate this file once supervisor has support for configuring agent logging
- src: release_deps/logging.yaml
dst: /opt/observiq-otel-collector/logging.yaml
file_info:
mode: 0640
owner: observiq-otel-collector
group: observiq-otel-collector
type: config|noreplace
- src: LICENSE
dst: /opt/observiq-otel-collector/LICENSE
file_info:
Expand Down Expand Up @@ -265,7 +254,6 @@ dockers:
extra_files:
- release_deps/VERSION.txt
- release_deps/plugins
- config/logging.stdout.yaml
- LICENSE
- id: scratch-arm64
goos: linux
Expand All @@ -288,7 +276,6 @@ dockers:
extra_files:
- release_deps/VERSION.txt
- release_deps/plugins
- config/logging.stdout.yaml
- LICENSE
- id: ubuntu-amd64
goos: linux
Expand Down Expand Up @@ -333,7 +320,6 @@ dockers:
extra_files:
- release_deps/VERSION.txt
- release_deps/plugins
- config/logging.stdout.yaml
- LICENSE
- release_deps/opentelemetry-java-contrib-jmx-metrics.jar
- id: ubuntu-arm64
Expand Down Expand Up @@ -379,7 +365,6 @@ dockers:
extra_files:
- release_deps/VERSION.txt
- release_deps/plugins
- config/logging.stdout.yaml
- LICENSE
- release_deps/opentelemetry-java-contrib-jmx-metrics.jar

Expand Down Expand Up @@ -408,7 +393,6 @@ dockers:
extra_files:
- release_deps/VERSION.txt
- release_deps/plugins
- config/logging.stdout.yaml
- LICENSE
- release_deps/opentelemetry-java-contrib-jmx-metrics.jar
- id: ubi8-arm64
Expand Down Expand Up @@ -436,7 +420,6 @@ dockers:
extra_files:
- release_deps/VERSION.txt
- release_deps/plugins
- config/logging.stdout.yaml
- LICENSE
- release_deps/opentelemetry-java-contrib-jmx-metrics.jar

Expand Down
1 change: 0 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,6 @@ release-prep:
@echo 'v$(CURR_VERSION)' > release_deps/VERSION.txt
./buildscripts/download-dependencies.sh release_deps
@cp -r ./plugins release_deps/
@cp config/logging.yaml release_deps/logging.yaml
@cp service/com.observiq.collector.plist release_deps/com.observiq.collector.plist
@jq ".files[] | select(.service != null)" windows/wix.json >> release_deps/windows_service.json
@cp service/observiq-otel-collector.service release_deps/observiq-otel-collector.service
Expand Down
2 changes: 0 additions & 2 deletions config/logging.stdout.yaml

This file was deleted.

9 changes: 0 additions & 9 deletions config/logging.yaml

This file was deleted.

1 change: 0 additions & 1 deletion docker/Dockerfile.scratch
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ COPY LICENSE /licenses/observiq-otel-collector.license
COPY release_deps/VERSION.txt /etc/otel/VERSION.txt

COPY release_deps/plugins /etc/otel/plugins
COPY config/logging.stdout.yaml /etc/otel/logging.yaml

# Scratch images contain nothing by default. The built image
# will contain only what was copied into it. This means it
Expand Down
8 changes: 0 additions & 8 deletions docker/Dockerfile.ubi8
Original file line number Diff line number Diff line change
Expand Up @@ -62,14 +62,6 @@ COPY opampsupervisor /collector/opampsupervisor
COPY release_deps/opentelemetry-java-contrib-jmx-metrics.jar /opt/opentelemetry-java-contrib-jmx-metrics.jar
COPY release_deps/plugins /etc/otel/plugins

COPY config/logging.stdout.yaml /etc/otel/logging.yaml

RUN chown otel:otel \
/etc/otel/logging.yaml

RUN chmod 0600 \
/etc/otel/logging.yaml

USER otel
WORKDIR /etc/otel

Expand Down
9 changes: 0 additions & 9 deletions docker/Dockerfile.ubuntu
Original file line number Diff line number Diff line change
Expand Up @@ -61,15 +61,6 @@ COPY observiq-otel-collector /collector/observiq-otel-collector
COPY opampsupervisor /collector/opampsupervisor

COPY release_deps/opentelemetry-java-contrib-jmx-metrics.jar /opt/opentelemetry-java-contrib-jmx-metrics.jar
COPY release_deps/plugins /etc/otel/plugins

COPY config/logging.stdout.yaml /etc/otel/logging.yaml

RUN chown otel:otel \
/etc/otel/logging.yaml

RUN chmod 0600 \
/etc/otel/logging.yaml

USER otel
WORKDIR /etc/otel
Expand Down
10 changes: 2 additions & 8 deletions scripts/install/install_macos.sh
Original file line number Diff line number Diff line change
Expand Up @@ -498,10 +498,10 @@ install_package() {
info "Copying artifacts to install directory..."
increase_indent

# This find command gets a list of all artifacts paths except logging.yaml, or opentelemetry-java-contrib-jmx-metrics.jar
# This find command gets a list of all artifacts paths except opentelemetry-java-contrib-jmx-metrics.jar
FILES=$(
cd "$TMP_DIR/artifacts"
find "." -type f -not \( -name logging.yaml -or -name opentelemetry-java-contrib-jmx-metrics.jar \)
find "." -type f -not \( -name opentelemetry-java-contrib-jmx-metrics.jar \)
)
# Move files to install dir
for f in $FILES; do
Expand All @@ -511,12 +511,6 @@ install_package() {
decrease_indent
succeeded

if [ ! -f "$INSTALL_DIR/logging.yaml" ]; then
info "Copying default logging.yaml..."
cp "$TMP_DIR/artifacts/logging.yaml" "$INSTALL_DIR/logging.yaml" || error_exit "$LINENO" "Failed to copy default logging.yaml to install dir"
succeeded
fi

# If an endpoint was specified, we need to write the supervisor.yaml
if [ -n "$OPAMP_ENDPOINT" ]; then
create_supervisor_config "$SUPERVISOR_YML_PATH"
Expand Down
1 change: 0 additions & 1 deletion scripts/install/install_unix.sh
Original file line number Diff line number Diff line change
Expand Up @@ -694,7 +694,6 @@ create_supervisor_config() {
command printf ' accepts_remote_config: true\n' >>"$supervisor_yml_path"
command printf ' reports_remote_config: true\n' >>"$supervisor_yml_path"
command printf 'agent:\n' >>"$supervisor_yml_path"
# TODO(dakota): Add logging config option when supervisor suppports it
command printf ' executable: "%s"\n' "$INSTALL_DIR/observiq-otel-collector" >>"$supervisor_yml_path"
command printf ' description:\n' >>"$supervisor_yml_path"
command printf ' non_identifying_attributes:\n' >>"$supervisor_yml_path"
Expand Down

0 comments on commit 551c22e

Please sign in to comment.