The Wayback Machine - https://web.archive.org/web/20220527121635/https://github.com/eclipse/che/issues/16869
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

php stack: the command Configure Apache Web Server DocumentRoot should be done in the entrypoint (command) or CMD (args) #16869

Open
sunix opened this issue May 7, 2020 · 2 comments
Labels
area/getting-started area/languages good first issue kind/enhancement lifecycle/frozen severity/P2

Comments

@sunix
Copy link
Contributor

@sunix sunix commented May 7, 2020

Is your enhancement related to a problem? Please describe.

Starting the php stack, we ask the user to do a step that could be done automatically:

https://github.com/eclipse/che-devfile-registry/blob/master/devfiles/php-web-simple/devfile.yaml#L53

- name: Configure Apache Web Server DocumentRoot
  actions:
    - type: exec
      component: php
      command: |
        if grep -q '/projects$' /etc/apache2/sites-available/000-default.conf; then
          sed -i 's|DocumentRoot /projects|DocumentRoot /projects/php-web-simple|' /etc/apache2/sites-available/000-default.conf
        else
          echo "DocumentRoot already configured!"
        fi

We could simply call that script in the entrypoint ( following by sleep as usual ).
it would avoid user having to do that him self

image

@sunix sunix added the kind/enhancement label May 7, 2020
@che-bot che-bot added the status/need-triage label May 7, 2020
@ibuziuk ibuziuk added the area/getting-started label May 8, 2020
@ibuziuk
Copy link
Contributor

@ibuziuk ibuziuk commented May 8, 2020

@sunix are you going to work on this task? Could you please set the labels based on priorities? (PHP stack is quite popular)

@rkratky rkratky added area/languages severity/P2 and removed status/need-triage labels May 8, 2020
@sunix sunix added the good first issue label Jun 26, 2020
@che-bot
Copy link
Contributor

@che-bot che-bot commented Jan 4, 2021

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale label Jan 4, 2021
@ericwill ericwill added lifecycle/frozen and removed lifecycle/stale labels Jan 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/getting-started area/languages good first issue kind/enhancement lifecycle/frozen severity/P2
5 participants