Bug 889957 - [fork][model refactor]Can't change user plan by RESTAPI
Summary: [fork][model refactor]Can't change user plan by RESTAPI
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OKD
Classification: Red Hat
Component: Master
Version: 2.x
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: ---
Assignee: Ravi Sankar
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-24 07:00 UTC by Rony Gong 🔥
Modified: 2015-05-15 00:53 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-13 23:16:43 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
development.log (3.34 KB, text/plain)
2012-12-24 07:00 UTC, Rony Gong 🔥
no flags Details

Description Rony Gong 🔥 2012-12-24 07:00:09 UTC
Created attachment 668336 [details]
development.log

Description of problem:
Can't change user plan by RESTAPI:
 curl -k -X PUT -H 'Accept: application/xml' --user qgong:111111 https://ec2-184-73-121-221.compute-1.amazonaws.com/broker/rest/user/ -d plan_id=megashift
Show error in log:
2012-12-24 01:30:49.597 [DEBUG] application/xml (pid:2603)
2012-12-24 01:30:49.601 [INFO ] Completed 500 Internal Server Error in 6ms (pid:2603)
2012-12-24 01:30:49.606 [FATAL] NoMethodError (undefined method `render_format_exception' for #<UserExtController:0x00000006c6f3d0>):
  app/controllers/user_ext_controller.rb:26:in `rescue in update'
  app/controllers/user_ext_controller.rb:23:in `update' (pid:2603)
2012-12-24 01:30:49.714 [INFO ] Rendered /opt/rh/ruby193/root/usr/share/gems/gems/actionpack-3.2.8/lib/action_dispatch/middleware/templates/rescues/_trace.erb (1.8ms) (pid:2603)
2012-12-24 01:30:49.718 [INFO ] Rendered /opt/rh/ruby193/root/usr/share/gems/gems/actionpack-3.2.8/lib/action_dispatch/middleware/templates/rescues/_request_and_response.erb (1.2ms) (pid:2603)
2012-12-24 01:30:49.718 [INFO ] Rendered /opt/rh/ruby193/root/usr/share/gems/gems/actionpack-3.2.8/lib/action_dispatch/middleware/templates/rescues/diagnostics.erb within rescues/layout (104.6ms) (pid:2603)
2012-12-24 01:30:49.746 [INFO ] Started GET "/broker/rest/domains/qgong16/applications/qjbossas/cartridges" for 203.114.244.88 at 2012-12-24 01:30:49 -0500 (pid:2603)
2012-12-24 01:30:49.749 [INFO ] Processing by EmbCartController#index as JSON (pid:2603)


Version-Release number of selected component (if applicable):
fork_ami_refctr1_372

How reproducible:
Always


Steps to Reproduce:
1. curl -k -X PUT -H 'Accept: application/xml' --user qgong:111111 https://ec2-184-73-121-221.compute-1.amazonaws.com/broker/rest/user/ -d plan_id=megashift

  
Actual results:
[qgong@localhost dev]$ curl -k -X PUT -H 'Accept: application/xml' --user qgong:111111 https://ec2-184-73-121-221.compute-1.amazonaws.com/broker/rest/user/ -d plan_id=megashift
<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="utf-8" />
  <title>Action Controller: Exception caught</title>
  <style>
    body { background-color: #fff; color: #333; }

    body, p, ol, ul, td {
      font-family: helvetica, verdana, arial, sans-serif;
      font-size:   13px;
      line-height: 18px;
    }

    pre {
      background-color: #eee;
      padding: 10px;
      font-size: 11px;
      white-space: pre-wrap;
    }

    a { color: #000; }
    a:visited { color: #666; }
    a:hover { color: #fff; background-color:#000; }
  </style>
</head>
<body>

<h1>
  NoMethodError
    in UserExtController#update
</h1>
<pre>undefined method `render_format_exception&#x27; for #&lt;UserExtController:0x00000006ce6390&gt;</pre>





Expected results:
Could set user plan sussessfully

Additional info:

Comment 1 Yujie Zhang 2012-12-24 09:09:39 UTC
The same issue exists on website, user can not upgrade their plan on website.

Comment 2 Rony Gong 🔥 2012-12-24 09:12:59 UTC
Also faild for --listsubaccounts (after add subaccount by RESTAPI)
root@ip-10-118-62-244 openshift]# oo-admin-ctl-user --listsubaccounts -l wsun


/usr/sbin/oo-admin-ctl-user:347:in `<main>': undefined method `find_subaccounts_by_parent_login' for CloudUser:Class (NoMethodError)

Comment 3 Rony Gong 🔥 2012-12-24 09:20:02 UTC
Sorry, please ingore the comment2, wrong comment!

Comment 4 Ravi Sankar 2013-01-04 01:59:30 UTC
Fixed, commit rev# fcb5499b09c34b3 [kraman/li/features/model_refactor branch]

Comment 5 Rony Gong 🔥 2013-01-04 05:16:33 UTC
Need wait for new build to verify!

Comment 6 Rony Gong 🔥 2013-01-05 02:46:41 UTC
Verified on fork_ami_refctr1_381
[qgong@localhost rhc2]$ curl -k -X PUT -H 'Accept: application/xml' --user qgong:111111 https://ec2-50-19-48-177.compute-1.amazonaws.com/broker/rest/user/ -d plan_id=megashift
<?xml version="1.0" encoding="UTF-8"?>
<response>
  <status>ok</status>
  <type>account</type>
  <data>
    <user>
      <login>qgong</login>
      <consumed-gears>3</consumed-gears>
      <capabilities>
        <subaccounts>false</subaccounts>
        <gear-sizes>
          <gear-size>small</gear-size>
          <gear-size>medium</gear-size>
        </gear-sizes>
        <max-storage-per-gear>30</max-storage-per-gear>
      </capabilities>
      <max-gears>16</max-gears>
      <plan-id>megashift</plan-id>
      <usage-account-id>1896035</usage-account-id>
      <consumed-gear-sizes>
        <small>3</small>
      </consumed-gear-sizes>
      <links>
        <link>
          <rel>Get SSH keys</rel>
          <method>GET</method>
          <href>https://ec2-50-19-48-177.compute-1.amazonaws.com/broker/rest/user/keys</href>
          <required-params/>
          <optional-params/>
        </link>
        <link>
          <rel>Add new SSH key</rel>
          <method>POST</method>
          <href>https://ec2-50-19-48-177.compute-1.amazonaws.com/broker/rest/user/keys</href>
          <required-params>
            <param>
              <name>name</name>
              <type>string</type>
              <description>Name of the key</description>
              <valid-options/>
              <invalid-options/>
            </param>
            <param>
              <name>type</name>
              <type>string</type>
              <description>Type of Key</description>
              <valid-options>
                <valid-option>ssh-rsa</valid-option>
                <valid-option>ssh-dss</valid-option>
                <valid-option>ecdsa-sha2-nistp256-cert-v01</valid-option>
                <valid-option>ecdsa-sha2-nistp384-cert-v01</valid-option>
                <valid-option>ecdsa-sha2-nistp521-cert-v01</valid-option>
                <valid-option>ssh-rsa-cert-v01</valid-option>
                <valid-option>ssh-dss-cert-v01</valid-option>
                <valid-option>ssh-rsa-cert-v00</valid-option>
                <valid-option>ssh-dss-cert-v00</valid-option>
                <valid-option>ecdsa-sha2-nistp256</valid-option>
                <valid-option>ecdsa-sha2-nistp384</valid-option>
                <valid-option>ecdsa-sha2-nistp521</valid-option>
              </valid-options>
              <invalid-options/>
            </param>
            <param>
              <name>content</name>
              <type>string</type>
              <description>The key portion of an rsa key (excluding ssh-rsa and comment)</description>
              <valid-options/>
              <invalid-options/>
            </param>
          </required-params>
          <optional-params/>
        </link>
      </links>
    </user>
  </data>
  <messages/>
  <version>1.3</version>
  <supported-api-versions>
    <supported-api-version>1.0</supported-api-version>
    <supported-api-version>1.1</supported-api-version>
    <supported-api-version>1.2</supported-api-version>
    <supported-api-version>1.3</supported-api-version>
  </supported-api-versions>
</response>


Note You need to log in before you can comment on or make changes to this bug.