Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

Fix sdb paths so that Salt loader picks them up properly #19

Merged
merged 10 commits into from
Nov 18, 2022
Merged

Fix sdb paths so that Salt loader picks them up properly #19

merged 10 commits into from
Nov 18, 2022

Conversation

ggiesen
Copy link
Contributor

@ggiesen ggiesen commented Nov 14, 2022

Adjust SDB paths (dropping the trailing 's') so they are picked up by Salt. Also, add template for SDB module.

Fixes #18

Copy link
Contributor

@waynew waynew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That'd do it 😂

One could argue that we may also want to offer more like:

  • auth
  • acl
  • beacons
  • cache
  • channel???
  • cli
  • cloud
  • grains
  • netapi??
  • output??
  • pillar??
  • platform??
  • proxy
  • returners
  • runners
  • spm
  • ???

But... that's probably more than we need for this PR.

@s0undt3ch
Copy link
Contributor

Can you rebase please?

@ggiesen
Copy link
Contributor Author

ggiesen commented Nov 18, 2022

That'd do it 😂

One could argue that we may also want to offer more like:

* auth

* acl

* beacons

* cache

* channel???

* cli

* cloud

* grains

* netapi??

* output??

* pillar??

* platform??

* proxy

* returners

* runners

* spm

* ???

But... that's probably more than we need for this PR.

yeah I'm aware, was starting small and wanted to see how this one went first. :)

@ggiesen
Copy link
Contributor Author

ggiesen commented Nov 18, 2022

Can you rebase please?

@s0undt3ch Done

@s0undt3ch s0undt3ch merged commit e672cc4 into saltstack:main Nov 18, 2022
@ggiesen ggiesen deleted the fix_sdb branch November 18, 2022 13:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with SDB loader
4 participants