sous-chefs / consul

Development repository for the consul cookbook
https://supermarket.chef.io/cookbooks/consul
Apache License 2.0
192 stars 244 forks source link

Consul 1.9.x - ui/ui_dir deprecated #591

Closed askmike1 closed 2 years ago

askmike1 commented 3 years ago

:speaking_head: Foreword

Thank for taking the time to fill this bug report fully. Without it we may not be able to fix the bug, and the issue may be closed without resolution.

:ghost: Brief Description

With 1.9.x, ui & ui_dir are deprecated in favor of ui_config.{enabled, dir, content_path}, cookbook needs to support this functionality

:pancakes: Cookbook version

4.5.2

:woman_cook: Chef-Infra Version

16.13.16

:tophat: Platform details

RHEL7

Steps To Reproduce

Steps to reproduce the behavior:

  1. Specify default['consul']['config']['ui_config.enabled']

:police_car: Expected behavior

It should set the new variale names

:heavy_plus_sign: Additional context

Add any other context about the problem here. e.g. related issues or existing pull requests.

askmike1 commented 3 years ago

There are a number of other deprecated fields as well that this cookbook doesn't seem to support the more modern replacements, including: acl_datacenter - This field is deprecated in Consul 1.4.0. See the primary_datacenter field instead. acl_default_policy - Deprecated in Consul 1.4.0. See the acl.default_policy field instead acl_down_policy - Deprecated in Consul 1.4.0. See the acl.down_policy field instead acl_agent_master_token - Deprecated in Consul 1.4.0. See the acl.tokens.agent_master field instead acl_agent_token - Deprecated in Consul 1.4.0. See the acl.tokens.agent field instead acl_master_token - Deprecated in Consul 1.4.0. See the acl.tokens.master field instead acl_replication_token - Deprecated in Consul 1.4.0. See the acl.tokens.replication field instead acl_token - Deprecated in Consul 1.4.0. See the acl.tokens.default field instead acl_ttl - Deprecated in Consul 1.4.0. See the acl.token_ttl field instead