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

Address user feedback and prepare new major version release #207

Merged
merged 9 commits into from
Nov 28, 2024

Conversation

zlogic
Copy link
Member

@zlogic zlogic commented Nov 22, 2024

  • DES-6665 Addressed an issue with Java version detection.
    • Use more fallbacks, and check the mx dump-mpr exit code before sending its output to jq
  • DES-6613 Create destination directory if it doesn't exist.
  • DES-6613 Bumped up CF Buildpack to v5.0.23.
  • Added documentation explaining how to use build.py script in existing pipelines.
  • Removed unused (unreachable) code from build.py.

zlogic and others added 5 commits November 21, 2024 15:29
* Check existence of java-version file instead of just the directory.
* Use show-java-version as a cleaner way to detect Java version.
* When using dump-mpr, read the entire output & wait for exit code - so that errors are not streamed to jq.
* Log how exactly a Mendix version was detected.
It could potentially be restored later, if build.py would also run the Docker build.
* Clarified requirements and limitations
* Added an upgrade guide explaining how to use build.py in existing pipelines
* Updated versions
@zlogic zlogic changed the title WIP: Address user feedback Address user feedback and prepare new major version release Nov 25, 2024
Any pipeline that isn't updated to use build.py will fail to run, as CF Buildpack only works with specific Ubuntu versions.
@zlogic zlogic marked this pull request as ready for review November 25, 2024 11:11
Clarified that any Python >=3.8 <4 is supported.
* Clarify that the destination directory will be created/cleaned up.
* Clarified that build.py should work with Mendix 8, 9 and 10 MPRs/MPKs.
Copy link
Collaborator

@Nidhi251289 Nidhi251289 left a comment

Choose a reason for hiding this comment

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

All comments are incorporated. Hence, approving this PR.

Docker Buildpack v6 no longer uses CF Buildpack to compile MPR (or MPK) files - to continue supporting newer versions of Mendix, Java and the base OS.
Instead, a custom `build.py` script will:

1. Prepare a clean [Docker context](https://docs.docker.com/build/concepts/context/) in the path specified by `--destination`. All files required to build the app image will be copied to this destination.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we also mention that the if the destaination directory is not created, it will also create the directory

@zlogic zlogic merged commit a3e0257 into latest Nov 28, 2024
1 check passed
@zlogic zlogic deleted the address-user-feedback branch November 28, 2024 14:31
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