Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: updated the logging-operator version #2818

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

feat: updated the logging-operator version #2818

wants to merge 15 commits into from

Conversation

tushar-wagh-nx
Copy link
Contributor

…or-logging

What problem does this PR solve?:
It will update the version of logging-operator and remove the older logging-operator-logging

Which issue(s) does this PR fix?:

https://jira.nutanix.com/browse/NCN-103706

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Checklist

  • If the PR adds a version bump, ensure there is no breaking change in Licensing model (or NA).
  • If a chart is changed or app configuration is significantly changed, the chart version is correctly incremented (so that apps are not automatically upgraded from a previous version of DKP).

@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off open-kommander-pr Automatically triggers the creation of a PR in Kommander repo services/logging-operator labels Nov 11, 2024
@tushar-wagh-nx tushar-wagh-nx self-assigned this Nov 11, 2024
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 11, 2024
@mesosphere-ci mesosphere-ci added do-not-merge/testing Do not merge because there is still on-going testing update-licenses signals mergebot to update licenses.d2iq.yaml labels Nov 11, 2024
@github-actions GitHub Actions
Copy link
Contributor

✅ Created Kommander branch to test kommander-applications changes: https://github.com/mesosphere/kommander/tree/kapps/main/bump-logging-operator

@coveralls
Copy link

coveralls commented Nov 11, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 11818378111

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 51.703%
Totals Coverage Status
Change from base Build 11814012008: 0.0%
Covered Lines: 167
Relevant Lines: 323
💛 - Coveralls

@tushar-wagh-nx tushar-wagh-nx changed the title feat: updated the logging-operator version and removed logging-operat… feat: updated the logging-operator version Nov 11, 2024
@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Nov 11, 2024
@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Nov 11, 2024
@tushar-wagh-nx tushar-wagh-nx force-pushed the bump-logging-operator branch 2 times, most recently from b314c69 to ddb1812 Compare November 11, 2024 15:51
@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Nov 11, 2024
@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Nov 11, 2024
@tushar-wagh-nx tushar-wagh-nx removed the ok-to-test Signals mergebot that CI checks are ready to be kicked off label Nov 12, 2024
@tushar-wagh-nx tushar-wagh-nx added the ok-to-test Signals mergebot that CI checks are ready to be kicked off label Nov 12, 2024
@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Nov 12, 2024
@tushar-wagh-nx tushar-wagh-nx added ok-to-test Signals mergebot that CI checks are ready to be kicked off and removed ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Nov 12, 2024
@tushar-wagh-nx
Copy link
Contributor Author

tushar-wagh-nx commented Nov 12, 2024 •
edited
Loading

Hi @mhrabovcin
Request you to please review this PR and let me know what I am missing here. I tried multiple image tags for fluentd but every time it's getting failing in Check licenses.d2iq.yaml

@tushar-wagh-nx tushar-wagh-nx removed the ok-to-test Signals mergebot that CI checks are ready to be kicked off label Nov 13, 2024
@mesosphere-ci mesosphere-ci added the ok-to-test Signals mergebot that CI checks are ready to be kicked off label Nov 13, 2024
@tushar-wagh-nx
feat: updated the logging-operator version and removed logging-operat…
0a39efd 
…or-logging
@tushar-wagh-nx
feat: integrated logging-operator-logging into main cm.yaml
7c1e84a
@tushar-wagh-nx
feat: updated the image paths in extra-images.txt
3f084cc
@tushar-wagh-nx
feat: updated the fluentd to v1.16-4.8-filters-build.155
fb8032d
@tushar-wagh-nx
feat: reverted the fluentd to v1.16-4.10-filters-build.156
70d4949
@tushar-wagh-nx
feat: added newline in logging-operator.yaml
9057a2d
@tushar-wagh-nx
feat: added newline in cm.yaml
6f6fb38
@tushar-wagh-nx
feat: updated the fluentd to v1.16-4.8-filters-build.155
6db115f
@tushar-wagh-nx
feat: updated the fluentd to v1.16.6
50a924d
@tushar-wagh-nx
feat: updated fluentd version to v1.16-4.10-filters-build.156
2cfe36c
@tushar-wagh-nx
feat: updated refs for fluentd,node-exporter,logging-operator and con…
d2fe09b 
…fig-reloader
@tushar-wagh-nx
feat: updated refs for fluentd
b37d0e3
@tushar-wagh-nx
feat: updated source.toolkit.fluxcd.io/v1beta2 to fluxcd.io/v1
860cb91
@tushar-wagh-nx
feat: reverted version to source.toolkit.fluxcd.io/v1beta2
0374197
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mhrabovcin mhrabovcin Awaiting requested review from mhrabovcin

At least 1 approving review is required to merge this pull request.

Assignees

@tushar-wagh-nx tushar-wagh-nx

Labels
do-not-merge/testing Do not merge because there is still on-going testing ok-to-test Signals mergebot that CI checks are ready to be kicked off open-kommander-pr Automatically triggers the creation of a PR in Kommander repo services/logging-operator size/L Denotes a PR that changes 100-499 lines, ignoring generated files. update-licenses signals mergebot to update licenses.d2iq.yaml
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tushar-wagh-nx @coveralls @mesosphere-ci