Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CNR: Disable pager101 integration test #3226

Merged
merged 1 commit into from
Dec 9, 2024
Merged

CNR: Disable pager101 integration test #3226

merged 1 commit into from
Dec 9, 2024

Conversation

tlimoncelli
Copy link
Contributor

@tlimoncelli tlimoncelli commented Dec 7, 2024

Problem:

The pager101 integration test files for CNR.

Solution:

  • I suspect our account has a limit in place.
  • This test is safe to ignore for most providers as most don't send API results 1 page at a time.

@tlimoncelli
Copy link
Contributor Author

CC @KaiSchwarz-cnic @AsifNawaz-cnic for visibility. If you want to debug this, please open a bugid. I'd be glad to collaborate.

@tlimoncelli tlimoncelli merged commit 006a434 into main Dec 9, 2024
17 checks passed
@tlimoncelli tlimoncelli deleted the tlim_cnr_101 branch December 9, 2024 20:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant