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.

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 expectations on cloud-config #6

Closed
drnic opened this issue Apr 24, 2018 · 2 comments
Closed

Reduce expectations on cloud-config #6

drnic opened this issue Apr 24, 2018 · 2 comments
Assignees

Comments

@drnic
Copy link

drnic commented Apr 24, 2018

As per #4 and #5, the goal of one base manifest is for bosh deploy manifests/harbor.yml -d harbor to "just work".

This is made more likely if the base manifest does not put unexpected requirements on the cloud-config that any normal BOSH/CF/CFCR user wouldn't already have.

From one of your manifests:

instance_groups:
- name: harbor
  vm_type: standard
  persistent_disk_type: 20G
  networks:
  - name: default
    static_ips:
    - 10.112.123.31

Each of these three items can be changed to reduce the chance that a user will have an error when initially deploying:

  • vm_type: can be changed to vm_resource: - see http://bosh.io/docs/manifest-v2/#instance-groups will delegate picking an instance type to the CPI, rather than require you to guess what vm_types are in a cloud-config
  • persistent_disk_type: can change to persistent_disk: 20480. The CPI has default cloud_properties for each disk. The persistent_disk_type: attribute is only required if the deployer wants to have their disk have non-default cloud_properties. Leave that to them if they need it.
  • networks: has a good assumption that all/most cloud-config has name: default. The one change to make is to remove static_ips from the base manifest. You don't know anything about a user's cloud-config and don't know what static_ips they have allocated.

/cc @cppforlife any other pro tips?

@jessehu jessehu self-assigned this May 15, 2018
@jessehu
Copy link
Contributor

jessehu commented May 17, 2018

comments addressed in 089c7c5

@jessehu jessehu closed this as completed May 17, 2018
@drnic
Copy link
Author

drnic commented May 17, 2018

Nice work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants