Navigation Menu

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

stack trace from linode driver #26798

Closed
jfindlay opened this issue Aug 31, 2015 · 2 comments
Closed

stack trace from linode driver #26798

jfindlay opened this issue Aug 31, 2015 · 2 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix help-wanted Community help is needed to resolve this P4 Priority 4 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. Salt-Cloud severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@jfindlay
Copy link
Contributor

versions

# salt --versions
Salt Version:
           Salt: 2015.8.0rc3-164-gae9d39c

Dependency Versions:
         Jinja2: 2.7.3
       M2Crypto: 0.21.1
           Mako: 1.0.0
         PyYAML: 3.11
          PyZMQ: 14.4.0
         Python: 2.7.9 (default, Mar  1 2015, 12:57:24)
           RAET: Not Installed
        Tornado: 4.2.1
            ZMQ: 4.0.5
           cffi: 0.8.6
       cherrypy: Not Installed
       dateutil: Not Installed
          gitdb: 0.5.4
      gitpython: Not Installed
          ioflo: Not Installed
        libnacl: Not Installed
   msgpack-pure: Not Installed
 msgpack-python: 0.4.2
   mysql-python: Not Installed
      pycparser: 2.10
       pycrypto: 2.6.1
         pygit2: Not Installed
   python-gnupg: Not Installed
          smmap: 0.8.2
        timelib: Not Installed

System Versions:
           dist: debian 8.1
        machine: x86_64
        release: 3.16.0-4-amd64
         system: debian 8.1

commands

# salt-cloud -l debug --script-args="-D" -p linode-ubuntu-14 jmoney-ubuntu-14

logs

[INFO    ] Status for Linode 1279023 is 'Being Created', waiting for 'Brand New'.
[DEBUG   ] file_roots is /root/salt-pkg-tests
[DEBUG   ] file_roots is /root/precipitate
[DEBUG   ] Requesting URL https://api.linode.com/?api_key=XXXXXXXXXX using GET method
[DEBUG   ] Response Status Code: 200
[DEBUG   ] Linode Response Status Code: 200
[ERROR   ] There was a profile error: object of type 'int' has no len()
Traceback (most recent call last):
  File "/root/salt/salt/cloud/cli.py", line 275, in run
    self.config.get('names')
  File "/root/salt/salt/cloud/__init__.py", line 1410, in run_profile
    ret[name] = self.create(vm_)
  File "/root/salt/salt/cloud/__init__.py", line 1256, in create
    output = self.clouds[func](vm_)
  File "/root/salt/salt/cloud/clouds/linode.py", line 436, in create
    update_linode(node_id, update_args={'Label': vm_['name']})
  File "/root/salt/salt/cloud/clouds/linode.py", line 1280, in update_linode
    if _validate_name(linode_id) is False:
  File "/root/salt/salt/cloud/clouds/linode.py", line 1523, in _validate_name
    name_length = len(name)
TypeError: object of type 'int' has no len()
@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around help-wanted Community help is needed to resolve this Salt-Cloud P4 Priority 4 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. labels Aug 31, 2015
@jfindlay jfindlay added this to the Approved milestone Aug 31, 2015
@rallytime rallytime self-assigned this Aug 31, 2015
@rallytime rallytime modified the milestones: Be -2, Approved Aug 31, 2015
@jfindlay
Copy link
Contributor Author

jfindlay commented Sep 1, 2015

@rallytime, I have a fix for this if you want me to submit it. I have tested it and it works.

jfindlay added a commit to jfindlay/salt that referenced this issue Sep 1, 2015
Fixes saltstack#26798.  At the start of the provisioning process, linode reports
VM names by their linode ID, which is represented by the driver as an
integer.
@rallytime
Copy link
Contributor

@jfindlay Can you give #26819 a try?

@rallytime rallytime added the fixed-pls-verify fix is linked, bug author to confirm fix label Sep 1, 2015
@jfindlay jfindlay closed this as completed Sep 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix help-wanted Community help is needed to resolve this P4 Priority 4 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. Salt-Cloud severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

No branches or pull requests

2 participants