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

MAGECLOUD-1618: Temporary Workaround to Clean-up "disable_locking" Parameter #161

Merged
merged 3 commits into from
Jan 25, 2018
Merged

MAGECLOUD-1618: Temporary Workaround to Clean-up "disable_locking" Parameter #161

merged 3 commits into from
Jan 25, 2018

Conversation

billygilbert
Copy link
Contributor

@billygilbert billygilbert commented Jan 24, 2018

Description

Removing disable_locking parameter if present in env.php

Fixed Issues (if relevant)

  1. Removing disable_locking parameter if present in env.php

Zephyr Tests

  1. None

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • Pull request was approved by architect
  • Pull request was approved by QA member
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)


if (isset($config['session']['redis']['disable_locking'])) {
$this->logger->info('Removing disable_locking env.php.');
unset($config['session']['redis'] ['disable_locking']);
Copy link
Member

Choose a reason for hiding this comment

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

What's that itty bitty space doing there?

@billygilbert billygilbert added Progress: testing in progress PR/issue status and removed Progress: review PR/Issue status labels Jan 24, 2018
@mveeramneni
Copy link

Tested with version 2.2.0 and 2.2.2, executed without any issues. QA passed.

@shiftedreality shiftedreality changed the title MAGECLOUD-1618 - Temporary Workaround to Clean-up "disable_locking" Parameter MAGECLOUD-1618: Temporary Workaround to Clean-up "disable_locking" Parameter Jan 25, 2018
@shiftedreality
Copy link
Member

Fix is minor. Architect approve not needed.

@shiftedreality shiftedreality merged commit a9a9b91 into magento:2002.0 Jan 25, 2018
@mveeramneni mveeramneni added Progress: accept PR/issue status and removed Progress: testing in progress PR/issue status labels Jan 25, 2018
@YPyltiai YPyltiai added Release: 2002.0.8 ECE-Tools Release and removed Release Line: 2002.0 labels May 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Progress: accept PR/issue status Release: 2002.0.8 ECE-Tools Release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants