Bug #35801

Hardcoded Production context

Added by Julian Kleinhans over 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Must have
Target version:
-
Start date:
2012-04-09
Due date:
% Done:

100%

Estimated time:

Description

Use surf with other context is not possible bcause there are several hardcoded lines with the Production context. See attached screen


Files

#1

Updated by Julian Kleinhans over 9 years ago

Example call:

 $ FLOW3_CONTEXT=Staging ./flow3 surf:deploy tutorials3

#2

Updated by Andreas Wolf about 9 years ago

I'd set the context in the deployment and offer a chance to override this with a local variable.

Additionally we might want to run a command for different contexts; maybe this can be handled by using an array as option "context". The command would then be repeated for each array entry.

#3

Updated by Gerrit Code Review about 9 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/14111

#4

Updated by Gerrit Code Review almost 9 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/16959

#5

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/16959

#6

Updated by Gerrit Code Review over 8 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/16959

#7

Updated by Gerrit Code Review over 8 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/16959

#8

Updated by Dominique Feyer over 8 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100
#9

Updated by Martin Ficzel over 8 years ago

The SymlinkConfiguration Task should be updated too.

Currently the CopyConfigurationTask copies all yaml-files from folder Build/Surf/__deployment__/Configuration to shared/Configuration but the SymlinkConfigurationTask creates only the Production Symlink for the release. I suggest to create the symlink only for the deployed context.

#10

Updated by Christopher Hlubek over 8 years ago

I'm pretty unhappy with the SymlinkConfigurationTask because it breaks the isolation of releases. I noted that also in the class documentation "Note: this might cause problems with concurrent access due to the cached configuration insided this directory.".

This comes from the way Flow builds the configuration cache and is not something Surf can solve. It's better to copy configuration to each release for isolation. Otherwise the new/old release can break if users access the application during deployment! So I would rather remove that task or disable it by default.

Greetings,

Christopher

Martin Ficzel wrote:

The SymlinkConfiguration Task should be updated too.

Currently the CopyConfigurationTask copies all yaml-files from folder Build/Surf/__deployment__/Configuration to shared/Configuration but the SymlinkConfigurationTask creates only the Production Symlink for the release. I suggest to create the symlink only for the deployed context.

#11

Updated by Martin Ficzel over 8 years ago

In that case the task should be disabled by default and another setup should be documented and encouraged. Maybe using subcontexts for the different deployment-targets could be an easy ans logical option. In that case nothing like copyConfiguration would be needed at all.

Greetings, Martin

Also available in: Atom PDF