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

Use the correct Docker Image for RocksJava on Linux #12169

Closed
wants to merge 8 commits into from

Conversation

adamretter
Copy link
Collaborator

No description provided.

@pdillinger
Copy link
Contributor

Sorry but now the changes will have to go to the GitHub Actions config. (After merge/rebase with #12163 ) Thanks for hacking on the docker image!

@adamretter adamretter force-pushed the correct-docker-image branch 10 times, most recently from 42419d8 to 0ca0338 Compare January 11, 2024 10:18
@facebook-github-bot
Copy link
Contributor

@ajkr has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@adamretter
Copy link
Collaborator Author

@pdillinger I have updated this for GitHub Actions now. There are two Folly build failures, I am not certain, but I think they are unrelated to my changes, can you confirm?

- name: Checkout
env:
GH_TOKEN: ${{ github.token }}
run: |
Copy link
Contributor

Choose a reason for hiding this comment

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

This is copy-pasting a complex piece of functionality. Presumably it can go into a sharable "uses" step.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@pdillinger I did try that, but unfortunately it is not possible as you end up in a "chicken or egg" situation. i.e. if you put the code in a separate GitHub action file of this repo, then it isn't yet available as the code hasn't been checked out from Git until after this step would run.

@facebook-github-bot
Copy link
Contributor

@ajkr merged this pull request in 5a26f39.

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

Successfully merging this pull request may close these issues.

4 participants