createrepo fails to update repo settings via REST API

Bug #1458369 reported by Mike Scherbakov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Vitaly Parakhin

Bug Description

Build #461
Steps to repro:
1) change password via UI
2) change password in /etc/fuel/client/config.yaml, so fuel CLI can work
3) create new ubuntu env
4) run fuel-createmirror. It will have:
....
 * INFO: Done
 * INFO: MOS mirror was created at: /var/www/nailgun/mos-ubuntu
 * INFO: Attempting to add created repositories to Nailgun...
 * INFO: Failed to add repositories for environment id=1 to Nailgun, please add them MANUALLY
 * INFO: Created repositories were set as defaults for new environments
 * INFO: In order to setup these repositories MANUALLY, you should
         go to Fuel UI, choose your cluster and go to the 'Settings' tab
         Replace the URI value for the following repositories:
         Repository "mos6.1-updates" URI="deb http://10.108.25.2:8080/mos-ubuntu mos6.1-updates main restricted"
         Repository "mos6.1-security" URI="deb http://10.108.25.2:8080/mos-ubuntu mos6.1-security main restricted"
         Repository "mos6.1-holdback" URI="deb http://10.108.25.2:8080/mos-ubuntu mos6.1-holdback main restricted"

 * INFO: Ubuntu partial mirror was created at: /var/www/nailgun/ubuntu-part
 * INFO: Attempting to add created repositories to Nailgun...
 * INFO: Failed to add repositories for environment id=1 to Nailgun, please add them MANUALLY
 * INFO: Created repositories were set as defaults for new environments
 * INFO: In order to setup these repositories MANUALLY, you should
         go to Fuel UI, choose your cluster and go to the 'Settings' tab
         REMOVE all repositories named "ubuntu-*" and change URI for the following one:

         Repository "ubuntu" new URI="deb http://10.108.25.2:8080/ubuntu-part trusty main"

Expected:
* Script will use /etc/fuel/client/config.yaml as a source of new login/pass, verify if it works first
* If it doesn't, it should fail right away and show error message, and what exactly doesn't work. Currently it says "failed to add repos" but doesn't explain why. It is not even clear, if it was authz issue or any other.

Mike Scherbakov (mihgen)
Changed in fuel:
assignee: nobody → Vitaly Parakhin (vparakhin)
Revision history for this message
Sebastian Kalinowski (prmtl) wrote :

/etc/fuel/client/config.yaml is deprecated and will be removed. Do not use it.

Revision history for this message
Vitaly Parakhin (vparakhin) wrote :

What would be the most appropriate source to get the correct password from?

Revision history for this message
Sebastian Kalinowski (prmtl) wrote :

Ideally prompt - storing passwords anywhere is a really bad idea

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to packages/centos6/fuel-createmirror (6.1)

Fix proposed to branch: 6.1
Change author: Vitaly Parakhin <email address hidden>
Review: https://review.fuel-infra.org/6936

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to packages/trusty/fuel-createmirror (6.1)

Fix proposed to branch: 6.1
Change author: Vitaly Parakhin <email address hidden>
Review: https://review.fuel-infra.org/6937

Changed in fuel:
status: New → In Progress
importance: Undecided → High
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to packages/centos6/fuel-createmirror (6.1)

Reviewed: https://review.fuel-infra.org/6936
Submitter: Vitaly Parakhin <email address hidden>
Branch: 6.1

Commit: fa0318b22aa1170c039774baff8aed3f08935ac7
Author: Vitaly Parakhin <email address hidden>
Date: Mon May 25 14:16:01 2015

Fixes to automatic repo updates

1) Added Fuel backend connectivity check
2) Added option to specify Fuel Master admin password
3) Fixed name of repos in script output

Closes-Bug: #1458370
Closes-Bug: #1458369

Change-Id: I4f1d84d18a68f4fac061ff4ad19a8e85063a01b6

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to packages/trusty/fuel-createmirror (6.1)

Reviewed: https://review.fuel-infra.org/6937
Submitter: Vitaly Parakhin <email address hidden>
Branch: 6.1

Commit: d4e7a2d0d383b3a94e80b0f2d3b2be05154819e8
Author: Vitaly Parakhin <email address hidden>
Date: Mon May 25 14:56:15 2015

Fixes to automatic repo updates

1) Added Fuel backend connectivity check
2) Added option to specify Fuel Master admin password
3) Fixed name of repos in script output

Change-Id: Ia1b7cb9b7146bd0f338993269f3ac064315be8fd
Closes-Bug: #1458370
Closes-Bug: #1458369

Changed in fuel:
status: In Progress → Fix Committed
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.