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

Add PlamenDoychev as ARP approver #793

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Conversation

PlamenDoychev
Copy link
Contributor

@PlamenDoychev PlamenDoychev commented Mar 9, 2024

  1. Reviewer
    [bbs] Use scheduling info instead of the whole desiredLRP  bbs#79
    [bbs] Add routing info endpoint bbs#66
    [bbs] Remove cpu_weight limit bbs#81
    [executor] Improve error handling on process start executor#91
    [locket] Add a keepalive timeout on the locket client  locket#12
    [route-emmiter] Use routing info bbs endpoint when syncing route-emitter#23
    [route-emmiter] Use routing_info for desired_lrp's when there are missing actual_lrp's route-emitter#26
    [route-emmiter] Use routing_info for desired_lrp's when there are missing actual_lrp's route-emitter#26
    [silk-release] Deduplicate Iptables Rules with Dynamic ASG's  silk-release#101
    [bbs] Add routing info endpoint bbs#66
    [route-emmiter] Use routing info bbs endpoint when syncing route-emitter#23
    [bbs] Remove cpu_weight limit bbs#81
  2. Co-Author
    [diego-release] [BBS PR REVIEW]: PoC for Security group filtering diego-release#678
    [bbs] [PoC] Security Group Filtering bbs#53
    [route-emmiter] PoC for Security Group Filtering route-emitter#18
    [bbs] Add routing info endpoint bbs#66
    [route-emmiter] Use routing info bbs endpoint when syncing route-emitter#23
    [bbs] [BBS] Use scheduling info instead of the whole desiredLRP  diego-release#883
    [executor] [Executor] Send error to app logs if starting a container fails  diego-release#892
    [diego-release] Calculate the CPUWeight directly in the Executor  diego-release#865
  3. Problems discovered and support provided
    [silk-release] actual_lrps internal_routes field does not get re-encrypted when rotating bbs_encryption_key diego-release#626
  • Active support and feedback provided during implementation and integration of the Dynamic ASGs
  • Active support and feedback provided during investigation and implementation of the IPTABLES rules de-duplication
  1. Active topic driver
  1. Community effort
    • Actively involved in community activities like discussions, RFC reviews&feedback, WG meetings etc.
  2. PM Role
    • Currently acting as a PO for the SAP CF-Runtime as such full access to community resources would be nice.

@beyhan beyhan added the wg label Mar 12, 2024
@beyhan
Copy link
Member

beyhan commented Mar 12, 2024

As stated in the Promotion to Reviewer or Approver two approvers of the Diego area needs to attest that @PlamenDoychev meets the criteria by reviewing this PR.

Copy link

@vlast3k vlast3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@ameowlia
Copy link
Member

🎉 Congrats!

@ameowlia ameowlia merged commit d06b4cd into cloudfoundry:main Mar 12, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants