* Automated using https://github.com/myii/ssf-formula/pull/172

* Automated using https://github.com/myii/ssf-formula/pull/167

* Automated using https://github.com/myii/ssf-formula/pull/163

# [1.0.0](https://github.com/saltstack-formulas/firewalld-formula/compare/v0.10.1...v1.0.0) (2020-04-18)

### Bug Fixes

* do not error on unknown os_family grain ([94d2b0b](94d2b0b97c))

### Code Refactoring

* split default maps into separate files ([afcf5e7](afcf5e7700))

### Tests

* **yaml_dump_spec:** update after splitting `map.jinja` ([d1f7a37](d1f7a37171))

### BREAKING CHANGES

* `map.jinja` has been upgraded from using `pillar.get`
to `config.get`.

fix: do not error on unknown os_family grain

While the default maps are mostly empty this sets the groundwork for
distribution specific defaults. The layout is based on the formula
template.

BREAKING CHANGE: `map.jinja` has been upgraded from using `pillar.get`
to `config.get`.

If the formula was ran on a system that reported an os_family that
wasn't one of "Debian", "RedHat", "Arch", "Suse" then the map.jinja
template would fail to render with "'NoneType' is not iterable.

This occurs because grains.filter_by will return None when it fails
match the grain to the input dictionary. The value is then blindly
passed into a dict.update() which causes the failure.

In this patch we ensure that the default values, as defined in
defaults.yaml, are always applied when grain matching fails.

## [0.10.1](https://github.com/saltstack-formulas/firewalld-formula/compare/v0.10.0...v0.10.1) (2020-04-18)

### Tests

* verify map output using `yaml_dump` ([69df9a6](69df9a62d6))

test: verify map output using `yaml_dump`

* Semi-automated using https://github.com/myii/ssf-formula/pull/159

* Automated using https://github.com/myii/ssf-formula/pull/158

# [0.10.0](https://github.com/saltstack-formulas/firewalld-formula/compare/v0.9.0...v0.10.0) (2020-04-02)

### Bug Fixes

* **zone.xml:** adjust whitespacing to pass tests & macro at top of file ([d8f0f47](d8f0f47a54))

### Features

* allow rich_rules to be specified as a dict ([cd4cec0](cd4cec0089))

### Styles

* **zone.xml:** remove all trailing whitespaces ([204efe5](204efe5fc7))

### Tests

* **zones_spec:** check content of rendered zone files ([6ebfc6f](6ebfc6f20c))

Allow rich_rules to be specified as a dictionary

* Checked using https://github.com/myii/ssf-formula/pull/145

When specifying rich rules as a dictionary ipsets and services can be
specified as lists. They will be expanded out by the jinja template into
individual rich rules for the parent zone.

* Automated using https://github.com/myii/ssf-formula/pull/132

* Automated using https://github.com/myii/ssf-formula/pull/129

# [0.9.0](https://github.com/saltstack-formulas/firewalld-formula/compare/v0.8.0...v0.9.0) (2020-02-12)

### Features

* **zone.xml:** allow more services definition inside zone ([8d0172f](8d0172f5c7))

Allow more services definition inside zone

# [0.8.0](https://github.com/saltstack-formulas/firewalld-formula/compare/v0.7.0...v0.8.0) (2020-02-12)

### Continuous Integration

* workaround issues with newly introduced `amazonlinux-1` [skip ci] ([ace3433](ace343353d))
* workaround issues with newly introduced `amazonlinux-1` [skip ci] ([b5a95f3](b5a95f35ab))
* **gemfile:** restrict `train` gem version until upstream fix [skip ci] ([908f5df](908f5df86c))
* **kitchen:** avoid using bootstrap for `master` instances [skip ci] ([0b82e43](0b82e43a15))
* **travis:** apply changes from build config validation [skip ci] ([6e1b876](6e1b876298))
* **travis:** opt-in to `dpl v2` to complete build config validation [skip ci] ([70dc9aa](70dc9aa3b4))
* **travis:** quote pathspecs used with `git ls-files` [skip ci] ([97afbb1](97afbb1575))
* **travis:** run `shellcheck` during lint job [skip ci] ([d8bede7](d8bede7082))
* **travis:** use `major.minor` for `semantic-release` version [skip ci] ([b96cc56](b96cc569fe))
* **travis:** use build config validation (beta) [skip ci] ([07e7900](07e79001cd))

### Features

* standardize license and hand over to saltstack formulas ([20cb8a6](20cb8a60d3))

* Automated using https://github.com/myii/ssf-formula/pull/127

* Automated using https://github.com/myii/ssf-formula/pull/127

Now multiple sections ending with `services` can be defined for each in pillar
and all of them will get merged into one service block in the zone. The goal is
to keep backward compatibility while allowing different services to be defined
in different pillars. So basically have various parts of the pillar affecting
the firewall without need to define everything centrally. Helpful for the
exceptions to the rules.

* Automated using https://github.com/myii/ssf-formula/pull/124

* Automated using https://github.com/myii/ssf-formula/pull/112

* Automated using https://github.com/myii/ssf-formula/pull/110

* Automated using https://github.com/myii/ssf-formula/pull/107

* Automated using https://github.com/myii/ssf-formula/pull/106

* Automated using https://github.com/myii/ssf-formula/pull/100

* Automated using https://github.com/myii/ssf-formula/pull/98

* Automated using https://github.com/myii/ssf-formula/pull/97

# [0.7.0](https://github.com/saltstack-formulas/firewalld-formula/compare/v0.6.2...v0.7.0) (2019-11-09)

### Bug Fixes

* **map.jinja:** fix `salt-lint` errors ([de4e191](de4e1915fb))
* **rubocop:** add fixes using `rubocop --safe-auto-correct` ([8136b75](8136b75fa0))
* **yamllint:** fix all errors ([0f808d6](0f808d6afb))

### Documentation

* **readme:** modify according to standard structure ([3df11fc](3df11fc75c))
* **readme:** move to `docs/` directory ([d47265f](d47265f974))

### Features

* **semantic-release:** implement for this formula ([c5f114d](c5f114d886))

feat(semantic-release): implement for this formula

```bash
firewalld-formula$ yamllint -s .
./pillar.example
  2:1       warning  missing document start "---"  (document-start)
  3:12      warning  truthy value should be one of [false, true]  (truthy)
  12:13     warning  truthy value should be one of [false, true]  (truthy)
  15:89     error    line too long (108 > 88 characters)  (line-length)
  19:13     warning  truthy value should be one of [false, true]  (truthy)
  30:89     error    line too long (363 > 88 characters)  (line-length)
  96:89     error    line too long (170 > 88 characters)  (line-length)
  108:15    error    wrong indentation: expected 12 but found 14  (indentation)
  115:13    error    empty value in block mapping  (empty-values)
  116:2     error    syntax error: found character '%' that cannot start any token
  152:89    error    line too long (112 > 88 characters)  (line-length)

./firewalld/defaults.yaml
  3:1       warning  missing document start "---"  (document-start)
```

```bash
Examining firewalld/map.jinja of type state
[209] Jinja comment should have spaces before and after: {# comment #}
firewalld/map.jinja:4
{## Start with  defaults from defaults.yaml ##}

[209] Jinja comment should have spaces before and after: {# comment #}
firewalld/map.jinja:7
{##

[201] Trailing whitespace
firewalld/map.jinja:16
  }, grain='os_family', merge=salt['pillar.get']('firewalld:lookup'))

[209] Jinja comment should have spaces before and after: {# comment #}
firewalld/map.jinja:19
{## Merge the flavor_map to the default settings ##}

[209] Jinja comment should have spaces before and after: {# comment #}
firewalld/map.jinja:22
{## Merge in salt:lookup pillar ##}
```

* Automated using https://github.com/myii/ssf-formula/pull/95

Remove name attribute in icmp-block-inversion

Firewalld does not parse the name attribute.

Log message:
firewalld[1999]: ERROR: Failed to load zone file 'public.xml':
PARSE_ERROR: icmp-block-inversion: Unexpected attribute name

Fix typo in icmp block inversion key name