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

feat: run both ucc and legacy modinput tests #343

Merged
merged 21 commits into from
Dec 4, 2024

Conversation

dvarasani-crest
Copy link
Contributor

@dvarasani-crest dvarasani-crest commented Oct 18, 2024

@dvarasani-crest dvarasani-crest changed the title feat: run both ucc and legacy modinput tests[WIP] feat: run both ucc and legacy modinput tests Oct 28, 2024
@dvarasani-crest dvarasani-crest marked this pull request as ready for review October 28, 2024 06:20
@dvarasani-crest dvarasani-crest requested a review from a team as a code owner October 28, 2024 06:20
@dvarasani-crest dvarasani-crest force-pushed the feat/run-both-ucc-and-legacy-modinput-tests branch 2 times, most recently from dd58379 to 7df7d65 Compare November 29, 2024 12:21
mkolasinski-splunk and others added 18 commits November 29, 2024 18:02
Test runs:
- [ ]
https://github.com/splunk/splunk-add-on-for-unix-and-linux/actions/runs/10632526511
- [ ]
https://github.com/splunk/splunk-add-on-for-salesforce/actions/runs/10632622057
- [ ]
https://github.com/splunk/splunk-add-on-for-amazon-web-services/actions/runs/10632591109
- [ ]
https://github.com/splunk/splunk-add-on-for-google-cloud-platform/actions/runs/10632560662
- [ ]
https://github.com/splunk/splunk-add-on-for-jboss/actions/runs/10632609697

Changelog:
- Extend Workflow Summary with results of tests - providing one glance
overview of tests results together with links to test reports
- Workflow dispatch with `custom-version` input to enable TA maintenance
releases
- List of OS for scripted input tests can now be defined as an input
- Splunk builds are partitioned - this feature was restored as is being
utilised in specific TA scenarios
- New input:`execute-tests-on-push-to-release` defaulting to `false` to
allow control over workflows execution on push event to `release/*`
branches
- replace `returntocorp/semgrep` with Splunk reusable workflow
`splunk/sast-scanning`
- remove cim-field-report from reusable workflow
- Fix errors when uploading diags for failed jobs when encountering
duplicated names in markers run scenario
- Fix argo logs s3 download path - now argo logs will be available to
review as a workflow artifact
- scripted-inputs-tests execution is now not limited to `main` branch
only

Deps updates:
- amannn/action-semantic-pull-request to v5.5.3
- trufflesecurity/trufflehog to v3.81.9
- addonfactory-test-matrix-action automatically updated to v2.1.7 (SC4S
update 3.28.1 -> 3.30.0)
- k8s-manifests bump v3.0.3 -> v3.0.5 (ESCU tests and cim-field-report
removal)

---------

Co-authored-by: Artem Rys <[email protected]>
Co-authored-by: kdoroszko-splunk <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: kgwizdz-splunk <[email protected]>
Co-authored-by: Adam Wownysz <[email protected]>
Co-authored-by: dvarasani-crest <[email protected]>
Co-authored-by: Marcin Bruzda <[email protected]>
Co-authored-by: harshilgajera-crest <[email protected]>
This PR bumps `k8s-manifests-branch` version.
It fixes the below items,
- Stopped the diag generation in case of passed tests
- fixed cim-compliance-report generation

Related PRs:
(Stop diag generation)
- splunk/ta-automation-app-of-apps#21
- splunk/ta-automation-k8s-manifests#104
-
https://cd.splunkdev.com/workflow-engine/ta/ta-automation-cluster/-/merge_requests/63
(fix cim-compliance-report)
- splunk/ta-automation-k8s-manifests#105


Test workflow run:
https://github.com/splunk/splunk-add-on-for-mysql/actions/runs/10767934733/job/29856231470
Fix the issue where automated releases where not created due to
incorrect conditions for pre-publish job.

Tests:
PR scenario:
https://github.com/splunk/test-addonfactory-repo/actions/runs/11027730555
release scenario:
https://github.com/splunk/test-addonfactory-repo/actions/runs/11028553116
custom version release:
https://github.com/splunk/test-addonfactory-repo/actions/runs/11047251558

---------

Co-authored-by: kdoroszko-splunk <[email protected]>
@dvarasani-crest dvarasani-crest force-pushed the feat/run-both-ucc-and-legacy-modinput-tests branch from 7df7d65 to 44df78b Compare November 29, 2024 12:32
@dvarasani-crest dvarasani-crest changed the base branch from main to develop November 29, 2024 12:32
dvarasani-crest and others added 2 commits December 2, 2024 11:50
This PR resolves the issue with the test results summary.
Also, fixed the issue with the `test-btool` workflow status check.

Issue reference:
workflow run:
https://github.com/splunk/splunk-add-on-for-amazon-web-services/actions/runs/12072808241
check the ucc modinput tests results summary. It shows total number of
tests and the errors count as 6. However, if you check the test report,
there were only 3 tests ran(each test case had error in the setup and
teardown method resulting in 6 errors).

Test workflow:
https://github.com/splunk/splunk-add-on-for-amazon-web-services/actions/runs/12136591898
README.md Outdated Show resolved Hide resolved
@kdoroszko-splunk
Copy link
Contributor

It seems like splunk-add-on-ucc-modinput-test-functional.log is being generated as an artifact for each test type (even for UI and KO). Is it possible to change that on our side?

@dvarasani-crest
Copy link
Contributor Author

It seems like splunk-add-on-ucc-modinput-test-functional.log is being generated as an artifact for each test type (even for UI and KO). Is it possible to change that on our side?

Log file is being generated because of the hooks implemented in : splunk_add_on_ucc_modinput_test.
In the AWS TA, the dependency splunk_add_on_ucc_modinput_test is not marked as optional in the pyproject.toml resulting in getting installed for each test type. However it was marked as optional in the Salesforce TA, and not getting installed for other test type there.

@dvarasani-crest dvarasani-crest merged commit 25a1ffa into develop Dec 4, 2024
6 checks passed
@dvarasani-crest dvarasani-crest deleted the feat/run-both-ucc-and-legacy-modinput-tests branch December 4, 2024 07:48
@github-actions github-actions bot locked and limited conversation to collaborators Dec 4, 2024
@srv-rr-github-token
Copy link

🎉 This PR is included in version 4.18.0 🎉

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.

5 participants