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.

Sign up for GitHub

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

Jump to bottom

volume must be from type gp3 #738

Closed
1 task done
goneri opened this issue Mar 25, 2022 · 2 comments · Fixed by #739
Closed
1 task done

volume must be from type gp3 #738

goneri opened this issue Mar 25, 2022 · 2 comments · Fixed by #739
Labels
bug This issue/PR relates to a bug has_pr needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage python3 traceback

Comments

@goneri
Copy link
Member

goneri commented Mar 25, 2022

Summary

https://23a2ceaa2a64dc5cbcfb-fd6ba484f6b5338258f817240cdc03a4.ssl.cf2.rackcdn.com/734/ae8e9dfe6bf822d4ebbb611141d69d0389db6e86/check/integration-amazon.aws-4/b73e89c/job-output.txt

2022-03-25 19:04:25.455408 | controller | TASK [ec2_vol : change existing volume to gp3] *********************************
2022-03-25 19:04:25.455522 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/amazon/aws/tests/integration/targets/ec2_vol/tasks/main.yml:474
2022-03-25 19:04:25.485799 | controller | Using module file /home/zuul/.ansible/collections/ansible_collections/amazon/aws/plugins/modules/ec2_vol.py
2022-03-25 19:04:25.486095 | controller | Pipelining is enabled.
2022-03-25 19:04:25.486275 | controller | <testhost> ESTABLISH LOCAL CONNECTION FOR USER: zuul
2022-03-25 19:04:25.486382 | controller | <testhost> EXEC /bin/sh -c 'ANSIBLE_DEBUG_BOTOCORE_LOGS=True /home/zuul/venv/bin/python && sleep 0'
2022-03-25 19:04:26.606624 | controller | changed: [testhost] => {
2022-03-25 19:04:26.606672 | controller |     "changed": true,
2022-03-25 19:04:26.606681 | controller |     "device": null,
2022-03-25 19:04:26.606688 | controller |     "invocation": {
2022-03-25 19:04:26.606694 | controller |         "module_args": {
2022-03-25 19:04:26.606701 | controller |             "aws_access_key": "ASIA6CCDWXDOA4WRHYET",
2022-03-25 19:04:26.606708 | controller |             "aws_ca_bundle": null,
2022-03-25 19:04:26.606715 | controller |             "aws_config": null,
2022-03-25 19:04:26.606722 | controller |             "aws_secret_key": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
2022-03-25 19:04:26.606735 | controller |             "debug_botocore_endpoint_logs": true,
2022-03-25 19:04:26.606742 | controller |             "delete_on_termination": false,
2022-03-25 19:04:26.606749 | controller |             "device_name": null,
2022-03-25 19:04:26.606755 | controller |             "ec2_url": null,
2022-03-25 19:04:26.606762 | controller |             "encrypted": false,
2022-03-25 19:04:26.606768 | controller |             "id": "vol-062b46452da5e6c5c",
2022-03-25 19:04:26.606775 | controller |             "instance": null,
2022-03-25 19:04:26.606781 | controller |             "iops": null,
2022-03-25 19:04:26.606788 | controller |             "kms_key_id": null,
2022-03-25 19:04:26.606794 | controller |             "modify_volume": true,
2022-03-25 19:04:26.606800 | controller |             "multi_attach": null,
2022-03-25 19:04:26.606807 | controller |             "name": null,
2022-03-25 19:04:26.606813 | controller |             "outpost_arn": null,
2022-03-25 19:04:26.606819 | controller |             "profile": null,
2022-03-25 19:04:26.606826 | controller |             "purge_tags": false,
2022-03-25 19:04:26.606832 | controller |             "region": "us-east-1",
2022-03-25 19:04:26.606839 | controller |             "security_token": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
2022-03-25 19:04:26.606845 | controller |             "snapshot": null,
2022-03-25 19:04:26.606851 | controller |             "state": "present",
2022-03-25 19:04:26.606863 | controller |             "tags": {},
2022-03-25 19:04:26.606871 | controller |             "throughput": null,
2022-03-25 19:04:26.606877 | controller |             "validate_certs": true,
2022-03-25 19:04:26.606884 | controller |             "volume_size": null,
2022-03-25 19:04:26.606890 | controller |             "volume_type": "gp3",
2022-03-25 19:04:26.606897 | controller |             "zone": "us-east-1a"
2022-03-25 19:04:26.606903 | controller |         }
2022-03-25 19:04:26.606910 | controller |     },
2022-03-25 19:04:26.606916 | controller |     "resource_actions": [
2022-03-25 19:04:26.606926 | controller |         "ec2:DescribeTags",
2022-03-25 19:04:26.606932 | controller |         "ec2:ModifyVolume",
2022-03-25 19:04:26.606939 | controller |         "ec2:DescribeVolumes"
2022-03-25 19:04:26.606945 | controller |     ],
2022-03-25 19:04:26.607022 | controller |     "volume": {
2022-03-25 19:04:26.607029 | controller |         "attachment_set": [
2022-03-25 19:04:26.607035 | controller |             {
2022-03-25 19:04:26.607042 | controller |                 "attach_time": "2022-03-25T19:03:57+00:00",
2022-03-25 19:04:26.607048 | controller |                 "delete_on_termination": false,
2022-03-25 19:04:26.607055 | controller |                 "device": "/dev/sdh",
2022-03-25 19:04:26.607062 | controller |                 "instance_id": "i-0f2ff72c1b3a2c8fb",
2022-03-25 19:04:26.607068 | controller |                 "status": "attached"
2022-03-25 19:04:26.607106 | controller |             }
2022-03-25 19:04:26.607113 | controller |         ],
2022-03-25 19:04:26.607120 | controller |         "create_time": "2022-03-25T19:03:41.905000+00:00",
2022-03-25 19:04:26.607126 | controller |         "encrypted": false,
2022-03-25 19:04:26.607133 | controller |         "id": "vol-062b46452da5e6c5c",
2022-03-25 19:04:26.607139 | controller |         "iops": 3000,
2022-03-25 19:04:26.607145 | controller |         "multi_attach_enabled": false,
2022-03-25 19:04:26.607152 | controller |         "size": 1,
2022-03-25 19:04:26.607158 | controller |         "snapshot_id": "",
2022-03-25 19:04:26.607164 | controller |         "status": "in-use",
2022-03-25 19:04:26.607171 | controller |         "tags": {
2022-03-25 19:04:26.607178 | controller |             "CamelCase": "SimpleCamelCase ��",
2022-03-25 19:04:26.607187 | controller |             "Name": "ansible-test-94852683-node-0002287877 - sdh",
2022-03-25 19:04:26.607200 | controller |             "ResourcePrefix": "ansible-test-94852683-node-0002287877",
2022-03-25 19:04:26.607207 | controller |             "Title Case": "Hello Cruel World ��",
2022-03-25 19:04:26.607212 | controller |             "lowercase spaced": "hello cruel world ��",
2022-03-25 19:04:26.607217 | controller |             "snake_case": "simple_snake_case ��"
2022-03-25 19:04:26.607222 | controller |         },
2022-03-25 19:04:26.607226 | controller |         "throughput": 125,
2022-03-25 19:04:26.607231 | controller |         "type": "gp3",
2022-03-25 19:04:26.607235 | controller |         "zone": "us-east-1a"
2022-03-25 19:04:26.607240 | controller |     },
2022-03-25 19:04:26.607245 | controller |     "volume_id": "vol-062b46452da5e6c5c",
2022-03-25 19:04:26.607249 | controller |     "volume_type": "gp3"
2022-03-25 19:04:26.607254 | controller | }
2022-03-25 19:04:26.615851 | controller |
2022-03-25 19:04:26.615888 | controller | TASK [ec2_vol : check that volume_type has changed] ****************************
2022-03-25 19:04:26.616046 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/amazon/aws/tests/integration/targets/ec2_vol/tasks/main.yml:482
2022-03-25 19:04:26.680595 | controller | ok: [testhost] => {
2022-03-25 19:04:26.680635 | controller |     "changed": false,
2022-03-25 19:04:26.680640 | controller |     "msg": "All assertions passed"
2022-03-25 19:04:26.680646 | controller | }
2022-03-25 19:04:26.687444 | controller |
2022-03-25 19:04:26.687479 | controller | TASK [ec2_vol : volume must be from type gp3 (idempotent)] *********************
2022-03-25 19:04:26.687592 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/amazon/aws/tests/integration/targets/ec2_vol/tasks/main.yml:506
2022-03-25 19:04:26.723465 | controller | Using module file /home/zuul/.ansible/collections/ansible_collections/amazon/aws/plugins/modules/ec2_vol.py
2022-03-25 19:04:26.723769 | controller | Pipelining is enabled.
2022-03-25 19:04:26.723959 | controller | <testhost> ESTABLISH LOCAL CONNECTION FOR USER: zuul
2022-03-25 19:04:26.724092 | controller | <testhost> EXEC /bin/sh -c 'ANSIBLE_DEBUG_BOTOCORE_LOGS=True /home/zuul/venv/bin/python && sleep 0'
2022-03-25 19:04:27.552448 | controller | FAILED - RETRYING: [testhost]: volume must be from type gp3 (idempotent) (3 retries left).Result was: {
2022-03-25 19:04:27.552487 | controller |     "attempts": 1,
2022-03-25 19:04:27.552492 | controller |     "changed": false,
2022-03-25 19:04:27.552498 | controller |     "module_stderr": "Traceback (most recent call last):\n  File \"<stdin>\", line 121, in <module>\n  File \"<stdin>\", line 113, in _ansiballz_main\n  File \"<stdin>\", line 61, in invoke_module\n  File \"/usr/lib64/python3.8/runpy.py\", line 207, in run_module\n    return _run_module_code(code, init_globals, run_name, mod_spec)\n  File \"/usr/lib64/python3.8/runpy.py\", line 97, in _run_module_code\n    _run_code(code, mod_globals, init_globals,\n  File \"/usr/lib64/python3.8/runpy.py\", line 87, in _run_code\n    exec(code, run_globals)\n  File \"/tmp/ansible_ec2_vol_payload_13puujmv/ansible_ec2_vol_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vol.py\", line 887, in <module>\n  File \"/tmp/ansible_ec2_vol_payload_13puujmv/ansible_ec2_vol_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vol.py\", line 853, in main\n  File \"/tmp/ansible_ec2_vol_payload_13puujmv/ansible_ec2_vol_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_vol.py\", line 446, in update_volume\n  File \"/tmp/ansible_ec2_vol_payload_13puujmv/ansible_ec2_vol_payload.zip/ansible_collections/amazon/aws/plugins/module_utils/modules.py\", line 330, in deciding_wrapper\n  File \"/home/zuul/venv/lib/python3.8/site-packages/botocore/client.py\", line 357, in _api_call\n    return self._make_api_call(operation_name, kwargs)\n  File \"/home/zuul/venv/lib/python3.8/site-packages/botocore/client.py\", line 676, in _make_api_call\n    raise error_class(parsed_response, operation_name)\nbotocore.exceptions.ClientError: An error occurred (IncorrectModificationState) when calling the ModifyVolume operation: Volume vol-062b46452da5e6c5c cannot be modified in modification state MODIFYING\n",
2022-03-25 19:04:27.552503 | controller |     "module_stdout": "",
2022-03-25 19:04:27.552506 | controller |     "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error",
2022-03-25 19:04:27.552510 | controller |     "rc": 1,
2022-03-25 19:04:27.552513 | controller |     "retries": 4
2022-03-25 19:04:27.552516 | controller | }

Issue Type

CI Bug Report

CI Jobs

https://23a2ceaa2a64dc5cbcfb-fd6ba484f6b5338258f817240cdc03a4.ssl.cf2.rackcdn.com/734/ae8e9dfe6bf822d4ebbb611141d69d0389db6e86/check/integration-amazon.aws-4/b73e89c/

Pull Request

#734

Additional Information



Code of Conduct


    

  •  I agree to follow the Ansible Code of Conduct
    • 


      		
    

      
  





        



            

              
            

        

      


      
    








      

    



      

  
          


  
    
  
  

    
  goneri 

      added a commit
        to goneri/amazon.aws
      that referenced
      this issue

    
      Mar 25, 2022
    
    

      


  

    

      


  

      
        @goneri
  





      

        
          ec2_vol/test: gives more time to convert vol to gp3
        

          
            
          

      


      

          

    
    
    

  

      


      

      


      
      

        
          ac82b85
        
      

    

  

    

      
We give 9s to convert the volumne to gp3, in some cases this is not
enough.
Closes: ansible-collections#738

    




    

  




  

  

    
  


  

      

    @goneri
goneri



    mentioned this issue
    
      Mar 25, 2022
    





  


  

      
   Merged

  







  









      

  
      



  

  @ansibullbot





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      



      

  


  

    

  





      


        


  
    

      
          
Files identified in the description:

None


If these files are inaccurate, please update the component name section of the description or use the !component bot command.


click here for bot help


      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
      



  

  @ansibullbot





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      



      

  


  

    

  





      


        


  
    

      
          
@goneri: Greetings! Thanks for taking the time to open this issue. In order for the community to handle your issue effectively, we need a bit more information.


Here are the items we could not find in your description:


    

  • ansible version
    • 

  • component name
    • 



Please set the description of this issue with this template:

https://raw.githubusercontent.com/ansible/ansible/devel/.github/ISSUE_TEMPLATE/bug_report.md


click here for bot help


      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
            

    

      
    

    


      

          @ansibullbot
ansibullbot




            added
      

  bug

  This issue/PR relates to a bug

      

  has_pr


      

  needs_info

  This issue requires further information. Please answer any outstanding questions

      

  needs_template

  This issue/PR has an incomplete description. Please fill in the proposed template correctly

      

  needs_triage


      

  python3


      

  traceback


  labels


      Mar 25, 2022

    

  



    

    


  


  
    
  
  

    
  softwarefactory-project-zuul bot

      pushed a commit
      that referenced
      this issue

    
      Mar 28, 2022
    
    

      


  

    

      


  

      
        @goneri
  





      

        
          ec2_vol/test: gives more time to convert vol to gp3 (#739)
        

          
            
          

      


      

          

    
    
    

  

    


      


      

      


      
      

        
          df865b4
        
      

    

  

    

      
ec2_vol/test: gives more time to convert vol to gp3

We give 9s to convert the volumne to gp3, in some cases this is not
enough.
Closes: #738

Reviewed-by: Alina Buzachis <None>

    




    

  




  


  
    
  
  

    
  patchback bot

      pushed a commit
      that referenced
      this issue

    
      Mar 28, 2022
    
    

      


  

    

      


  

      
        @goneri
      
        @patchback
  





      

        
          ec2_vol/test: gives more time to convert vol to gp3 (#739)
        

          
            
          

      


      

          

    
    
    

  

      


      

      


      
      

        
          5708849
        
      

    

  

    

      
ec2_vol/test: gives more time to convert vol to gp3

We give 9s to convert the volumne to gp3, in some cases this is not
enough.
Closes: #738

Reviewed-by: Alina Buzachis <None>
(cherry picked from commit df865b4)

    




    

  




  

  

    
  


  

      

    @patchback
patchback
bot


    mentioned this issue
    
      Mar 28, 2022
    





  


  

      
   Merged

  







  




  


  
    
  
  

    
  patchback bot

      pushed a commit
      that referenced
      this issue

    
      Mar 28, 2022
    
    

      


  

    

      


  

      
        @goneri
      
        @patchback
  





      

        
          ec2_vol/test: gives more time to convert vol to gp3 (#739)
        

          
            
          

      


      

          

    
    
    

  

      


      

      


      
      

        
          e398d2d
        
      

    

  

    

      
ec2_vol/test: gives more time to convert vol to gp3

We give 9s to convert the volumne to gp3, in some cases this is not
enough.
Closes: #738

Reviewed-by: Alina Buzachis <None>
(cherry picked from commit df865b4)

    




    

  




  

  

    
  


  

      

    @patchback
patchback
bot


    mentioned this issue
    
      Mar 28, 2022
    





  


  

      
   Merged

  







  




  


  
    
  
  

    
  softwarefactory-project-zuul bot

      pushed a commit
      that referenced
      this issue

    
      Mar 28, 2022
    
    

      


  

    

      


  

      
        @patchback
  





      

        
          ec2_vol/test: gives more time to convert vol to gp3 (#739) (#744)
        

          
            
          

      


      

          

    
    
    

  

    


      


      

      


      
      

        
          ed09bd5
        
      

    

  

    

      
[PR #739/df865b4c backport][stable-3] ec2_vol/test: gives more time to convert vol to gp3

This is a backport of PR #739 as merged into main (df865b4).
We give 9s to convert the volumne to gp3, in some cases this is not
enough.
Closes: #738

    




    

  




  


  
    
  
  

    
  softwarefactory-project-zuul bot

      pushed a commit
      that referenced
      this issue

    
      Apr 12, 2022
    
    

      


  

    

      


  

      
        @patchback
  





      

        
          ec2_vol/test: gives more time to convert vol to gp3 (#739) (#742)
        

          
            
          

      


      

          

    
    
    

  

    


      


      

      


      
      

        
          006e8ab
        
      

    

  

    

      
[PR #739/df865b4c backport][stable-2] ec2_vol/test: gives more time to convert vol to gp3

This is a backport of PR #739 as merged into main (df865b4).
We give 9s to convert the volumne to gp3, in some cases this is not
enough.
Closes: #738

Reviewed-by: Mark Chappell <None>

    




    

  




  


  
    
  
  

    
  abikouo 

      pushed a commit
        to abikouo/amazon.aws
      that referenced
      this issue

    
      Oct 24, 2023
    
    

      


  

    

      


  

      
        @ansible-zuul
  





      

        
          Merge pull request ansible-collections#738 from Andersson007/test_1
        

          
            
          

      


      

          

    
    
    

  

    


      


      

      


      
      

        
          3fbca8f
        
      

    

  

    

      
Copy ignore-2.12.txt to ignore-2.13.txt

SUMMARY
Relates to ansible-collections/overview#45 (comment)

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>

    




    

  













  
  

    

      

  





  




  
  

              

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


  



  






  
                  




    

  


      
  

    Assignees
  



        

    No one assigned







      


  


  

    Labels
  



    

      

    bug

  This issue/PR relates to a bug
  

    has_pr

  

    needs_info

  This issue requires further information. Please answer any outstanding questions
  

    needs_template

  This issue/PR has an incomplete description. Please fill in the proposed template correctly
  

    needs_triage

  

    python3

  

    traceback









      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          
  

    

      

        
          

            
    

    Development
  




              
  
Successfully merging a pull request may close this issue.





    


  
          
      ec2_vol/test: gives more time to convert vol to gp3

    
      goneri/amazon.aws
    





        
      

    

  


      

    

  
      

  

    

      2 participants
    

    

        
          @goneri 
        
          @ansibullbot