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

[Bug]: not enable limiter in restful v1 #39556

Open
1 task done
smellthemoon opened this issue Jan 23, 2025 · 1 comment
Open
1 task done

[Bug]: not enable limiter in restful v1 #39556

smellthemoon opened this issue Jan 23, 2025 · 1 comment
Assignees
@smellthemoon
Labels
kind/bug Issues or changes related a bug triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone
2.5.5

Comments

@smellthemoon
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Environment

- Milvus version:
- Deployment mode(standalone or cluster):
- MQ type(rocksmq, pulsar or kafka):    
- SDK version(e.g. pymilvus v2.0.0rc2):
- OS(Ubuntu or CentOS): 
- CPU/Memory: 
- GPU: 
- Others:

Current Behavior

not enable limiter in restful v1

Expected Behavior

No response

Steps To Reproduce



Milvus Log


No response


Anything else?


No response

      		
    

      
  





        



            

              
            

        

      


      
    








      

    



      

  
            

    

      
    

    


      

          @smellthemoon
smellthemoon




            added
      

  kind/bug

  Issues or changes related a bug

      

  needs-triage

  Indicates an issue or PR lacks a `triage/foo` label and requires one.

  labels


      Jan 23, 2025

    

  



    


  

  

    
  


  

      

  This was referenced Jan 23, 2025





  


  

      
   Merged

  






      






  


  

      
   Merged

  






      






  


  

      
   Merged

  






  









      

  
      



  

  @smellthemoon





  


    

      

  

    

        
    
    

  


      
          
  
      
            Copy link

  

      
    

  


  

      

  
    Contributor


      

  Author


  


  

    

  





      


        


  
    

      
          
/assign

      		
    

  





        


            

            

              
            

        

      


      

            
  

    
  
    
    

  

  



    












      

  
            


    

    

      
    

    


      

          @yanliang567
yanliang567




          added this to the   2.5.5 milestone


      Jan 24, 2025

    

  



    

    

      
    

    


      

          @yanliang567
yanliang567




            added
      

  triage/accepted

  Indicates an issue or PR is ready to be actively worked on.

    and removed
      

  needs-triage

  Indicates an issue or PR lacks a `triage/foo` label and requires one.

  labels


      Jan 24, 2025

    

  



    

    

      
    

    


      

          @yanliang567
yanliang567




              removed their assignment


      Jan 24, 2025

    

  



  


  
    
  
  

    
  sre-ci-robot 

      pushed a commit
      that referenced
      this issue

    
      Jan 24, 2025
    
    

      


  

    

      


  

      
        @smellthemoon
  





      

        
          fix: [2.4]not enable rate limiter for restful v1 (#39554)
        

          
            
          

      


      

          

    
    
    

  

    


      


      

          

    
    
    

  

  

    
  



      


      
      

        
          6eadcac
        
      

    

  

    

      
issue: #39556 
pr: #39553 
2.5: #39555

Signed-off-by: lixinguo <[email protected]>
Co-authored-by: lixinguo <[email protected]>

    




    

  




  


  
    
  
  

    
  sre-ci-robot 

      pushed a commit
      that referenced
      this issue

    
      Jan 24, 2025
    
    

      


  

    

      


  

      
        @smellthemoon
  





      

        
          fix: not enable rate limiter for restful v1 (#39553)
        

          
            
          

      


      

          

    
    
    

  

    


      


      

          

    
    
    

  

  

    
  



      


      
      

        
          47d280d
        
      

    

  

    

      
issue: #39556

Signed-off-by: lixinguo <[email protected]>
Co-authored-by: lixinguo <[email protected]>

    




    

  




  


  
    
  
  

    
  sre-ci-robot 

      pushed a commit
      that referenced
      this issue

    
      Jan 24, 2025
    
    

      


  

    

      


  

      
        @smellthemoon
  





      

        
          fix: [2.5]not enable rate limiter for restful v1 (#39555)
        

          
            
          

      


      

          

    
    
    

  

    


      


      

          

    
    
    

  

  

    
  



      


      
      

        
          c832793
        
      

    

  

    

      
issue: #39556 
pr: #39553

Signed-off-by: lixinguo <[email protected]>
Co-authored-by: lixinguo <[email protected]>

    




    

  













  
  

    

      

  





  




  
  

              

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


  



  






  
                  




    

  


      
  

    Assignees
  



        

      

        

  
      @smellthemoon
  
    smellthemoon
    

      








      


  


  

    Labels
  



    

      

    kind/bug

  Issues or changes related a bug
  

    triage/accepted

  Indicates an issue or PR is ready to be actively worked on.








      

  

    

        

    Projects
  


        




    None yet




  



      


  

    
  

    Milestone
  


      
    
  
    2.5.5
  





    
      
          


  

    
      

        
    

    Development
  




          
    
No branches or pull requests







    
  




      

    

  
      

  

    

      2 participants
    

    

        
          @smellthemoon 
        
          @yanliang567