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

Install jit plugins before NUTs #1145

Merged
merged 11 commits into from
Sep 18, 2023
Merged

Install jit plugins before NUTs #1145

merged 11 commits into from
Sep 18, 2023

Conversation

iowillhoit
Copy link
Contributor

What does this PR do?

Breaks the JIT plugins into their own external nuts job.
Installs JIT plugins before running nuts
Sets SF_DATA_DIR so that JIT plugins dont get installed with every TestSession.create

What issues does this PR fix or reference?

@W-14122700@

@iowillhoit
Copy link
Contributor Author

iowillhoit commented Sep 15, 2023

mdonnalley
mdonnalley previously approved these changes Sep 15, 2023
@iowillhoit
Copy link
Contributor Author

QA proof:

  • Setting the SF_DATA_DIR here
  • The JIT install step has the env var here
  • Temp logging step shows install in tmp dir here
  • The time is faster with this method

@iowillhoit iowillhoit merged commit 1a66a6e into main Sep 18, 2023
12 checks passed
@iowillhoit iowillhoit deleted the ew/jit-nuts branch September 18, 2023 18:46
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.

2 participants