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

Ansible-2.9.7 Fails to Deliver NETCONF Packets #69065

Closed
yanzhangi opened this issue Apr 21, 2020 · 7 comments
Closed

Ansible-2.9.7 Fails to Deliver NETCONF Packets #69065

yanzhangi opened this issue Apr 21, 2020 · 7 comments
Assignees
@Qalthos @danielmellado @rohitthakur2590
Labels
affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. collection:community.network collection Related to Ansible Collections work has_pr This issue has an associated PR. module This issue/PR relates to a module. needs_collection_redirect https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md networking Network category support:community This issue/PR relates to code supported by the Ansible community.

Comments

@yanzhangi
Copy link
Contributor

yanzhangi commented Apr 21, 2020 •
edited
Loading

SUMMARY

When the NETCONF packet is delivered, "Unable to decode JSON from response to get() Received'None'" is displayed.
However, this problem does not occur when the Ansible version is 2.9.6.

ISSUE TYPE
  • Bug Report
COMPONENT NAME

https://github.com/ansible/ansible/tree/v2.9.7/lib/ansible/modules/network/cloudengine/ce_config.py

ANSIBLE VERSION

ansible-2.9.7

CONFIGURATION
OS / ENVIRONMENT
STEPS TO REPRODUCE



EXPECTED RESULTS



ACTUAL RESULTS







      		
    

      
  





        

            

            

              
            

        

      


      
    








      

    



      

  
          

  

    
  

  


    

        @sivel
sivel




          added
      

  support:network

  This issue/PR relates to code supported by the Ansible Network Team.

      

  needs_triage

  Needs a first human triage before being processed.

  labels


    Apr 21, 2020

  





  

  

    
  

  


    

        @rohitthakur2590
rohitthakur2590




        removed
  the 

  needs_triage

  Needs a first human triage before being processed.
 label


    Apr 22, 2020

  





  







      

  
      



  

  @ansibot





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      

  
    Contributor


      

  


  

    

      

      
          ansibot
  

      

      

      commented


        Apr 24, 2020


      
    


  





      


        


  
    

      
          
Files identified in the description:



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


click here for bot help


      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
          

  

    
  

  


    

        @ansibot
ansibot




          added
      

  affects_2.9

  This issue/PR affects Ansible v2.9

      

  bug

  This issue/PR relates to a bug.

      

  collection

  Related to Ansible Collections work

      

  collection:community.network


      

  module

  This issue/PR relates to a module.

      

  needs_collection_redirect

  https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md

      

  networking

  Network category

      

  support:community

  This issue/PR relates to code supported by the Ansible community.

    and removed
      

  support:network

  This issue/PR relates to code supported by the Ansible Network Team.

  labels


    Apr 24, 2020

  










      

  
      



  

  @nickrusso42518





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      



      

  


  

    

  





      


        


  
    

      
          
This also occurs in v2.9.7 when using the netconf_config module to interact with Cisco IOS-XE devices. This is true both for the ansible.netcommon collection and the "base" Ansible installation.  Downgrading to v2.9.6 fixes the problem.

      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
      



  

  @yanzhangi





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      

  
    Contributor


      

  Author


  


  

    

      

      
          yanzhangi
  

      

      

      commented


        Apr 26, 2020


      
  •

  

    
      

        
          edited
          
        
        
      

    
    
      
          Loading

      
    
  



    


  





      


        


  
    

      
          
Hello, Russo, do you know why this problem arises?  And how do we solve this problem?

      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
      



  

  @danielmellado





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      

  
    Contributor


      

  


  

    

  





      


        


  
    

      
          
This also happens on devel, we'll be reverting both commits on devel and 2.9.7 for the time being until a complete fix is shipped. Also this skipped our radar due to the testing changing after migrating to collections. Thanks for pointing this out!

      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
          


  
    
  
  

    
  danielmellado 

      added a commit
        to danielmellado/ansible
      that referenced
      this issue

    
      Apr 27, 2020
    
    

      


  

    

      


  

      
        @danielmellado
  





      

        
          Revert "Fix missing persistent connection messages (ansible#68496)" (a…
        

          
            
          

      


      

          

    
    
    

  

    


      


      

      


      
      

        
          8c2509a
        
      

    

  

    

      
…nsible#69147)

This reverts commit 5f6427b.
as it breaks netconf connection. This will be a temporary measure
for unlocking CI until a proper fix is shipped.

Fixes ansible#69065

(cherry picked from commit 9217aee)
Signed-off-by: Daniel Mellado <dmellado@redhat.com>

    




    

  




  

  

    
  


  

      

    @danielmellado
danielmellado



    mentioned this issue
    
      Apr 27, 2020
    





  


  

      
   Merged

  







  




  


  
    
  
  

    
  danielmellado 

      added a commit
        to danielmellado/ansible
      that referenced
      this issue

    
      Apr 27, 2020
    
    

      


  

    

      


  

      
        @danielmellado
  





      

        
          Revert "Fix missing persistent connection messages (ansible#68496)" (a…
        

          
            
          

      


      

          

    
    
    

  

    


      


      

      


      
      

        
          15524db
        
      

    

  

    

      
…nsible#69147)

This reverts commit 5f6427b.
as it breaks netconf connection. This will be a temporary measure
for unlocking CI until a proper fix is shipped.

Fixes ansible#69065

(cherry picked from commit 9217aee)
Signed-off-by: Daniel Mellado <dmellado@redhat.com>

    




    

  




  


  
    
  
  

    
  mattclay 

      pushed a commit
      that referenced
      this issue

    
      Apr 27, 2020
    
    

      


  

    

      


  

      
        @danielmellado
      
        @mattclay
  





      

        
          Revert "Fix missing persistent connection messages (#68496)" (#69147)
        

          
            
          

      


      

          

    
    
    

  

      


      

      


      
      

        
          a4f13ac
        
      

    

  

    

      
This reverts commit 5f6427b.
as it breaks netconf connection. This will be a temporary measure
for unlocking CI until a proper fix is shipped.

Fixes #69065

(cherry picked from commit 9217aee)
Signed-off-by: Daniel Mellado <dmellado@redhat.com>

    




    

  









      

  
      



  

  @yanzhangi





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      

  
    Contributor


      

  Author


  


  

    

  





      


        


  
    

      
          
@danielmellado  Well, thank you very much. I'll keep track of that.

      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
          

  

    
  

  


    

        @ansibot
ansibot




        added
  the 

  has_pr

  This issue has an associated PR.
 label


    Apr 28, 2020

  










      

  
      



  

  @nickrusso42518





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      



      

  


  

    

  





      


        


  
    

      
          


Hello, Russo, do you know why this problem arises? And how do we solve this problem?




I just downgraded to v2.9.6 in the interest of time, and gave a +1 on this issue to signal that I saw it too.

      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
      



  

  @danielmellado





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

      

  
    Contributor


      

  


  

    

  





      


        


  
    

      
          
The fix for this issue is merged and available on Ansible 2.9.9

      		
    

  





        


            

              
            

        

      


      

            
  

    
  
  

  

  



    












      

  
          

  


  

  

    
  

  


    

        @ansible
ansible




      locked and limited conversation to collaborators


    Jun 10, 2020

  














  
  

    

      

  





  




  
  

             

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


  


  






  
                  




    

  


      
  

    Assignees
  



        

      

        

  
    @Qalthos
  
    Qalthos
    

      

      

        

  
    @danielmellado
  
    danielmellado
    

      

      

        

  
    @rohitthakur2590
  
    rohitthakur2590
    

      








      


  


  

    Labels
  



    

      

    affects_2.9

  This issue/PR affects Ansible v2.9
  

    bug

  This issue/PR relates to a bug.
  

    collection:community.network

  

    collection

  Related to Ansible Collections work
  

    has_pr

  This issue has an associated PR.
  

    module

  This issue/PR relates to a module.
  

    needs_collection_redirect

  https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md
  

    networking

  Network category
  

    support:community

  This issue/PR relates to code supported by the Ansible community.








      

  

    

        

    Projects
  


        




    None yet




  



      

  

    
  

    Milestone
  


      No milestone





    
      
          


  

    
      

        
    

    Development
  




              
No branches or pull requests







    
  




        
      

    

  
      

  

    

      7 participants
    

    

        
          @sivel 
        
          @Qalthos 
        
          @danielmellado 
        
          @ansibot 
        
          @nickrusso42518 
        
          @rohitthakur2590 
        
          @yanzhangi