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

feat: add logging to agent to track usage [DET-3878] #1110

Merged
merged 1 commit into from
Aug 18, 2020

Conversation

aaron276h
Copy link
Contributor

@aaron276h aaron276h commented Aug 18, 2020

Description

This PR adds logging to enable to track agent usage in an offline manner.

Log messages added:

determined-master  | INFO[2020-08-18T13:28:21Z] agent connected ip: 172.31.43.136 slots: 8    id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:28:30Z] starting container id: 839d0f7d-c1c9-4124-8933-9210295f9864 slots: 1 task handler: /notebooks/f55c0380-ae30-4e9c-91d9-7a15a28545b2  id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:28:45Z] starting container id: fd964d08-4206-4b20-aefa-ce7293872e9b slots: 1 task handler: /experiments/8/54f63b44-bf25-4a9f-9757-cd64c632fe48  id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:29:21Z] stopped container id: fd964d08-4206-4b20-aefa-ce7293872e9b id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:29:21Z] starting container id: 6e28d0ac-2cab-44e8-9ef2-df148cc1592d slots: 0 task handler: /experiment-8-checkpoint-gc  id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:29:26Z] stopped container id: 6e28d0ac-2cab-44e8-9ef2-df148cc1592d  id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:30:33Z] stopped container id: 839d0f7d-c1c9-4124-8933-9210295f9864  id=i-061172e664f7c82de system=master type=agent
determined-master  | INFO[2020-08-18T13:30:33Z] agent disconnected                            id=i-061172e664f7c82de system=master type=agent

@stoksc
Copy link
Contributor

stoksc commented Aug 18, 2020

@aaron276h check 1 2 3 check

@aaron276h aaron276h merged commit 7a30cdb into determined-ai:master Aug 18, 2020
azhou-determined pushed a commit that referenced this pull request Dec 7, 2023
- Update the token invalidation code to use the latest API on SaaS
- Return `jwt.ErrTokenExpired` when the token has been invalidated so that it is treated as an auth error and tiggers re-auth
wes-turner pushed a commit that referenced this pull request Feb 2, 2024
- Update the token invalidation code to use the latest API on SaaS
- Return `jwt.ErrTokenExpired` when the token has been invalidated so that it is treated as an auth error and tiggers re-auth
@dannysauer dannysauer added this to the 0.13.0 milestone Feb 6, 2024
rb-determined-ai pushed a commit that referenced this pull request Feb 29, 2024
- Update the token invalidation code to use the latest API on SaaS
- Return `jwt.ErrTokenExpired` when the token has been invalidated so that it is treated as an auth error and tiggers re-auth
amandavialva01 pushed a commit that referenced this pull request Mar 18, 2024
- Update the token invalidation code to use the latest API on SaaS
- Return `jwt.ErrTokenExpired` when the token has been invalidated so that it is treated as an auth error and tiggers re-auth
eecsliu pushed a commit that referenced this pull request Apr 18, 2024
- Update the token invalidation code to use the latest API on SaaS
- Return `jwt.ErrTokenExpired` when the token has been invalidated so that it is treated as an auth error and tiggers re-auth
eecsliu pushed a commit to determined-ai/determined-release-testing that referenced this pull request Apr 22, 2024
…1110)

- Update the token invalidation code to use the latest API on SaaS
- Return `jwt.ErrTokenExpired` when the token has been invalidated so that it is treated as an auth error and tiggers re-auth
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants