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

Provider overrides appear to be broken #27209

Closed
justinta opened this issue Sep 17, 2015 · 4 comments
Closed

Provider overrides appear to be broken #27209

justinta opened this issue Sep 17, 2015 · 4 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior Core relates to code central or existential to Salt fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around ZD The issue is related to a Zendesk customer support ticket.
Milestone

Comments

@justinta
Copy link

On an ubuntu system I set the provider to yumpkg/yumpkg5 and was still able to install packages. This should cause a stack trace.
@terminalmage @basepi

Output below shows yumpkg5 as the provider, the minion restarted, what os is running, and htop being installed.

root@li162-102:~# cat /etc/salt/minion | grep -A 2 providers
# via the providers option, in this case the default module will be
# overwritten by the specified module. In this example the pkg module will
# be provided by the yumpkg5 module instead of the system default.
#providers:
#  pkg: yumpkg5
providers:
  pkg: yumpkg5
#
root@li162-102:~# service salt-minion restart
salt-minion stop/waiting
salt-minion start/running, process 8718
root@li162-102:~# salt '*' grains.get os
li162-102.members.linode.com:
    Ubuntu
root@li162-102:~# salt '*' pkg.install htop
li162-102.members.linode.com:
    ----------
    htop:
        ----------
        new:
            1.0.2-3
        old:
@justinta
Copy link
Author

#26994

@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Sep 18, 2015
@jfindlay jfindlay added this to the Approved milestone Sep 18, 2015
@basepi
Copy link
Contributor

basepi commented Sep 21, 2015

I also think based on some customer testing that even in cases where it tries to override, it's not packing __salt__ and other values properly in the overridden provider. I'll be looking into this this week.

@basepi basepi self-assigned this Sep 21, 2015
@basepi basepi modified the milestones: B 12, Approved Sep 21, 2015
@meggiebot meggiebot added the ZD The issue is related to a Zendesk customer support ticket. label Sep 21, 2015
@meggiebot
Copy link

ZD 416

@basepi basepi added Core relates to code central or existential to Salt and removed Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Sep 21, 2015
@basepi basepi added the fixed-pls-verify fix is linked, bug author to confirm fix label Sep 24, 2015
@basepi
Copy link
Contributor

basepi commented Sep 24, 2015

Fixed in #27388

@basepi basepi closed this as completed Sep 25, 2015
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 Core relates to code central or existential to Salt fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around ZD The issue is related to a Zendesk customer support ticket.
Projects
None yet
Development

No branches or pull requests

4 participants