-
Notifications
You must be signed in to change notification settings - Fork 3
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
Reduce fetch depth #none #2
Open
jbygdell
wants to merge
61
commits into
jbygdell:master
Choose a base branch
from
neicnordic:reduce_fetch_depth
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
Conversation
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 labels for metadata and move to an alpine base image
This is used to inject the metadata variables
Feature/docker image
Adding templates
Use static shovels
Update rabbitmq config to enable TLS for local connections. The following restrictions apply; verification of both the server and client is enforced. Connection is not allowed unless mutual verification is ok. TLS version 1.2 or above is required.
This is due to how the new connection string looks like.
Add ca-certificates so that we can validate publicly signed certificates.
Add TLS config for local connections
Fixing permissions on some files read by rabbitmq user
Improve security by not starting the container as root. As long as injected files are placed in the appropriate paths they will still be owned by the rabbitmq user.
Allow the user to explicitly name the default `vhost` to something other than `/`. Add option to disable peer verification of tls certificates. Remove unnecessary ownership changes of injected files.
Add config options
Update build hook to use nbis repository
Add default values to some ENVs to specify where the persisted config files are located.
Remove bash specific tests and ensure that the config files are written to persistent storage.
Sets up the MQ for standalone configuration when the ENV `CEGA_CONNECTION` is absent
One further step in migrating from the (l)ega naming schema.
Standalone
We should publish to this queue whenever we have successfully verified a file.
Shovel messages from files.verified
New config
New config
Add backup queue
Update README.md
fix missing mappings queue and source for stand-alone
bump version & publish dockerhub & github registry #none
Adds the NOTLS env to disable SSL completely
This is not needed anymore since Docker Hub is not building images any longer.
NoTLS option #minor
Set fetch depth to 1
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Reduce fetch depth to 1 in order to prevent issues with new tags