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

Update/module version #65

Merged
merged 5 commits into from
Nov 7, 2024
Merged

Update/module version #65

merged 5 commits into from
Nov 7, 2024

Conversation

vjdbj
Copy link
Member

@vjdbj vjdbj commented Oct 14, 2024

what

  • Added subscription_id for provider in root module
  • Also updated the modules release version in examples.

why

  • subscription_id is compulsory to add for terraform planning

@clouddrove-ci clouddrove-ci self-assigned this Oct 14, 2024
@clouddrove-ci
Copy link
Member

Terraform Security Scan Failed

Show Output 



      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
          

  
  

  
      @vjdbj
  vjdbj


          requested review from
    13archit and 
    d4kverma



    October 14, 2024 18:25    
    









      

  
        





      

  
      



  

  @clouddrove-ci





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
  Member


      

  


  

    

  





      


        


  
    

      
          
Terraform Security Scan Failed


Show Output




      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
        

  

      

  13archit
  

  
          

            
                  13archit
  

            

            reviewed


            
                
                  Oct 15, 2024
                
            
            
          


          



              
    View reviewed changes
  


          






      

  
  
  
            
  
      

            

          
                versions.tf

              
                Outdated
          
            Show resolved
            Hide resolved
        


    
          
            
  
    
    

          



  








  








      

  
        





      

  
      



  

  @clouddrove-ci





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
  Member


      

  


  

    

  





      


        


  
    

      
          
Terraform Security Scan Failed


Show Output




      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
        





      

  
      



  

  @clouddrove-ci





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
  Member


      

  


  

    

  





      


        


  
    

      
          
Terraform Security Scan Failed


Show Output




      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
          

  

    
  

  


        
        @d4kverma
  d4kverma



        merged commit 667c86a
      into

      
  
    master
  


    Nov 7, 2024

      

        19 checks passed
      


      

        
          
  
    
    

        
      

  
















  
  

    

      

  




    


    

      

              

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


  



      

    

  




  
          




      

  

    
    

    
  

    Reviewers
  


    


        

          

  
      @13archit
  
    13archit



          
            
              
            
          
          13archit left review comments

        

      

          

  
      @d4kverma
  
    d4kverma


                Awaiting requested review from d4kverma


        
      


      






    

  


      
  

    Assignees
  



        

      

        

  
      @clouddrove-ci
  
    clouddrove-ci
    

      








      


  


  

    Labels
  



    

    None yet







      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      No milestone





    
      
          


  

    
      

        
    

    Development
  



          


Successfully merging this pull request may close these issues.



  


    
  




      

    

  
      

  

    

      4 participants
    

    

        
          @vjdbj 
        
          @clouddrove-ci 
        
          @13archit 
        
          @d4kverma