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

Align name of updates for Ubuntu client tools #880

Open
Bischoff opened this issue May 5, 2021 · 1 comment
Open

Align name of updates for Ubuntu client tools #880

Bischoff opened this issue May 5, 2021 · 1 comment

Comments

@Bischoff
Copy link
Contributor

Bischoff commented May 5, 2021

We have the following repository names:

  • tools_pool_repo.list --> pool
  • Ubuntu2004-Client-Tools.list --> updates

Shouldn't we align the second one to tools_update_repo.list?

Sorry in advance if I missed something, I am still getting lost in our repositories jungle.

@srbarrios
Copy link
Member

srbarrios commented May 5, 2021

I think we just need to remove the line - file: /etc/apt/sources.list.d/Ubuntu{{ short_release }}-Client-Tools.list

tools_update_repo:
  pkgrepo.managed:
    - file: /etc/apt/sources.list.d/Ubuntu{{ short_release }}-Client-Tools.list
{% if 'head' in grains.get('product_version') | default('', true) %}
{% set tools_repo_url = 'http://' + grains.get("mirror") | default("download.suse.de", true) + '/ibs/Devel:/Galaxy:/Manager:/Head:/Ubuntu' + release + '-SUSE-Manager-Tools/xUbuntu_' + release %}
{% elif 'beta' in grains.get('product_version') | default('', true) %}
{% set tools_repo_url = 'http://' + grains.get("mirror") | default("download.suse.de/ibs", true) + '/SUSE/Updates/Ubuntu/' + release + '-CLIENT-TOOLS-BETA/x86_64/update/' %}
{% elif 'released' in grains.get('product_version') | default('', true) %}
{% set tools_repo_url = 'http://' + grains.get("mirror") | default("download.suse.de/ibs", true) + '/SUSE/Updates/Ubuntu/' + release + '-CLIENT-TOOLS/x86_64/update/' %}
# We only have one shared Client Tools repository, so we are using 4.1 for 4.0 and 3.2
{% elif 'nightly' in grains.get('product_version') | default('', true) %}
{% set tools_repo_url = 'http://' + grains.get("mirror") | default("download.suse.de", true) + '/ibs/Devel:/Galaxy:/Manager:/Head:/Ubuntu' + release + '-SUSE-Manager-Tools/xUbuntu_' + release %}
{% elif 'uyuni-master' in grains.get('product_version') | default('', true) %}
{% set tools_repo_url = 'http://' + grains.get("mirror") | default("download.opensuse.org", true) + '/repositories/systemsmanagement:/Uyuni:/Master:/Ubuntu' + short_release + '-Uyuni-Client-Tools/xUbuntu_' + release %}
{% else %}
{% set tools_repo_url = 'http://' + grains.get("mirror") | default("download.opensuse.org", true) + '/repositories/systemsmanagement:/Uyuni:/Stable:/Ubuntu' + short_release + '-Uyuni-Client-Tools/xUbuntu_' + release %}
{% endif %}
    - name: deb {{ tools_repo_url }} /
    - key_url: {{ tools_repo_url }}/Release.key

If doesn't work, because it needs to have appended ".list", then... just rename the file param

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants