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.

Sign up for GitHub

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

Jump to bottom

Feature/requirement checks #6

Merged
merged 45 commits into from
Oct 1, 2023
Merged

Feature/requirement checks #6

merged 45 commits into from
Oct 1, 2023

Conversation

martipoe
Copy link
Contributor

New deployer task check:requirements for checking deployment prerequisites

maikschneider and others added 30 commits April 13, 2023 09:25
@maikschneider
feat: init check recipie
a8dd1bb
feat: task check:group and output table
2ccac7e
feat: web directory permission check
92bcd7d
feat: rename task for user checks and ensure SSH user matches deploye…
df73136 
…r remote_user
feat: check:env
3ad42ff
Merge branch 'main' into feature/requirement-checks
4f72919
fix: hide check tasks
af2fbde
feat: check:instance
ffcafdd
feat: check:baseUrl and check:releaseUrl
047527f
feat: rename env checks
69385f4
feat: consolidate .env variable checks in check:env_vars
eb83883
feat: rename check:env to check:env_perms
4e9b067
feat: ensure dns records exist with check:domains
252cefa
feat: ensure urls return valid http response codes with check:urls
efff3a8
feat: check database connection with check:mysql
d365682
feat: check for always required php extensions with check:php_extensions
4158d58
feat: check for locales with check:locales
de14a8d
cleanup: formatting
2f80ebf
feat: check for php apcu extension
428089a
cleanup: reduce calls to vars utility
d47ff5e
fix: closing tags
e79319d
feat: check:php_settings to test mandatory phpfpm settings
c8fa1c1
cleanup: variable naming
cd2fa3b
feat: check:vhost* to ensure Apache vhost meets requirements
02c1bed
feat: include aliases in check:domains
24d0cc9
feat: check:urls handle failed requests and check aliases
51908cd
feat: check:vhost_base for aliases defined in .env
e4b8950
cleanup: style
c4ef195
feat: improved error handling
7e3b0cf
feat: re-order tasks
dd781fb
maikschneider
maikschneider requested changes Apr 19, 2023
View reviewed changes
Copy link
Member

@maikschneider maikschneider left a comment

Choose a reason for hiding this comment

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

Great job! 👍🏻
Just minor cosmetic issues that can be resolved very easy :)

recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
recipe/check_requirements.php Outdated Show resolved Hide resolved
@maikschneider maikschneider mentioned this pull request May 16, 2023
Closed
@maikschneider maikschneider merged commit a553bc6 into main Oct 1, 2023
1 check passed
@maikschneider maikschneider deleted the feature/requirement-checks branch February 20, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maikschneider maikschneider maikschneider approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@martipoe @maikschneider