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-8328: add normandie status and knox status to ping requests and hosts_and_agents sql table #1744

Closed
wants to merge 3 commits into from

Conversation

Tweedeldee
Copy link
Contributor

@Tweedeldee Tweedeldee commented Nov 6, 2024

Description

Add normandie status and knox status to ping requests and hosts_and_agents sql table
CDP-8328

Development status

  • Normandie status is defined by the presence of a valid normandie's certificate.
  • Knox status is currently NOT retrieved.
  • Both fields are emitted in ping requests and stored in hosts_and_agents SQL table

Test in devapp

deploy-agent alone, with valid Normandie certificate

Commands

sudo PINLOG_MIN_LOG_LEVEL=DEBUG PINLOG_LOG_TO_STDERR=1 PINLOG_STDERR_LEVEL=DEBUG PINLOG_LOG_DIR=/var/log/deployd IS_PINTEREST=true METRIC_PORT_HEALTH=18126 METRIC_CACHE_PATH=/mnt/deployd/metrics.cache PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin ~/code/teletraan/deploy-agent/bazel-bin/deployd/deploy-agent -e prod -f /etc/deployagent.conf --use-facter

Output, slightly reformated

D23:08:40 [deployd.client.client] PingRequest(hostId=i-0fa4dfea2b1b5f531, hostName=devrestricted-jpierre, hostIp=10.9.196.125, agentVersion=1.2.65-pinterest-1.0.2, autoscalingGroup=None, availabilityZone=us-east-1a, ec2Tags={"Name": "devrestricted-jpierre", "devapp_version": "None", "service_mapping": "dev", "usage_tag": "Dev Apps", "username": "jpierre", "availability_zone": "us-east-1a"}, stageType=None, groups=['singer-prod-sidecar', 'envoy-devapp', 'tcollector-prod-sidecar', 'mcrouter-devapp', 'mcrouter-prod-sidecar', 'metrics-agent-prod-sidecar', 'zk_update_monitor-prod-sidecar'], accountId=998131032990, 

normandieStatus=OK, knoxStatus=NotImplementedYet, 

reports=PingReport([...]) -> PingResponse(opCode=NOOP, deployGoal=None)

deploy-agent along with deploy-service

@github-actions github-actions bot added deploy-service Includes changes to deploy-service deploy-agent Includes changes to deploy-agent labels Nov 6, 2024
Base automatically changed from jpierre/CDP-8328-2 to master November 6, 2024 19:11
@Tweedeldee Tweedeldee changed the title CDP-8328: add normandie status and knox status to ping requests and h… CDP-8328: add normandie status and knox status to ping requests and hosts_and_agents sql table Nov 6, 2024
@Tweedeldee Tweedeldee closed this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-agent Includes changes to deploy-agent deploy-service Includes changes to deploy-service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant