Skip to content

Fix/xsup 74825/cisco thousand eyes fetch events mechanisem - #45516

Open
MosheEichler wants to merge 1 commit into
masterfrom
fix/XSUP-74825/CiscoThousandEyes-fetch-events-mechanisem
Open

Fix/xsup 74825/cisco thousand eyes fetch events mechanisem#45516
MosheEichler wants to merge 1 commit into
masterfrom
fix/XSUP-74825/CiscoThousandEyes-fetch-events-mechanisem

Conversation

@MosheEichler

Copy link
Copy Markdown
Contributor

Status

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

Related Issues

https://jira-dc.paloaltonetworks.com/browse/XSUP-74186

Description

Fix: when no events are found, advance the checkpoint to end_date (the point through which we've confirmed there's nothing new) instead of losing it (now - 1 minute).

Must have

  • Tests
  • Documentation

@content-bot

Copy link
Copy Markdown
Contributor

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CiscoThousandEyes/Integrations/CiscoThousandEyes
   CiscoThousandEyes.py1201091%234–237, 265, 267, 354–357
TOTAL1201091% 

Tests Skipped Failures Errors Time
15 0 💤 0 ❌ 0 🔥 3.500s ⏱️

@MosheEichler
MosheEichler force-pushed the fix/XSUP-74825/CiscoThousandEyes-fetch-events-mechanisem branch from decf2c8 to 68e12c2 Compare August 13, 2026 16:39
@content-bot

Copy link
Copy Markdown
Contributor

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

@content-bot

Copy link
Copy Markdown
Contributor

Validate summary
The following errors were thrown as a part of this pr: BA133.
The following errors cannot be ignored: BA133.
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.
The following errors don't run as part of the nightly flow and therefore can be force merged: BA133.

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

@content-bot

Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: e4942a84b485c0b8

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants