Skip to content

Do not run the uploader with the MDM role#26491

Merged
codingllama merged 4 commits intomasterfrom
codingllama/dt-uploader
May 18, 2023
Merged

Do not run the uploader with the MDM role#26491
codingllama merged 4 commits intomasterfrom
codingllama/dt-uploader

Conversation

@codingllama
Copy link
Copy Markdown
Contributor

Do not run the uploader with the MDM role, MDM service instances don't have the necessary permissions to write events.

Follow up from #26395.

https://github.com/gravitational/teleport.e/issues/826

@codingllama codingllama requested a review from zmb3 May 17, 2023 22:29
@github-actions github-actions Bot requested review from atburke and smallinsky May 17, 2023 22:29
Comment thread lib/service/service.go Outdated
Comment thread lib/service/service.go Outdated
Comment thread lib/service/service.go Outdated
@codingllama codingllama force-pushed the codingllama/dt-uploader branch from f6fb029 to 18ad2f3 Compare May 18, 2023 13:26
@codingllama
Copy link
Copy Markdown
Contributor Author

PTAL?

Comment thread lib/service/service.go Outdated
Comment thread lib/service/service.go Outdated
@codingllama codingllama force-pushed the codingllama/dt-uploader branch from 18ad2f3 to 9ae7d77 Compare May 18, 2023 16:54
@codingllama
Copy link
Copy Markdown
Contributor Author

PTAL @zmb3 ?

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from atburke May 18, 2023 17:21
@codingllama codingllama enabled auto-merge May 18, 2023 17:36
@codingllama codingllama added this pull request to the merge queue May 18, 2023
Merged via the queue into master with commit e927379 May 18, 2023
@codingllama codingllama deleted the codingllama/dt-uploader branch May 18, 2023 17:49
@public-teleport-github-review-bot
Copy link
Copy Markdown

@codingllama See the table below for backport results.

Branch Result
branch/v13 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants