Xsup 67244/update crypto docker#44123
Conversation
… Microsoft Defender for Cloud Apps, CertificateExtract, MS365DefenderAddComment, PFXAnalyzer, Microsoft Graph Security
🤖 AI-Powered Code Review AvailableYou can leverage AI-powered code review to assist with this PR! Available Commands:
|
|
🤖 Analysis started. Please wait for results... |
🤖 AI Review DisclaimerThis 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. |
marketplace-ai-reviewer
left a comment
There was a problem hiding this comment.
Thanks for your contribution to the Microsoft integrations! I've reviewed your changes and noticed a few recurring structural and metadata issues across the updated packs.
Please ensure that name or display is the very first key in your YAML files, verify that you are using the latest demisto/crypto Docker image, and update the pack metadata files to include "Microsoft" in the keywords and the missing created key.
Let me know if you have any questions while making these updates!
Additionally, please address the following file-level notes:
Packs/MicrosoftCloudAppSecurity/Integrations/MicrosoftCloudAppSecurity/MicrosoftCloudAppSecurity.yml: CRITICAL: The first key defined at the root level of the.ymlfile must always benameordisplay. Currently, the file starts withcategory. Please reorder the keys so thatnameordisplayis at the very top.Packs/MicrosoftGraphAPI/Integrations/MicrosoftGraphAPI/MicrosoftGraphAPI.yml: CRITICAL: The first key defined at the root level of the.ymlfile must always benameordisplay. Currently, the file starts withcategory. Please reorder the keys so thatnameordisplayis at the very top.Packs/MicrosoftGraphSecurity/Integrations/MicrosoftGraphSecurity/MicrosoftGraphSecurity.yml: CRITICAL: The first key defined at the root level of the.ymlfile must always benameordisplay. Currently, the file starts withcategory. Please reorder the keys so thatnameordisplayis at the very top.Packs/Microsoft365Defender/pack_metadata.json: The mandatory keycreatedis missing from the pack metadata. Please add it.Packs/MicrosoftCloudAppSecurity/pack_metadata.json: The vendor name (Microsoft) must appear in thekeywordslist. Please add it.Packs/MicrosoftGraphAPI/pack_metadata.json: The vendor name (Microsoft) must appear in thekeywordslist. Please add it.Packs/MicrosoftGraphSecurity/pack_metadata.json: The vendor name (Microsoft) must appear in thekeywordslist. Please add it.
@BEAdi, @shmuel44 please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
This comment has been minimized.
This comment has been minimized.
This PR was automatically updated by a GitHub Action
To stop automatic version bumps, add the |
This PR was automatically updated by a GitHub Action
To stop automatic version bumps, add the |
This comment has been minimized.
This comment has been minimized.
This PR was automatically updated by a GitHub Action
To stop automatic version bumps, add the |
|
Validate summary Verdict: PR can be force merged from validate perspective? ✅ |
🔍 AI Triage Report AvailableAn automated triage report has been generated for this pipeline. Status: 📋 Triage Report
|
|
Force merge - 1 test playbook is failing with the same error as here, on a PR with no change (#44183) |
Related Issues
related: https://jira-dc.paloaltonetworks.com/browse/XSUP-67244
Description