-
Notifications
You must be signed in to change notification settings - Fork 414
Add new partitioner script, which can do job on first boot #1239
Conversation
27d5b9e
to
e2d302b
Compare
e2d302b
to
03075d0
Compare
Also refactor mantl-storage-setup a bit
e91aefa
to
8c380f3
Compare
Add docker-io's docker support from @stevendborrelli |
@@ -0,0 +1,6 @@ | |||
[dockerrepo] |
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.
There are 2 docker repos in the PR.
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.
Correct, I'll remove mine
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.
This has been done.
Documentation needs to be updated before this can go in. |
Still need to be included/linked from main doc
We should update the Packer image after this is merged. |
Order of execution | ||
------------------ | ||
|
||
All configuration rules reside in ``/etc/mantl/filesystems.d``, and processed |
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.
Maybe we could include an example configuration? Or include roles/lvm/templates/mantl-volume-group.conf.j2
is an example?
@avnik outlined these testing criteria:
|
on a clean install on AWS, I get the following error on the ansible run:
any idea? @avnik @stevendborrelli |
same thing on an upgrade from 1.0.3 |
This package cleans up docker containers, images and volues by adding systemd timers tht run commands like docker rm
@ryane I guess it because we not update package with mantl-storage-setup, I'll do this atm (because I think code of partitioner enough stable). |
- "--dns {{ private_ipv4 }}" | ||
- "--log-driver={{ docker_log_driver }}" | ||
- "{% if docker_selinux_enabled %}--selinux-enabled {% endif %}" | ||
- "{% if kube_build is defined %}--dns-search {{ cluster_name }}{% endif %}" |
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.
is kube_build ever set anywhere?
successful clean build on GCE. However, I am seeing some unexpected docker errors. I restarted the nginx-consul service on a node and received the following:
The container is marked as
|
the issue doesn't seem to be directly related to the code in this pr but just may be a bug with the current version of docker. There are some similar issues open:
After discussing, we will merge this and workaround the immediate problem in #1390. We will open new issues for any other that may crop up. |
Fixes #1240