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

Fix tests after password encryption #466

Merged
merged 50 commits into from
Oct 17, 2024
Merged

Fix tests after password encryption #466

merged 50 commits into from
Oct 17, 2024

Conversation

cbaxley
Copy link
Collaborator

@cbaxley cbaxley commented Oct 10, 2024 •
edited
Loading

🗣 Description

The tests need to use the new passwords and urls that the encryption changes added

🧪 Testing

Make sure the tests in the pipelines pass

✅ 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.

@cbaxley cbaxley changed the base branch from main to release-2.0.0 October 10, 2024 10:51
@cbaxley
Don't remove the Azure resources at the end of the Linux only tests
3b8db23
@cbaxley
Change the variables to work with the 2.0 pipeline
bcf92c5
@cbaxley
Run the cluster run workflow to debug the tests
df108a1
@cbaxley
Run ansible playbook to set fleet
0e3b95d
@cbaxley
Log secrets in set_fleet.yml
f9e986c
@cbaxley
Debug setting the environment variables in set_fleet.yml
adf8557
@cbaxley
Set the debug mode in set_fleet.yml
084b027
@cbaxley
Log the Fleet API call details in set_fleet.yml
f526d9e
@cbaxley
Loop through Fleet API calls in set_fleet.yml
a6d791c
@cbaxley
Change the Fleet API call to loop through attempts in set_fleet.yml
e5c37d5
@cbaxley
Try to set the Fleet API with retries in set_fleet.yml
754cf6c
@cbaxley
Attempt to output the Fleet API call details in set_fleet.yml
a196563
@cbaxley
A new way to handle the Fleet API call in set_fleet.yml
22ad2c1
@cbaxley
Export the check_fleet_api.yml file in set_fleet.yml
de554ec
@cbaxley
Make sure the password is in the check_fleet_api.yml file
832920d
@cbaxley
Exit the loop if the Fleet API call succeeds in check_fleet_api.yml
0089796
@cbaxley
Wait a little longer for the results to be written to the index
a6630e8
@cbaxley
Update the cluster.yml workflow to wait a little longer for the resul…
077cdfa 
…ts to be written to the index
@cbaxley
Delay in a different way
ac39458
@cbaxley
Attempt a different looping method
ff25324
@cbaxley
Remove the set-fleet script from the installer
877e268
@cbaxley
Reverts to old loop method
2fefce6
@cbaxley
Check that fleet is ready in an external script
d6fc98f
@cbaxley
Call the ansible playbook from the install script
f4cbd7a
@cbaxley
Get the CA fingerprint from the Elasticsearch container
9ec8542
@cbaxley
Adds headers to the curl commands in the set_fleet.yml playbook
03ac6a9
@cbaxley
Address the hosts and fleet API issues
5b603b9
@cbaxley
Change the way we login to the Kibana API
35e9121
cbaxley and others added 14 commits October 16, 2024 07:54
@cbaxley
Increase the timeout for the Endpoint Policy API calls
519bcf8
@cbaxley
Increase the timeout for the Endpoint Policy API calls
c9de1e1
@cbaxley
Increase the timeout for the Defend Policy API calls
05c5ad0
@cbaxley
Only print debug information if debug_mode is true
50e9cb5
@cbaxley
Keeps the azure resources on builds
3efbe3e
@mreeve-snl
Fixing Error with certs where the permissions should only be on first
d924ea1 
generation!
@mreeve-snl
Remove sysctl edits to lower privileged ports and add 443 to kibana c…
4dd138a 
…ontainer
@mreeve-snl
Add notes on starting vms via azure cli to testing v2
7eb4e5d
@mreeve-snl
Fix ansible errors in checking for passwords that are created
4a9109e
@mreeve-snl
Add debugging commands, and remove references to 443 for kibana from
8bf0477 
debug commands
@cbaxley
Update the cluster.yml file to use the new IP address for the Azure i…
d06c3f3 
…nstance
@cbaxley
Only allow the ip address of the host to connect to the azure instance
3d2fdef
@mreeve-snl
remove unnecassary script
b9801c6
@cbaxley
Move ansible files to the ansible directory
18fb506
@mreeve-snl
Moving Upgrade Readme into upgrade directory
2ea0568
@mreeve-snl
Merge branch 'cbaxley-fix-tests-after-password-encryption' of github.…
a75bbb5 
…com:cisagov/LME into cbaxley-fix-tests-after-password-encryption
@mreeve-snl
Add upgrading docs and remove dev notes
b3610b6
@mreeve-snl
Update main readme docs:
f96cde0 
- add section for LME introductory content
- disclaimer around small simple siem
- add Pre-Requisites page
- add Upgrading 1.4 -> 2.0 docs
- note on lme-frontend coming later
- remove references to lmed and make docs accurate to current
  installation process
- add TODOs for docs that still need updated
@mreeve-snl
Adding updated cloud docs and firewall explanation
e26f0bc
@mreeve-snl
Update FAQ and Pre-requisites
a9da11d
mreeve-snl
mreeve-snl approved these changes Oct 17, 2024
View reviewed changes
Copy link
Collaborator

@mreeve-snl mreeve-snl left a comment

Choose a reason for hiding this comment

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

LGTM

@mreeve-snl mreeve-snl merged commit 1733589 into release-2.0.0 Oct 17, 2024
4 checks passed
@mreeve-snl
Copy link
Collaborator

merging and closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mreeve-snl mreeve-snl mreeve-snl approved these changes

@rishagg01 rishagg01 rishagg01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
LME-Development
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cbaxley @mreeve-snl @rishagg01