Skip to content

Fix for local integration tests by ignoring warning#3172

Closed
Adam-D-Lewis wants to merge 4 commits intomainfrom
claude/session-011CUa5QJJymQiEmrhtmCzcy
Closed

Fix for local integration tests by ignoring warning#3172
Adam-D-Lewis wants to merge 4 commits intomainfrom
claude/session-011CUa5QJJymQiEmrhtmCzcy

Conversation

@Adam-D-Lewis
Copy link
Copy Markdown
Member

@Adam-D-Lewis Adam-D-Lewis commented Oct 28, 2025

The recent release of google-api-core 2.28.0 introduced FutureWarnings about Python 3.10 EOL support that are causing the local integration test workflow to fail during the nebari init step.

This change adds a warning filter in the main CLI module to suppress these warnings, allowing the package to stay up-to-date while avoiding CI failures. This approach is consistent with the warning filter already added to tests/conftest.py in commit d175fa2.

Fixes the failure in: https://github.com/nebari-dev/nebari/actions/runs/18882140966/job/53887875554

🤖 Generated with Claude Code

Reference Issues or PRs

What does this implement/fix?

Put a x in the boxes that apply

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds a feature)
  • Breaking change (fix or feature that would cause existing features not to work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Other (please describe):

Documentation

  • For new features or enhancements, a corresponding PR has been opened in the documentation repository (if applicable)
    • Link to docs PR:

Testing

  • Did you test the pull request locally?
  • Did you add new tests?

How to test this PR?

Any other comments?

The recent release of google-api-core 2.28.0 introduced FutureWarnings
about Python 3.10 EOL support that are causing the local integration
test workflow to fail during the nebari init step.

This change adds a warning filter in the main CLI module to suppress
these warnings, allowing the package to stay up-to-date while avoiding
CI failures. This approach is consistent with the warning filter already
added to tests/conftest.py in commit d175fa2.

Fixes the failure in: https://github.com/nebari-dev/nebari/actions/runs/18882140966/job/53887875554

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@Adam-D-Lewis Adam-D-Lewis requested a review from a team as a code owner October 28, 2025 19:06
@Adam-D-Lewis Adam-D-Lewis requested review from dcmcand and marcelovilla and removed request for a team October 28, 2025 19:06
@Adam-D-Lewis Adam-D-Lewis changed the title Fix local integration tests by ignoring warning Partial fix for local integration tests by ignoring warning Oct 28, 2025
@Adam-D-Lewis
Copy link
Copy Markdown
Member Author

They are still failing til we set the github-actions.nebari.dev DNS record.

@Adam-D-Lewis Adam-D-Lewis changed the title Partial fix for local integration tests by ignoring warning Fix for local integration tests by ignoring warning Oct 29, 2025
@Adam-D-Lewis
Copy link
Copy Markdown
Member Author

DNS record is now updated

@github-project-automation github-project-automation bot moved this from New 🚦 to Done 💪🏾 in 🪴 Nebari Project Management Oct 29, 2025
@Adam-D-Lewis
Copy link
Copy Markdown
Member Author

Apparently this is not needed

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

Labels

None yet

Projects

Status: Done 💪🏾

Development

Successfully merging this pull request may close these issues.

2 participants