Skip to content

Remove obsolete files from kubernetes-addons/Observability/#821

Merged
yongfengdu merged 1 commit intoopea-project:mainfrom
eero-t:rm-old-files
Feb 20, 2025
Merged

Remove obsolete files from kubernetes-addons/Observability/#821
yongfengdu merged 1 commit intoopea-project:mainfrom
eero-t:rm-old-files

Conversation

@eero-t
Copy link
Copy Markdown
Collaborator

@eero-t eero-t commented Feb 19, 2025

Description

Remove files obsoleted by previous PRs:

  • YAML & JSON files that are non-configurable duplicates of things already installed by Helm charts
  • Large grafana-configs & pcm.metrics files not referred or used by anything

(Cleanup after PR #800, done as separate PR to ease review.)

Issues

n/a.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Dependencies

n/a.

Tests

CI.

* YAML & JSON files are non-configurable duplicates of things already
  installed by Helm charts
* grafana-configs & pcm.metrics are unused large leftoever files

Signed-off-by: Eero Tamminen <eero.t.tamminen@intel.com>
@eero-t
Copy link
Copy Markdown
Collaborator Author

eero-t commented Feb 19, 2025

My branch / PR has no merge commit, just a single commit only removes files, on top of main.

@poussa @lianhao Any idea what this is about:

Merging is blocked
This branch must not contain merge commits. 

EDIT: these seem to come now for all PRs...

@yongfengdu yongfengdu merged commit 235f22f into opea-project:main Feb 20, 2025
5 checks passed
@eero-t eero-t deleted the rm-old-files branch February 21, 2025 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants