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

[BUG] vault ext_pillar does not support KV v2 properly #62651

Closed
2 tasks done
lkubb opened this issue Sep 11, 2022 · 0 comments · Fixed by #62653
Closed
2 tasks done

[BUG] vault ext_pillar does not support KV v2 properly #62651

lkubb opened this issue Sep 11, 2022 · 0 comments · Fixed by #62653
Labels
Bug broken, incorrect, or confusing behavior needs-triage Vault

Comments

@lkubb
Copy link
Contributor

lkubb commented Sep 11, 2022

Description
Using KV v2 paths in the Vault ext_pillar results in "data" and "metadata" pillar keys, where "data" contains the expected pillar values.

Setup
Essential master configuration:

[...]
ext_pillar:
  - vault: path=salt/pillar/path
[...]

Please be as specific as possible and give set-up details.

  • on-prem machine
  • VM (KVM)

Expected behavior
The pillar to contain data.

Screenshots

$ salt-run pillar.show_pillar minion
[...]
data:
    ----------
    foo:
        bar
metadata:
    ----------
    created_time:
[...]

Versions Report

salt --versions-report (Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)
Salt Version:
          Salt: 3004.2

Dependency Versions:
          cffi: Not Installed
      cherrypy: Not Installed
      dateutil: Not Installed
     docker-py: Not Installed
         gitdb: 2.0.5
     gitpython: 2.1.9
        Jinja2: 2.11.3
       libgit2: Not Installed
      M2Crypto: 0.38.0
          Mako: Not Installed
       msgpack: 0.6.2
  msgpack-pure: Not Installed
  mysql-python: Not Installed
     pycparser: Not Installed
      pycrypto: 3.10.1
  pycryptodome: 3.14.1
        pygit2: Not Installed
        Python: 3.8.12 (default, Sep 26 2021, 13:12:50)
  python-gnupg: Not Installed
        PyYAML: 5.3.1
         PyZMQ: 18.0.1
         smmap: 2.0.3
       timelib: Not Installed
       Tornado: 4.5.3
           ZMQ: 4.3.4

System Versions:
          dist: openbsd 7.0
        locale: utf-8
       machine: amd64
       release: 7.0
        system: OpenBSD
       version: OpenBSD 7.0

Additional context
This part does not check for KV v2. PR incoming soon.

@lkubb lkubb added Bug broken, incorrect, or confusing behavior needs-triage labels Sep 11, 2022
@lkubb lkubb changed the title [BUG] vault ext_pillar does not support KV v2 [BUG] vault ext_pillar does not support KV v2 properly Sep 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior needs-triage Vault
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants