#49: Create the "cikit-webserver" role #90

BR0kEN- · 2018-01-23T17:41:11Z

BR0kEN- · 2018-01-23T17:47:41Z

cmf/all/.cikit/vars/web-server.yml

+---
+web_server_vhosts:
+  - name: "{{ (site_url.split('//') | last) if site_url is defined and localhost else 'JenkinsCI' }}"
+    root: "{{ webroot }}/{{ '' if site_url is defined and localhost else '%1/' }}docroot"


TODO: Need to have the builds as subdomains (sort of VirtualDocumentRoot) when only Nginx is used.

… scratch

…for installation

…t an option there

BR0kEN- · 2018-01-29T08:09:07Z

Notes for the record in changelog (manual actions)

Replace the following inside of .cikit/vars/ports.yml:
- apache_port_ssl by web_port_ssl
- apache_port by web_port
Remove the .cikit/vars/apache.yml.
Create the .cikit/vars/web-server.yml.

BR0kEN- · 2018-01-29T09:40:35Z

@lokeoke, @temoor-gilmutdinov the PR is ready to be reviewed but still WIP. I would appreciate your time to looking at this.

The tests I ran:

Create the project.

cikit self-update --version=issues/49
cikit init --project=webserver
cd webserver

Check Nginx operation in a VM built from scratch.

EXTRA_VARS="--web-server=nginx --php-version=7.1 --nodejs-version=6 --ruby-version=2.4.0 --solr-version=no --mssql-install=no" vagrant up

Create a droplet with Nginx and ensure builds are available.

cikit matrix/droplet --limit=HOSTNAME --droplet-add
cikit provision --limit=HOSTNAME.DROPLET

Destroy the previous machine and check Nginx operation in a VM built from scratch.

vagrant destory -f
EXTRA_VARS="--web-server=apache2 --php-version=7.1 --nodejs-version=6 --ruby-version=2.4.0 --solr-version=no --mssql-install=no" vagrant up

Create a droplet with Apache 2 and ensure builds are available.

cikit matrix/droplet --limit=HOSTNAME --droplet-delete=DROPLET
cikit matrix/droplet --limit=HOSTNAME --droplet-add
cikit provision --limit=HOSTNAME.DROPLET

Change web server on demand inside of VM and check whether an application behaves correctly.

CIKIT_TAGS="web-server" EXTRA_VARS="--web-server=nginx" vagrant provision

Ensure the error about the impossibility to change the server on an existing CI droplet is gotten.

cikit provision --limit=HOSTNAME.DROPLET

BR0kEN- · 2018-01-31T14:15:23Z

@lokeoke, @temoor-gilmutdinov, I'm gonna merge this on the weekend since the tests are passing. So, I'd appreciate your opinions.

…chosen web server

BR0kEN- · 2018-01-31T15:42:31Z

Merge has been scheduled for February 3, 2018 (233bf59#diff-ec969b7d93a4e2e93c446ab2df13ae70R3).

…ation

…eing run

…on to README.md

BR0kEN- · 2018-02-02T21:20:55Z

Merge was rescheduled to February 4, 2018 (5c7e605) to give reviewers, @temoor-gilmutdinov and @lokeoke, more time for evaluation.

lokeoke

I was unable to test it though it's a cool feature and I like how upgrade path is provided 👍

timur-hilmutdinov · 2018-02-03T11:17:21Z

docs/_changelog/2018-02-04/README.md

+
+## Required manual actions
+
+Right after updating the package via `cikit self-update` please make sure all the projects you have based on CIKit will receive the following modifications.


I guess it should end with colon(:)

BR0kEN- added the enhancement label Jan 23, 2018

BR0kEN- self-assigned this Jan 23, 2018

BR0kEN- commented Jan 23, 2018

View reviewed changes

BR0kEN- force-pushed the issues/49 branch 3 times, most recently from beee118 to 5889725 Compare January 25, 2018 15:26

BR0kEN- added the in progress label Jan 25, 2018

BR0kEN- added 23 commits January 28, 2018 19:23

#49: Rename the "cikit-apache" to "cikit-apache2"

6744ec1

#49: Include the "cikit-webserver" role to "provision.yml"

d97eb3a

#49: Modify the "cikit-php" role, including the FPM configuration

45a6bf4

#49: Correct the link to Ansible documentation

5120498

#49: Allow to install "nginx" locally

aeda392

#49: Restore previous position of the web server installation

16f0c93

#49: Ensure conflicting web server is not running on remote

bf31194

#49: Allow installing "apache2" or "nginx" locally when starting from…

a8ba06f

… scratch

#49: Remove web servers services reloads

d57199b

#49: Try to run provisioning on Travis

3faf849

#49: Try to build project with "sudo"

3cf7cde

#49: Move provision tests into single Shell script

3fa4d32

#49: Oops, correct the name of variable

7df48de

#49: Try to eliminate host key check

37df4dc

#49: Allow to use "--limit=localhost" to run playbook locally

10ad073

#49: Treat "localhost" as an alias of "127.0.0.1"

e7a3ef1

#49: Sync "cikit" utility inside of VM to the very same directory as …

535c309

…for installation

#49: Corrent the path to utility

7a8acdf

#49: Try "xenial" on Travis

b9052f8

#49: Restore inventory tests

64eec0b

#49: Improve the main Nginx config

b1034bc

#49: Do not run "provision" tests on Travis since Ubuntu Xerial is no…

13ea5cc

…t an option there

#49: [unrelated] Change the link to documentation in Jenkins

be26c64

BR0kEN- added 4 commits January 28, 2018 20:17

#49: Allow to overwrite the "apache2" vhosts within the project

c60428a

#49: Restart the "apache2" service after reconfiguring

f198e28

#49: Add documentation

5c3b13f

#49: Simplify definition of the virtual hosts

911a169

BR0kEN- requested review from lokeoke and timur-hilmutdinov January 29, 2018 09:23

Merge with upstream

3e4124b

BR0kEN- added 4 commits January 31, 2018 16:59

#49: Simplify PHP configuration for the specific web server

c15a8c9

#49: Allow "pre" and "post" configuration of the PHP, depending on a …

563fc1f

…chosen web server

#49: Restore old name of "cikit.j2" template in the "ports.yml"

e65de60

#49: Add the changelog record and increase CIKit version to "1.0.2"

233bf59

BR0kEN- added 7 commits January 31, 2018 21:13

Resolves #49: Add migration path to 1.0.2

940cacc

Resolves #49: Make changelog more descriptive and adjust the document…

b337955

…ation

#49: Simplify the condition for computing the environment provision b…

af94048

…eing run

Resolves #49: Add the link to documentation of web server configurati…

185cc7c

…on to README.md

Resolves #49: Add a new note to the changelog record

766e986

Resolves #49: Reschedule merge to February 4

5c7e605

Resolves #49: Add more info to the changelog record

5f16ed4

lokeoke previously approved these changes Feb 3, 2018

View reviewed changes

timur-hilmutdinov previously approved these changes Feb 3, 2018

View reviewed changes

Resolves #49: Add logical "colon"

fdd1f96

BR0kEN- dismissed stale reviews from timur-hilmutdinov and lokeoke via fdd1f96 February 4, 2018 10:10

BR0kEN- merged commit 64ceed7 into master Feb 4, 2018

BR0kEN- deleted the issues/49 branch March 25, 2018 12:54

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#49: Create the "cikit-webserver" role #90

#49: Create the "cikit-webserver" role #90

BR0kEN- commented Jan 23, 2018 •

edited

BR0kEN- Jan 23, 2018

BR0kEN- commented Jan 29, 2018 •

edited

BR0kEN- commented Jan 29, 2018

BR0kEN- commented Jan 31, 2018

BR0kEN- commented Jan 31, 2018

BR0kEN- commented Feb 2, 2018

lokeoke left a comment

timur-hilmutdinov Feb 3, 2018


		## Required manual actions

		Right after updating the package via `cikit self-update` please make sure all the projects you have based on CIKit will receive the following modifications.

#49: Create the "cikit-webserver" role #90

#49: Create the "cikit-webserver" role #90

Conversation

BR0kEN- commented Jan 23, 2018 • edited

BR0kEN- Jan 23, 2018

Choose a reason for hiding this comment

BR0kEN- commented Jan 29, 2018 • edited

Notes for the record in changelog (manual actions)

BR0kEN- commented Jan 29, 2018

BR0kEN- commented Jan 31, 2018

BR0kEN- commented Jan 31, 2018

BR0kEN- commented Feb 2, 2018

lokeoke left a comment

Choose a reason for hiding this comment

timur-hilmutdinov Feb 3, 2018

Choose a reason for hiding this comment

BR0kEN- commented Jan 23, 2018 •

edited

BR0kEN- commented Jan 29, 2018 •

edited