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

Release 1.10.0 #69

Closed
wants to merge 31 commits into from
Closed

Release 1.10.0 #69

wants to merge 31 commits into from

Conversation

achekulaev
Copy link
Member

@achekulaev achekulaev commented Jul 9, 2018

  • Switch to global Node.js installation
  • Ditch nvm and install Node.js globally from NodeSource repos
  • List images after build (to check image sizes)
  • Using and extending the default supervisord configuration
  • Starting supervisord using /etc/supervisor/supervisord.conf config, which includes /etc/supervisor/conf.d/*.conf
  • Removed non-essential settings from the custom supervisor.conf - pretty much all of them are covered in /etc/supervisor/supervisord.conf (in Debian Stretch)
  • Build refactoring

  • Cloud9 IDE integration

  • Cloud9 build and tests

  • Fix missing yarn

  • Optimize chmod usage

Reduces a layer size by 26MB

  • Removed duplicate COPY statement in 5.6

[ci skip]

  • Cloud9: Dockerfile updates
  • Set noninteractive mode during the build (package install) process
  • Cleanup packages
  • structure Dockerfile similar to the base image files
  • Cloud9 permissions issue fix
  • Moved c9sdk into /home/docker/c9sdk, since it needs to be able to write into its folder even after UID for the docker users changes at container startup
  • Install as docker user, so we don't have to fix permissions
  • clear npm cache (-20MB)
  • Fix chmod in startup.sh

Use 1000:1000 by default if HOST_UID/HOST_GID are not set (e.g. in PWD environments)

  • Updated comment in php configs

Point to the correct config file for PHP FPM overrides

[ci skip]

  • Moved the push script into a standalone file - scripts/docker-push.sh
  • Refactored and optimized the script logic
  • Updated docksal install command
  • Using bash
  • Added DOCKSAL_VERSION to add an option to override the version via build variables
  • Added parameter description for tag_and_push

[ci skip]

  • Moved gomplate up to global tool section next to gosu (added GOSU_VERSION variable)
  • Moved platformsh CLI up to other phar based installs
  • Use curl -fsSL for downloads

-f - fail if HTTP return code is not 200
-sS - silent mode, but show error messages if failed
-L - follow HTTP redirects

  • Removed --force-yes from apt-get install

It's deprecated and should not be used

  • Suppress apt-key warnings

"Warning: apt-key output should not be parsed (stdout is not a terminal)"

  • Fixed an orphaned curl -fssL in 5.6

  • Use consolidation/cgr instead of composer global require

cgr handles installing global composer dependencies into individual folders automatically

  • Switch to installing mg2-codegen via Composer

See:

  • DRUPAL_CONSOLE_VERSION => DRUPAL_CONSOLE_LAUNCHER_VERSION

lmakarov and others added 30 commits May 31, 2018 00:29
- Ditch nvm and install Node.js globally from NodeSource repos
- List images after build (to check image sizes)
- Starting supervisord using /etc/supervisor/supervisord.conf config, which includes /etc/supervisor/conf.d/*.conf
- Removed non-essential settings from the custom supervisor.conf - pretty much all of them are covered in /etc/supervisor/supervisord.conf (in Debian Stretch)
Reduces a layer size by 26MB
- Set noninteractive mode during the build (package install) process
- Cleanup packages
- structure Dockerfile similar to the base image files
- Moved c9sdk into /home/docker/c9sdk, since it needs to be able to write into its folder even after UID for the docker users changes at container startup
- Install as docker user, so we don't have to fix permissions
- clear npm cache (-20MB)
Use 1000:1000 by default if HOST_UID/HOST_GID are not set (e.g. in PWD environments)
Point to the correct config file for PHP FPM overrides

[ci skip]
- Moved the push script into a standalone file - scripts/docker-push.sh
- Refactored and optimized the script logic
- Using bash
- Added DOCKSAL_VERSION to add an option to override the version via build variables
Make sure the latest release and stable tags match. Closes #56
* Added Platform.sh cli tool

* Added a generic secret variable conversion function (`SECRET_VAR1` => `VAR1`)

* Added integration to test Platform.sh

* Updated Terminus to 1.8.1

* Updated README.md
* Added test to confirm Terminus is working correctly in the container
- Moved gomplate up to global tool section next to gosu (added GOSU_VERSION variable)
- Moved platformsh CLI up to other phar based installs
-f - fail if HTTP return code is not 200
-sS - silent mode, but show error messages if failed
-L - follow HTTP redirects
It's deprecated and should not be used
"Warning: apt-key output should not be parsed (stdout is not a terminal)"
cgr handles installing global composer dependencies into individual folders automatically
@achekulaev achekulaev closed this Jul 9, 2018
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.

4 participants