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

Log redirection, service side #1727

Merged
merged 5 commits into from
Mar 29, 2022
Merged

Log redirection, service side #1727

merged 5 commits into from
Mar 29, 2022

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Mar 28, 2022

Summary of the Pull Request

This PR sets up the service side of the log redirection work #1711

  • A log container is created or reused when e create a job
  • When scheduling the task we send the log location to the agent
    The expected log structure looks lieke
    {fuzzContainer}/logs/{job_id}/{task_id}/{machine_id}/1.log

the agent side of the work will be done in #1721

chkeita added 2 commits March 28, 2022 12:27
- a log is created or reused when e create a job
- when scheduling the task we send the log location to the agent
The expected log structure looks liek
{fuzzContainer}/logs/{job_id}/{task_id}/{machine_id}/1.log
@chkeita chkeita requested a review from tevoinea March 28, 2022 19:35
@stishkin
Copy link
Contributor

One more - what's the container deletion policy ?

Can any user delete "logs" container ? If so won't the delete other user's logs ?

@chkeita
Copy link
Contributor Author

chkeita commented Mar 28, 2022

One more - what's the container deletion policy ?

Can any user delete "logs" container ? If so won't the delete other user's logs ?
@stishkin
That is true for all container currently today. any body can delete all containers.

docs/webhook_events Outdated Show resolved Hide resolved
@chkeita chkeita enabled auto-merge (squash) March 29, 2022 18:17
@chkeita chkeita merged commit 7add51f into microsoft:main Mar 29, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants