Skip to content

First version of stackstate version of base check - #1

Merged
craffit merged 4 commits into
masterfrom
setup-checks-base
Jul 10, 2019
Merged

craffit merged 4 commits into
masterfrom
setup-checks-base

Conversation

@craffit

@craffit craffit commented Jul 4, 2019 •

Copy link
Copy Markdown
Collaborator

This is a first increment of the work.

  • Includes branded version of base check
  • Included the branded stsdev command line for managing/testing check installations
  • Includes CI

@craffit
craffit requested a review from TizianoPerrucci July 4, 2019 11:30
@craffit
craffit merged commit 80bf4d4 into master Jul 10, 2019
@TizianoPerrucci
TizianoPerrucci deleted the setup-checks-base branch April 21, 2020 08:01
LouisParkin added a commit that referenced this pull request May 18, 2026
Follow-up to 61e9d4f — the pipeline (2534428169) surfaced two blockers
the earlier sweep missed:

1. stackstate_checks_dev/setup.py:36 still hard-pinned pydantic==2.9.1,
   causing dep resolution conflicts with stackstate_checks_base[deps]==2.12.5
   in every check that has a tox env (11 of 12 failing test_* jobs).
   setup.py files weren't in the earlier sweep grep — only requirements*.in/txt
   were. Now bumped to 2.12.5.

2. Pydantic 2.9 -> 2.12 changed the docs URL embedded in ValidationError
   messages from errors.pydantic.dev/2.9/v/... to .../2.12/v/...
   Tests in 7 files asserted against the literal expected message string,
   so all 16 assertions in test_agent_check.py/test_agent_check_v2.py broke
   (with same brittleness in 5 other test files that never ran due to #1).
   Updated all 7 to /2.12/. NOTE: these are still version-pinned and will
   break again on the next pydantic bump — version-agnostic assertion
   helper would be the durable fix, deferred to a separate refactor.
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