Skip to content

Aclp logs support#799

Open
pskupien-akamai wants to merge 31 commits into
linode:devfrom
pskupien-akamai:aclp-logs-destination
Open

Aclp logs support#799
pskupien-akamai wants to merge 31 commits into
linode:devfrom
pskupien-akamai:aclp-logs-destination

Conversation

@pskupien-akamai
Copy link
Copy Markdown

📝 Description

Adds support for monitor logs functionality
What does this PR do and why is this change necessary?

  1. Adds full SDK support for monitor Logs API. This includes both the destinations and streams.
  2. This is a new feature not yet supported in the official version of SDK, but is built upon ready-to-merge pull request in linode_api4-python.
  3. Adds integration tests and documentation for newly implemented features.
    1. Full e2e test for streams require a stream to complete provisioning - based on experience during development this can take up to an hour. Similarly to what's already present for database related tests, an environment variable (RUN_ACLP_LOGS_STREAM_TESTS) needs to be set as true or yes to enable testing the Stream portion of the API.
    2. Logs Destination is tested only for akamai storage bucket cases - custom https storage endpoints are not tested but supported.

✔️ How to Test

This feature is based on ready-to-merge pull request in linode_api4-python. To test this feature before the merge it's required to install this package via running terminal command which enables LogsDestination components:
pip install --upgrade --force-reinstall "git+https://github.com/linode/linode_api4-python.git@refs/pull/686/head#egg=linode_api4"

How do I run the relevant unit/integration tests?

  1. Follow usual installation/setup steps
  2. run make test-int TEST_SUITE="monitor_logs_destination" to test logs destinations
  3. before running stream tests you need to run export RUN_ACLP_LOGS_STREAM_TESTS=true
  4. run make test-int TEST_SUITE="monitor_logs_stream" to test streams

Notes:

  • Stream integration tests by default are skipped if no export RUN_ACLP_LOGS_STREAM_TESTS=true is provided

pskupien-akamai and others added 30 commits May 8, 2026 11:17
…tion-api-signed' into DPS-42284-ACLP-Logs-Streams-API-internal-review
…gned' into DPS-42284-ACLP-Logs-Streams-API-internal-review
…I-internal-review

ACLP Logs - Implement streams api + tests
…-aclp-logs-destination-api-revert-requirements

revert requirements.txt
…-aclp-logs-destination-api-signed

Dps 42283 aclp sdk ansible aclp logs destination api signed
@pskupien-akamai pskupien-akamai requested review from a team as code owners May 15, 2026 08:29
@pskupien-akamai pskupien-akamai requested review from jriddle-linode and psnoch-akamai and removed request for a team May 15, 2026 08:29
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