Skip to content

api test changes for docker #668

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

Merged
merged 3 commits into from
May 21, 2025

Conversation

rishagg01
Copy link
Collaborator

🗣 Description

Cluster - API automation test changes

💭 Motivation and context

To make the tests work after the docker pipeline code changes

Modifies API cluster tests to align with the code changes

All tests are supposed to work and pass after these changes are implemented.

📷 Screenshots (DELETE IF UNAPPLICABLE)

🧪 Testing

I ran the tests locally and they all passed. Need to run them on pipeline.

✅ Pre-approval checklist

  • 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

Updated Elastic agent version assertion
@rishagg01 rishagg01 requested review from cbaxley and NVivero May 14, 2025 13:13
@rishagg01
Copy link
Collaborator Author

Docker Pipeline / build-24-04 (pull_request) failed at test container installation set up.
image

@cbaxley
Copy link
Collaborator

cbaxley commented May 14, 2025

Docker Pipeline / build-24-04 (pull_request) failed at test container installation set up. image

You can just rerun that individual job. You don't have to run all three if one fails.

@rishagg01
Copy link
Collaborator Author

Docker Pipeline / build-24-04 (pull_request) failed at test container installation set up. image

You can just rerun that individual job. You don't have to run all three if one fails.

I did. All tests have passed and ready to merge. Awaiting approval.

@rishagg01 rishagg01 merged commit 8cec1cb into cbaxley-644-docker-pipeline May 21, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from 🆕 Product Backlog to ✅ Done in LME-Development May 21, 2025
@rishagg01 rishagg01 deleted the rishagg-dockerpipeline2 branch May 21, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants