-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/registry drone roles #42
Open
PabloPie
wants to merge
22
commits into
master
Choose a base branch
from
feature/registry-drone-roles
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
2847a29
Rename caliopen cert and key to match naming
PabloPie a57645a
Install also the base *.caliopen.org cert
PabloPie dde0ff6
Add role for docker registry
PabloPie 1d4c868
Add base role for vms not connected to private vlan
PabloPie 3f0a2de
Role to install docker on machines that need it
PabloPie beca854
Add role for drone server
PabloPie 5a0cb4b
Nginx not running as a container the IP is needed
PabloPie 16bcf7a
Fix naming and end of files
PabloPie dba782f
Variable is now named caliopen_domain_base
PabloPie b24fa8b
VM should also be created...
PabloPie b6fe69c
nginx conf files do not need http directive
PabloPie bc0aebd
Upgrade machine on start
PabloPie 11db4ff
fix docker compose command
PabloPie 1a0ace9
Create auth directory
PabloPie f3926a4
add tag to skip monitoring tasks
PabloPie 1d040fa
Update caliopen certs
PabloPie 1d8d349
Password change
PabloPie 7d3551d
Getting the release codename from the machine allows more version fle…
PabloPie 7df4d75
Expose directly drone on port 8000 to avoid conflicts with nginx
PabloPie 8a3b7fa
Tags no longer needed
PabloPie 861f50d
add_header directive correction
PabloPie 7593456
Building applications needs more RAM and cpu
PabloPie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
registry is deployed with extra parameter --skip-tags monitoring
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not fan of tags for that purpose, as we will forget to give --skip-tags flag.
May include the nginx part directly in registry task ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can just install it even if it isn't monitored atm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok