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

fix: combining splunk versions #246

Merged
merged 2 commits into from
Mar 26, 2024

Conversation

harshilgajera-crest
Copy link
Contributor

Added stage to combine splunk versions an use it in modinput test job.

Job in AWS addon where modinut is running successfully with python3.9 (Matrix of 24 jobs 3*8): https://github.com/splunk/splunk-add-on-for-amazon-web-services/actions/runs/8431526533

Job in MSCS addon : https://github.com/splunk/splunk-add-on-for-microsoft-cloud-services/actions/runs/8431532363

@harshilgajera-crest harshilgajera-crest marked this pull request as ready for review March 26, 2024 10:08
@harshilgajera-crest harshilgajera-crest requested a review from a team as a code owner March 26, 2024 10:08
@@ -276,6 +277,12 @@ jobs:
run: |
echo "splunk={\"version\":\"unreleased-python3_9-7027496d63d8\", \"build\":\"7027496d63d8\", \"islatest\":false, \"isoldest\":false}" >> "$GITHUB_OUTPUT"
echo "sc4s={\"version\":\"2.49.5\", \"docker_registry\":\"ghcr.io/splunk/splunk-connect-for-syslog/container2\"}" >> "$GITHUB_OUTPUT"
- name: combined_Splunkmatrix
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider removing python39_Splunk and adding logic to combined_Splunkmatrix.
Then all test jobs would require update of matrix and removal of python39 section.

Copy link
Contributor

Choose a reason for hiding this comment

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

Lets postpone those changes for now and focus on delivering fix for modinput tests.

@mkolasinski-splunk mkolasinski-splunk self-requested a review March 26, 2024 10:34
@harshilgajera-crest harshilgajera-crest merged commit 2a5af03 into main Mar 26, 2024
6 checks passed
@harshilgajera-crest harshilgajera-crest deleted the fix/python3.9-splunk-matrix-issue branch March 26, 2024 10:39
@github-actions github-actions bot locked and limited conversation to collaborators Mar 26, 2024
@srv-rr-github-token
Copy link

🎉 This PR is included in version 4.15.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants