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

Mreeve 461 post install #477

Merged
merged 11 commits into from
Oct 23, 2024
Merged

Mreeve 461 post install #477

merged 11 commits into from
Oct 23, 2024

Conversation

mreeve-snl
Copy link
Collaborator

@mreeve-snl mreeve-snl commented Oct 22, 2024

🗣 Description

This adds features to run the post install script and adds in documentation as well.

Features/Fixes:

  • Remove optional password setting and add in small changes to Readme.MD
    
  • Updates and general fixes across docs
    
  • creates post-install.yml: 
    
    1. setting up fleet
    2. fixing a few issues with wazuh (in a future release this won't be necessary)
    3. setting up custom LME dashboards
    4. setting up wazuh's dashboards
    5. setting up a read only user for analysts to connect and query LME's data
  • Fixing passwords in init-setup
    
  • adding starting dashboards docs + scripting
    
  • Remove memory limitations in kibana/elasticsearch quadlets
    

💭 Motivation and context

closes #461
closes #443
We need something to run post-install steps to finalize setup of LME containers/services.

🧪 Testing

tested post-install.yml on a clean box, also the pipeline will test to see if things break

will do a test of all steps on a clean box before merge

✅ Pre-approval checklist

  • There is a gitIssue that this PR resolves
  • Git Issue that this PR solves has been selected in the Development section
  • The PR's base branch has been modified to be the proper branch.
  • Changes are limited to a single goal AND
    the title reflects this in a clear human readable format
  • Issue that this PR solves has been selected in the Development section
  • I have read and agree to LME's CONTRIBUTING.md document.
  • The PR adheres to LME's requirements in RELEASES.md
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated to reflect the changes in this PR.

✅ Pre-merge Checklist

  • All tests pass.
  • PR has been tested and the documentation for testing is above.
  • Squash and merge all commits into one PR level commit.

✅ Post-merge Checklist

  • Delete the branch to keep down number of branches.
  • The PR is labeled with feat for an added new feature, update for an update, OR fix for a fix.
  • The PR contains Resolves #<issue #> so that merging it closes out the corresponding issue. For example Resolves #132.

@mreeve-snl mreeve-snl changed the base branch from main to release-2.0.0 October 22, 2024 03:47
Copy link
Collaborator

@cbaxley cbaxley left a comment

Choose a reason for hiding this comment

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

We won't have them install the old dashboards. We have an importer in the upgrade steps that renames them and imports them.

@mreeve-snl
Copy link
Collaborator Author

Passed linux tests,
running on cluster, then merging: https://github.com/cisagov/LME/actions/runs/11465538956

@mreeve-snl
Copy link
Collaborator Author

@cbaxley
Copy link
Collaborator

cbaxley commented Oct 22, 2024

We won't have them install the old dashboards. We have an importer in the upgrade steps that renames them and imports them.

This is fine. :)

@mreeve-snl
Copy link
Collaborator Author

testing post install on a new machine, since thats not in the tests then merging

@mreeve-snl
Copy link
Collaborator Author

OK this is good to merge whenever :) I'll merge this tomorrow if I don't see it merged then

@mreeve-snl mreeve-snl merged commit abba9d3 into release-2.0.0 Oct 23, 2024
4 checks passed
@mreeve-snl mreeve-snl deleted the mreeve-461-post-install branch October 23, 2024 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
3 participants