Skip to content

DUO Admin - update docker #42573

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
Open
adi88d wants to merge 4 commits into
base: master
Choose a base branch
from
Open

DUO Admin - update docker #42573

adi88d wants to merge 4 commits into from
+33 −3

Conversation

@adi88d
Copy link
Contributor

@adi88d adi88d commented Jan 7, 2026 •
edited
Loading

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-15312

Description

Updated the Docker image to: demisto/vendors-sdk:1.0.0.6383083 for Duo Event Collector and DUO Admin

Must have

  • Tests
  • Documentation

@adi88d adi88d marked this pull request as ready for review January 7, 2026 10:29
@github-actions
Copy link

github-actions bot commented Jan 7, 2026 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/DuoAdminApi/Integrations/DuoAdminApi
   DuoAdminApi.py1102379%69, 73–75, 125, 135, 156, 166–168, 170–171, 173–174, 177, 179–180, 183, 228–230, 232, 239
Packs/DuoAdminApi/Integrations/DuoEventCollector
   DuoEventCollector.py2343186%67–68, 71–72, 76–80, 82, 84, 137, 183, 309–311, 317–318, 336, 441, 450, 462–463, 467, 471–474, 481, 483–484
TOTAL3445484% 

Tests Skipped Failures Errors Time
31 0 💤 0 ❌ 0 🔥 2.558s ⏱️

@adi88d adi88d added docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. labels Jan 7, 2026
@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 7, 2026
@content-bot
Copy link
Collaborator

content-bot commented Jan 7, 2026

🤖 Content AI Reviewer: Analysis started. Please wait for results...

@content-bot
Copy link
Collaborator

content-bot commented Jan 7, 2026

Review not published as no human-reviewer user was assigned.

@adi88d adi88d requested a review from DeanArbel January 7, 2026 11:33
@adi88d adi88d added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 7, 2026
@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Jan 7, 2026
@content-bot
Copy link
Collaborator

content-bot commented Jan 7, 2026

🤖 Content AI Reviewer: Analysis started. Please wait for results...

@content-bot
Copy link
Collaborator

content-bot commented Jan 7, 2026

🤖 Content-bot Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

content-bot
content-bot reviewed Jan 7, 2026
View reviewed changes
Copy link
Collaborator

@content-bot content-bot left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! It appears that the integration YAML files required for the Docker image update mentioned in the description are missing from this PR. Please ensure all relevant files are included to support the version bump.

Thanks

@DeanArbel please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@demisto demisto deleted a comment from content-bot Jan 7, 2026
@content-bot
Copy link
Collaborator

content-bot commented Jan 7, 2026

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

@fancybear-dev
Copy link

fancybear-dev commented Jan 9, 2026

Does this address; https://help.duo.com/s/article/9451?language=en_US?

We have reason to believe the Duo Admin XSOAR integration currently uses an outdated version affected by the referenced deprecation document.

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

Reviewers

@content-bot content-bot content-bot left review comments

@DeanArbel DeanArbel Awaiting requested review from DeanArbel

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

docs-approved ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines skip-ai-review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@adi88d @content-bot @fancybear-dev