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.

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

Update Online/Offline Installer for Observability #103

Merged

Conversation

taohe1012
Copy link
Contributor

@taohe1012 taohe1012 commented Aug 18, 2022

Description

Documentation update of Installer and offline installer for CSM
Observability deployment:

  • online installer support for Observability PowerScale;
  • online installer support for Observability PowerScale Authorization;
  • update offline installer documentation for Observability PowerScale;
  • update offline installer for Observability to enable Authorization;
  • add upgrade steps for offline installer in documentation.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#452

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have made corresponding changes to the documentation
  • I have maintained backward compatibility

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Test A: manual test for online installer: the Observability including powerscale is installed and upgraded successfully.

  • Test B: manual test for online installer: the Observability with enabling powerscale/powerflex Authorization is installed and upgraded successfully.

  • Test C: manual test for offline installer: the Observability with powerscale/powerflex Authorization is installed and upgraded successfully.

@taohe1012 taohe1012 force-pushed the feature-422-update-observability-installation-documentation branch 2 times, most recently from 069d60f to 68880bb Compare August 18, 2022 10:13

**Note**: Assuming that Your Secrets from CSI Drivers have been copied to the Karavi Observability namespace in the phase of `#Offline Karavi Observability Helm Chart Installer`.

**Note**: Optionally, you could provide your own [configurations](../docs/GETTING_STARTED_GUIDE.md#Configuration). A sample values.yaml file is located [here](https://github.com/dell/helm-charts/blob/main/charts/karavi-observability/values.yaml).
Copy link
Contributor

@baoy1 baoy1 Aug 18, 2022

Choose a reason for hiding this comment

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

Is ../docs/GETTING_STARTED_GUIDE.md#Configuration a valid location?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@baoy1 not available now, will remove this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@baoy1 updated, its link is removed

@taohe1012 taohe1012 force-pushed the feature-422-update-observability-installation-documentation branch from 68880bb to 5f6f82d Compare August 18, 2022 12:14
YianZong
YianZong previously approved these changes Aug 19, 2022
forrestxia
forrestxia previously approved these changes Aug 19, 2022
baoy1
baoy1 previously approved these changes Aug 19, 2022
P-Cao
P-Cao previously approved these changes Aug 19, 2022
installer/README.md Outdated Show resolved Hide resolved
installer/README.md Outdated Show resolved Hide resolved
installer/README.md Outdated Show resolved Hide resolved
Documentation update of Installer and offline installer for CSM
Observability deployment:
 - online installer support for Observability PowerScale;
 - online installer support for Observability PowerScale Authorization;
 - update offline installer documentation for Observability PowerScale;
 - update offline installer for Observability to enable Authorization;
 - add upgrade steps for offline installer in documentation.
@taohe1012 taohe1012 dismissed stale reviews from P-Cao, baoy1, and forrestxia via 5b242d4 August 19, 2022 13:05
@taohe1012 taohe1012 force-pushed the feature-422-update-observability-installation-documentation branch from 5f6f82d to 5b242d4 Compare August 19, 2022 13:05
@taohe1012 taohe1012 merged commit 1939fe4 into main Aug 22, 2022
@taohe1012 taohe1012 deleted the feature-422-update-observability-installation-documentation branch August 22, 2022 06:50
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.

7 participants