Skip to content

ACLP : Webhook alert channels - #732

Draft
mawasthy-lgtm wants to merge 10 commits into
linode:devfrom
mawasthy-lgtm:feat/add-webhook-channel
Draft

ACLP : Webhook alert channels#732
mawasthy-lgtm wants to merge 10 commits into
linode:devfrom
mawasthy-lgtm:feat/add-webhook-channel

Conversation

@mawasthy-lgtm

Copy link
Copy Markdown
Contributor

Updating python sdk to include webhook channel management api

What does this PR do and why is this change necessary?

Adds full SDK support for Webhook channel create, verify, delete requests used by tests and examples.

This change is necessary because:

The SDK did not have webhook channel management support added.

✔️ How to Test
Clone the repository
Prepare environment (zsh / macOS)

✔️create & activate venv (recommended)
python3 -m venv .venv
source .venv/bin/activate

install deps
python -m pip install --upgrade pip

Install runtime dependencies:
pip3 install requests polling deprecated

Install dev/test extras
pip3 install -e '.[dev,test]'

test deps
pip3 install pytest mock httpretty pytest-rerunfailures

✔️How do I run the relevant unit tests?
cd <PROJECT_DIRECTORY>/linode_api4-python
Unit tests: python -m pytest test/unit -q

Unit test for all monitor: python3 -m pytest test/unit/objects/monitor_test.py -q -s
python3 -m pytest test/unit/groups/monitor_api_test.py -q -s -v

Single unit: python3 -m pytest test/unit/objects/monitor_test.py::MonitorTest -q -s
python3 -m pytest test/unit/groups/monitor_api_test.py::MonitorAlertDefinitionsTest::test_create_verify_delete_webhook_channel -q -s -v

What are the steps to reproduce the issue or verify the changes?
End to end integration testing should suffice

✔️How do I run the relevant integration tests?
Integration tests (E2E):
Note:
#if you have PAT token with write access only to Monitor and read for rest of the services for integration
export SKIP_E2E_FIREWALL=1 # optional: skip firewall autouse fixture
export LINODE_TOKEN="YOUR_REAL_TOKEN" # required for integration
export SKIP_E2E_FIREWALL=1 # optional: skip firewall autouse fixture

python3 -m pytest test/integration/models/monitor/test_monitor.py::test_webhook_channel_crud -q -s

@mawasthy-lgtm
mawasthy-lgtm force-pushed the feat/add-webhook-channel branch from 20fdf94 to 5eea1dc Compare August 14, 2026 08:19
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.

1 participant