Because the output of the bodhi API is different than the one from pdc, the get_stream_branches does not return list of dicts, but a list (of active releases).get_stream_branches now uses get_pagure_branches to get the package active branches
get_release_branches should keep returning the same as before, a dict.

Ready for review.

Finally flake8 gave up and let me be, phew!

One little change added, no need to loop through active releases since we can exclude the archived.

@onosek would you be the right person to review these changes?

Hello, I am sorry for the delay. Initially, I started reviewing the PR, but later I was assigned to a project with higher priority. I am back now so I can focus on this again.
I will look at it once more, because I forgot the details since then.

In the past, I tried to search for "service_levels" (sls) among old branch requests. I didn't find any usage. I don't know whether it still has some purpose. I would say it doesn't. But the safer approach is just to strip the PRC (like you did) and keep the rest.

please don't forget fedpkg releases-info

ack show_releases_info

cli.py
547:        parser.set_defaults(command=self.show_releases_info)
1450:    def show_releases_info(self):

def show_releases_info(self):
        server_url = self.config.get('{0}.pdc'.format(self.name), 'url')

I think Lenka changed show_releases_info correctly. It gets data from the Bodhi.

I will just change

    active_branches = []
    for active_branch in query_bodhi(server_url):
        if active_branch not in active_branches:
            active_branches.append(active_branch)

|
|
v

    active_branches = set(query_bodhi(server_url))

because it appears to be more readable to me.

Commit 9a7f4629 fixes this pull-request

I finally did some testing. Thanks for the change.

Thanks a lot for the review!