Skip to content

chart merge #1382

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
Merged
pepov merged 17 commits into from
Jul 24, 2023
Merged

chart merge #1382

pepov merged 17 commits into from
Jul 24, 2023

Conversation

@pepov
Copy link
Member

@pepov pepov commented Jul 18, 2023 •
edited
Loading

  • copy the logging-operator-logging chart from the helm-charts repo and simplify it until it works
  • add a test receiver deployment to the chart and an example values file to be able to test effectively a simple scenario. my hope is that we can also use this as a quickstart
  • copy docs generation from the helm-charts repo
  • get rid of the charts in the e2e folder

@pepov pepov marked this pull request as ready for review July 18, 2023 16:51
@pepov pepov requested review from ahma and sagikazarmark July 18, 2023 16:51
sagikazarmark
sagikazarmark approved these changes Jul 18, 2023
View reviewed changes
Copy link
Member

@sagikazarmark sagikazarmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Made a few minor comments

pepov added 17 commits July 24, 2023 08:41
@pepov
helm: copy current version of logging-operator-logging chart back
4f4b221 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: prepare l-o values.yaml to support l-o-l values under the loggi…
b39ca7e 
…ng key. prepare templates to look for values under the new key.

Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: move and rename relocated resources to all have the logging prefix
b992a70 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: add the logging.enabled flag to disable logging by default
8123d48 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: fix template references
c783dc3 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: remove original logging-operator-logging chart
ea7e272 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: remove oracle scc and broken tls setup between fluentd and flue…
7245f10 
…ntbit

Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: introduce logging-operator.releasename template function
aee70fb 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: introduce standalone fluentbitagent resource
c6a4f0e 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: rename template
b27bbcd 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: add example receiver deployment (disabled by default) with an e…
6742fd1 
…xample config to be able to easily test the core functions

Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: generate chart docs
5fc7757 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: delete e2e charts
cd3a1aa 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: remove obsolete creationTimestamp from clusterrole
50bbebc 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: correct chart references
bc9b8fc 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
chore: run make manifests
b154f8c 
Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov
helm: remove version related info from templates as it is non-relevan…
5281ded 
…t (valid info is generated packaging time)

Signed-off-by: Peter Wilcsinszky <[email protected]>
@pepov pepov merged commit 5e36c50 into master Jul 24, 2023
@pepov pepov deleted the chart-merge branch July 24, 2023 07:49
@pepov pepov mentioned this pull request Jul 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sagikazarmark sagikazarmark sagikazarmark approved these changes

@ahma ahma ahma approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pepov @sagikazarmark @ahma