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

Implement the "setup" command for Docker installation via SSH #6

Open
wants to merge 30 commits into
base: trunk
Choose a base branch
from
Open

Implement the "setup" command for Docker installation via SSH #6

wants to merge 30 commits into from

Conversation

jorgecar
Copy link
Contributor

@jorgecar jorgecar commented Jun 6, 2023 •
edited
Loading

This pull request addresses issue #5 by implementing the "setup" command, which acquires a server lock to prevent concurrent operations and facilitates the installation of Docker via SSH.

Changes Made

  • Developed a server lock mechanism to acquire and release locks for exclusive server access.
  • Implemented error handling to handle scenarios where the server lock cannot be acquired, providing informative error messages for troubleshooting.
  • Enhanced the "setup" command to detect the presence of Docker on the server.
  • Added functionality to install Docker using the appropriate package manager via SSH if it is not already installed.
  • Improved feedback and error messaging during the installation process to assist users in resolving any encountered issues.
  • Ensured proper release of the server lock upon completion or in case of any errors during the installation process33.

Testing done

  • Tested the "setup" command on various servers with different configurations to ensure successful server lock acquisition and Docker installation.
  • Validated the behavior of error handling and error messaging during the installation process.

jorgecar added 12 commits June 6, 2023 09:22
@jorgecar
feat: Makes the SetupCommand class
cc85329
@jorgecar
feat: Makes the LockAcquireCommand class
ee1acad
@jorgecar
feat: Makes the LockReleaseCommand class
0eb31b7
@jorgecar
feat: Implements the Processes\ServerLock::class
dde242e
@jorgecar
feat: Implements Processes\ProcessBase::class
b9c1f60
@jorgecar
feat: Implements the ProcessRunner::class
c3edeac
@jorgecar
feat: Binds the ConsoleOutput to the Service Container
c2e5e9a 
This will be used to write a message to the Console from anywhere in the application.
@jorgecar
feat: Implements the Manifest::primaryServer method
21c3851
@jorgecar
feat: Implements the trait RunnableOverSSH
5d2f955 
This trait has the logic to run any script on a remote server via SSH.

I put it separately because it is where the connection to the remote server is defined, and in case of errors, the code is totally isolated.
@jorgecar
feat: Defines the path for the lockfile on the config/drzl.php
f97a935
@jorgecar
feat: Implements the lock:acquire command
017e98f
@jorgecar
feat: Implements the lock:release command
64341d1
@jorgecar jorgecar linked an issue Jun 6, 2023 that may be closed by this pull request
Implement "setup" command for Docker installation via SSH #5
Open
jorgecar added 17 commits June 13, 2023 14:12
@jorgecar
feat: Implement the servers method
20c5f32 
This new method is used to get the first server too.
@jorgecar
feat: Implement the BaseCommand class
883e49e 
This way I avoid having to define the same constructor in all commands to inject the manifest.
@jorgecar
refactor: Remove the construct and extends from BaseCommand
149eda4
@jorgecar
feat: Implement a trait to run commands with the server locked
405b2ae
@jorgecar
feat: Implements the Docker process class
6509d17
@jorgecar
feat: Implements the withOutput & docker method
d541f5f 
- withOutput: this method active the stream output for the processes

- docker: is a shortcut for execute docker cli
@jorgecar
feat: Implements the condicional stream output
cc67d5b 
Accepts a boolean parameter stream to know if the stream has to be written to the console.
@jorgecar
refactor: Use the Command Output for the process stream
ad37f44 
The Command Output is an OutputStyle object which is more sophisticated than the ConsoleOutput object.

It also avoids the magic of bind the output object on the service provider.
@jorgecar
feat: Implements a withOutput method to set the Command output in the…
293ca57 
… process
@jorgecar
feat: Implements the ProcessBase interface
f04ee22 
It is required to declarate the withOutput method for all the Process objects
@jorgecar
feat: Declares the ProcessInterface implementation
daa4cf9
@jorgecar
feat: Implements the server:bootstrap command
6b8f3e2
@jorgecar
feat: Calls the server:bootstrap command from setup command
ec397f8
@jorgecar
feat: Adds a custom Exception Handler
f3cd501
@jorgecar
refactor: Handles exception from the custom Exception Handler
caddfb2
@jorgecar
refactor: Coding style
6bafbb0
@jorgecar
docs(lock:acquire): Set the command description
7422316
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement "setup" command for Docker installation via SSH
1 participant
@jorgecar