Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/kibana] Migration Tracker #14935

Closed
10 tasks
compleatang opened this issue Jun 20, 2019 · 12 comments · Fixed by #23844
Closed
10 tasks

[stable/kibana] Migration Tracker #14935

compleatang opened this issue Jun 20, 2019 · 12 comments · Fixed by #23844

Comments

@compleatang
Copy link

compleatang commented Jun 20, 2019

In collaboration with the team at elastic, @monotek and I the maintainers of the kibana chart have decided to put the chart into a "pre-depreaction" status. Given that the elasticsearch chart is moving over it makes sense to move Kibana over too.

Old Chart: https://github.com/helm/charts/tree/master/stable/kibana
New Chart: https://github.com/elastic/helm-charts/tree/master/kibana

This issue should operate as the central tracking issue for work that has been and will be accomplished in this migration path.

General

  • Add a pre-migration notice to stable/kibana's README
  • Close off all issues that deal with v7.X upgrades and refer them to this issue and/or the elastic charts for their improvements to the stable/kibana chart
  • Decide a deadline for "actually" deprecating the chart and follow the procedures here
  • Finalize whether elastic/kibana needs a migration guide similar to elastics ... my early view is that there is no need due to the relatively stateless positioning of stable/kibana

Specific Functionality

The below functionality of the stable/kibana chart should ideally be migrated to elastic/kibana:

  • PVCs for plugins (whole file)
  • Service Account flexibility (whole file rather than simple invocation
  • Services:
    • Labels
    • loadBalancerSourceRanges
    • clusterIP (ability to customize)
    • loadBalancerIP
    • customizable selectors
    • non-cluster-IP service types
    • authProxyEnabled functionality
  • Ingress:
  • Dashboard import feature
  • Deployments
    • deployment annotations
    • revisionHistoryLimit
    • podAnnotations
    • podLabels
    • init containers work
    • pullPolicy
    • commandline.args
    • envFromSecrets
    • authProxyEnabled
    • probes flexibility (timing mostly)
    • extraVolumeMounts
    • extraContainers
    • extraConfigMapMounts
    • extraVolumes
    • dashboardImport.enabled
    • plugins.enabled

Test functionality

It's unclear to me at this time what we should do with the tests because it appears that the helm-elastic repo has it's own python based test runner.

cc @monotek @Crazybus @jethr0null if I missed anything.

@stale
Copy link

stale bot commented Jul 20, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 20, 2019
@compleatang
Copy link
Author

not stale. still actionable.

@stale stale bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 22, 2019
@monotek
Copy link
Collaborator

monotek commented Aug 6, 2019

pr to add dashboard import: elastic/helm-charts#244

@stale
Copy link

stale bot commented Sep 5, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Sep 5, 2019
@monotek
Copy link
Collaborator

monotek commented Sep 7, 2019

Not stale.

@stale stale bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Sep 7, 2019
@stale
Copy link

stale bot commented Oct 7, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 7, 2019
@monotek
Copy link
Collaborator

monotek commented Oct 8, 2019

Not stale.

@stale stale bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 8, 2019
@stale
Copy link

stale bot commented Nov 7, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 7, 2019
@monotek
Copy link
Collaborator

monotek commented Nov 8, 2019

@compleatang
Imho we can deprecate the chart.

Im pretty sure we'll see PRs pretty soon, if some of the features are needed in the official chart.
To be honest, i don't even use Kibana at the moment, as we switched to Stackdriver at work and Im i also thinking about to try out Loki for a private project, so i lost a bit of interest in EFK stack.

@stale stale bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 8, 2019
@compleatang
Copy link
Author

actually, same, we've moved to stackdriver as well.

@monotek
Copy link
Collaborator

monotek commented Oct 5, 2020

I've added a pr to deprecate the chart: #23844

@compleatang
Copy link
Author

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants