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

Add authcfg, role, user and usergroup API providers #829

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bdeferme
Copy link

Pull Request (PR) description

Adds a Puppet provider using the API for the following resources:

  • Zabbix Authconfig
  • Zabbix User
  • Zabbix Usergroup
  • Zabbix Roles

This Pull Request (PR) fixes the following issues

n/a

@bdeferme
Copy link
Author

bdeferme commented Sep 9, 2022

Any news on reviewing 😃 ?

@smortex
Copy link
Member

smortex commented Sep 9, 2022

CI was recently fixed. Can you please rebase your changes on top of the main branch?

From your working directory:

git fetch origin         # Download the latest code we have here
git rebase origin/master # Move your commits on top of the main branch
git push -f              # Send the changes to update the PR (-f is required because we re-wrote history)

@smortex smortex changed the title Api providers Add authconfig, role, user and usergroup API providers Sep 9, 2022
@smortex smortex changed the title Add authconfig, role, user and usergroup API providers Add authcfg, role, user and usergroup API providers Sep 9, 2022
@smortex smortex added the enhancement New feature or request label Sep 9, 2022
@bdeferme
Copy link
Author

I think the test just needs to be re-started

@teluq-pbrideau
Copy link
Contributor

@bdeferme I’m also interrested in this! Could you rebase from master to check if tests are fixed? If there is a way I could help, to fix test issues, etc, I could do the same as I did with #828 if you’re OK with it (create a new PR where I could fix stuff).

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

Successfully merging this pull request may close these issues.

3 participants