Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CDP-8544: Added an endpoint to get active host ids #1743

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

vitalii-honchar
Copy link
Contributor

@vitalii-honchar vitalii-honchar commented Nov 6, 2024

Context

Rodimus should have a possibility to fallback to check running hosts in the Teletraan service. This PR adds a new endpoint which returns active (running) hosts from the Teletraan database.

Tests

Test Script

#!/bin/bash

TOKEN=''

curl -i -X POST --location 'https://<teletraan_url>/v1/hosts/active' \
    --header 'Content-Type: application/json' \
    --header "Authorization: token $TOKEN" \
    --data '["i-00ed0c81d932f4038", "i-045152ef8256fcb18"]'

Results

Active Hosts

["i-00ed0c81d932f4038","i-045152ef8256fcb18"]

Not Active Hosts

[]

@github-actions github-actions bot added the deploy-service Includes changes to deploy-service label Nov 6, 2024
@vitalii-honchar vitalii-honchar marked this pull request as ready for review November 7, 2024 19:54
@vitalii-honchar vitalii-honchar requested a review from a team as a code owner November 7, 2024 19:54
Copy link
Contributor

@tylerwowen tylerwowen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Please run mvn spotless:apply

@vitalii-honchar vitalii-honchar merged commit 520d3db into master Nov 7, 2024
6 checks passed
@vitalii-honchar vitalii-honchar deleted the vhonchar/get-hosts-by-ids branch November 7, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-service Includes changes to deploy-service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants