Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Update Task Heartbeat to include Job_id #594

Merged
merged 14 commits into from
Feb 26, 2021
Merged

Update Task Heartbeat to include Job_id #594

merged 14 commits into from
Feb 26, 2021

Conversation

nharper285
Copy link
Contributor

@nharper285 nharper285 commented Feb 24, 2021 •
edited by bmc-msft
Loading

I made some quick tweaks to include job_id in the task_heartbeat logging.

nharper285 and others added 9 commits November 23, 2020 11:26
@nharper285
Updating yml file to run config endpoint command with tenant/authorit…
293dd06 
…y id.
@nharper285
Adding -DD parameter to unzip command. Forces file replacement irresp…
cbf9dd3 
…ective of timestamp.
@nharper285
Revert "Adding -DD parameter to unzip command. Forces file replacemen…
deb6c5d 
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
@bmc-msft
Merge branch 'main' into main
5a7d9bb
@nharper285
Adding -DD parameter to unzip command. Forces file replacement irresp…
daddaa1 
…ective of timestamp.
@nharper285
Revert "Adding -DD parameter to unzip command. Forces file replacemen…
4fa2486 
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
@nharper285
Merge branch 'main' of https://github.com/nharper285/onefuzz into main
d84ac5d
@nharper285
Merge remote-tracking branch 'upstream/main' into main
4273a01
@nharper285
Adding job_id to task heartbeat telemetry.
855d42f
bmc-msft
bmc-msft suggested changes Feb 24, 2021
View reviewed changes
src/pytypes/onefuzztypes/models.py Outdated Show resolved Hide resolved
src/agent/onefuzz-agent/src/tasks/heartbeat.rs Outdated Show resolved Hide resolved
nharper285 and others added 4 commits February 24, 2021 16:03
@nharper285 @bmc-msft
Update src/agent/onefuzz-agent/src/tasks/heartbeat.rs
7595844 
Co-authored-by: bmc-msft <41130664+bmc-msft@users.noreply.github.com>
@nharper285 @bmc-msft
Update src/pytypes/onefuzztypes/models.py
31c668f 
Co-authored-by: bmc-msft <41130664+bmc-msft@users.noreply.github.com>
@nharper285
Correcting fields.
6770803
@nharper285
Merge branch 'user/noharper/task-heartbeat-update' of https://github.…
500e0b1 
…com/nharper285/onefuzz into user/noharper/task-heartbeat-update
@bmc-msft
Copy link
Contributor

Running integration tests

@ghost
Copy link

ghost commented Feb 26, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@bmc-msft bmc-msft merged commit 06f45f3 into microsoft:main Feb 26, 2021
@bmc-msft bmc-msft linked an issue Feb 28, 2021 that may be closed by this pull request
Update Task Heartbeat Logging to include Job_Id #546
Closed
@ghost ghost locked as resolved and limited conversation to collaborators Apr 16, 2021
@nharper285 nharper285 deleted the user/noharper/task-heartbeat-update branch October 20, 2021 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bmc-msft bmc-msft bmc-msft approved these changes

@chkeita chkeita chkeita approved these changes

Assignees
No one assigned
Labels
auto-merge passed-integration-tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Task Heartbeat Logging to include Job_Id
3 participants
@nharper285 @bmc-msft @chkeita