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

[draft] add empytdir mount to ee container in task pod #1699

Draft
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

kdelee
Copy link
Member

@kdelee kdelee commented Jan 31, 2024

SUMMARY

Mount a memory backed emptyDir to the ee container /tmp so receptor writes the job events its streaming back from job pods to memory instead of the container writable layer, which is copy-on-write.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
ADDITIONAL INFORMATION

so far just works in local kind cluster
would like to test out and compare apples:apples and measure RAM usage, job event processing performance

I've built the operator container here: quay.io/kdelee/awx-operator:receptor-emptydir

Doesn't require any special awx image

@kdelee
Copy link
Member Author

kdelee commented Jan 31, 2024

Wonder if there is a desire to guard this in if statement with some var -- that said, I see the proliferation of vars as also a problem 😄

Copy link
Member

@TheRealHaoLiu TheRealHaoLiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be done as a ee_extra_volume_mounts and task_extra_volumes and not used as a default...

memory are 💵

@kdelee kdelee marked this pull request as draft March 20, 2024 13:41
@kdelee kdelee changed the title add empytdir mount to ee container in task pod [draft] add empytdir mount to ee container in task pod Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants