salt: pkgrepo.absent does not work

on 2015.05 applying the pkgrepo.absent state shows no errors, but silently fails removing the file with the deb line on ubuntu.

----------
          ID: logentries-repo
    Function: pkgrepo.absent
        Name: deb http://rep.logentries.com/ precise main
      Result: True
     Comment: Removed package repo deb http://rep.logentries.com/ precise main
     Started: 07:09:34.239943
    Duration: 122.632 ms
     Changes:   
              ----------
              repo:
                  deb http://rep.logentries.com/ precise main

debug log isn’t helpful, too:

[INFO    ] Running state [deb http://rep.logentries.com/ precise main] at time 07:18:45.521783
[INFO    ] Executing state pkgrepo.absent for deb http://rep.logentries.com/ precise main
[INFO    ] {'repo': 'deb http://rep.logentries.com/ precise main'}
[INFO    ] Completed state [deb http://rep.logentries.com/ precise main] at time 07:18:45.636724

i’ve just changed the whole state from pkgrepo.managed to pkgrepo.absent, so i’m sure there can’t be any typo or missing data.

About this issue

Original URL
State: closed
Created 9 years ago
Comments: 15 (8 by maintainers)

Commits related to this issue

Remove removal of uri trailing backslash Fixes #27181 — committed to wolfpackmars2/salt by wolfpackmars2 5 years ago
remove another rstrip Part of #27181 — committed to wolfpackmars2/salt by wolfpackmars2 5 years ago
Remove remaining rstrips #27181 — committed to wolfpackmars2/salt by wolfpackmars2 5 years ago
remove calls to strip_uri Remove calls to strip_uri in utils.pkg.deb. Debian now stores trailing backslashes #27181 — committed to wolfpackmars2/salt by wolfpackmars2 5 years ago
Do not strip trailing forward slash from repo URI Fixes #27181 Removed instances where trailing forward slash is modified in the URI. By modifying the specified URI, later specifying pkgrepo.absent... — committed to wolfpackmars2/salt by wolfpackmars2 4 years ago
Do not strip trailing forward slash from repo URI Fixes #27181 Removed instances where trailing forward slash is modified in the URI. By modifying the specified URI, later specifying pkgrepo.absent... — committed to saltstack/salt by wolfpackmars2 4 years ago

Most upvoted comments

I’m seeing something similar as well. I am using 2016.3.3.

Minion sources file /etc/apt/sources.list.d/gocd.list:

deb https://download.go.cd /
deb http://download.go.cd/gocd-deb /

SLS directive:

'deb http://download.go.cd/gocd-deb /':
  pkgrepo.absent

Error:

----------
          ID: deb http://download.go.cd/gocd-deb /
    Function: pkgrepo.absent
      Result: False
     Comment: Repo deb http://download.go.cd/gocd-deb / doesn't exist in the sources.list(s)
     Started: 16:14:55.665334
    Duration: 113.954 ms
     Changes:
----------

PredatorVI on Sep 27, 2016