[telemetry_chargeback] Update ingest data yaml based on feedback#392
Conversation
- Add retrieve_loki_data.yml for querying Loki API with retry logic - Update ingest_loki_data.yml for uploading data to Loki - Update load_loki_data.yml for loading Loki data - Update README.md with task documentation
ayefimov-1
force-pushed
the
alexy_ck_job_6b
branch
from
2a9f5ad to
a9c68db
Compare
|
Build failed (check pipeline). Post ✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 2h 37m 31s |
|
recheck |
ayefimov-1
force-pushed
the
alexy_ck_job_6b
branch
2 times, most recently
from
75ea31c to
73fc1b4
Compare
ayefimov-1
changed the title
|
Build failed (check pipeline). Post ✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 46m 01s |
|
recheck |
ayefimov-1
force-pushed
the
alexy_ck_job_6b
branch
from
73fc1b4 to
a0c0847
Compare
| ansible.builtin.fail: | ||
| msg: | ||
| - "Scenario: {{ scenario_name }}" | ||
| - "Status: {{ loki_response.status | default('Unknown') }}" |
There was a problem hiding this comment.
nit: don't think I don't think any default value below are needed, uri module should always return some value.
|
|
||
| # Success | ||
| - name: "Confirm ingestion success" | ||
| - name: "Ingest Status Message" |
There was a problem hiding this comment.
nit: if TEST Push data to Loki {{ scenario_name }} is success, that debug task is not needed. Otherwise it will run tasks from rescue.
| - name: "POST data to Loki" | ||
| ansible.builtin.uri: | ||
| ## loki_push_url will be used in future role developement | ||
| # loki_push_url is configured in setup_loki_env.yml |
There was a problem hiding this comment.
nit: if that is needed, good to have a task with fail, like:
- name: Check if loki_push_url is set
when: loki_push_url is not defined
ansible.builtin.fail:
msg: |
Please specify 'loki_push_url' in setup_loki_env.yml
4 participants
Authored-by: @ayefimov-1
AI Assisted by: Claude