subscription-manager icon indicating copy to clipboard operation
subscription-manager copied to clipboard

2101510: [1.28] Fix D-Bus Register() and update entitlement certs

Open jirihnidek opened this issue 3 years ago • 4 comments

  • BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2101510
  • Backport of PR #3080
    • It wasn't possible to do simple cherry-pick
  • Card ID: ENT-5185
  • When D-Bus method Register() was called with argument enable_content == True and entitlement mode was used, then entitlement certificates were not installed on the system
  • Added several unit tests

jirihnidek avatar Aug 29 '22 10:08 jirihnidek

There are some issues with unit tests. I will fix them...

jirihnidek avatar Aug 29 '22 10:08 jirihnidek

The failed CI jobs is related to rpmlint. It seems that this commit has to be backported:

https://github.com/candlepin/subscription-manager/commit/4af3f18f50c2e30090f7ae28df2e8e45c695348c

We use Fedora 35 and it seems that the current rpmlint does not support --file CLI option anymore. I would like to backport this change in different PR.

jirihnidek avatar Aug 30 '22 14:08 jirihnidek

The failed CI jobs is related to rpmlint. It seems that this commit has to be backported:

4af3f18

We use Fedora 35 and it seems that the current rpmlint does not support --file CLI option anymore. I would like to backport this change in different PR.

Yes please. I'm curious to know whether that commit is enough, or more commits from main are needed. Let's start with that one :)

ptoscano avatar Aug 30 '22 16:08 ptoscano

It looks like that #3121 fixes the issue with rpmlint.

jirihnidek avatar Aug 31 '22 08:08 jirihnidek