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

Fix backend update racing problem with dpvs-agent and healthcheck. #937

Merged
merged 20 commits into from
Apr 10, 2024

Commits on Jan 23, 2024

  1. API: /${SVCID}/rs/health return the specified service detail when fet…

    …ched wrong vs-version from client
    you-looks-not-tasty authored and ywc689 committed Jan 23, 2024
    Configuration menu
    Copy the full SHA
    9648beb View commit details
    Browse the repository at this point in the history
  2. lock snapshot

    you-looks-not-tasty authored and ywc689 committed Jan 23, 2024
    Configuration menu
    Copy the full SHA
    f9ab755 View commit details
    Browse the repository at this point in the history
  3. format snapshot id

    you-looks-not-tasty authored and ywc689 committed Jan 23, 2024
    Configuration menu
    Copy the full SHA
    e278598 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    3bf1c3f View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    c2c4d8d View commit details
    Browse the repository at this point in the history
  6. tools/healthcheck: fix rs weight update problem when rs's weight chan…

    …ged adminstratively
    
    When healthcheck program updating the rs's weight, it should consider:
    
    * rs's weight may change adminstratively
    * rs availabilty may change due to the server is closed or crashed
    
    Healthcheck stores the original weight when it sends down notification in order to restore
    the rs's weight when it gets healthy later. The problem is original weight is syncd from
    dpvs periodically and not always up-to-date, which may result in incorrect weight update in
    healthcheck up notification.
    
    Signed-off-by: ywc689 <[email protected]>
    ywc689 committed Jan 23, 2024
    Configuration menu
    Copy the full SHA
    dfa9c96 View commit details
    Browse the repository at this point in the history

Commits on Jan 24, 2024

  1. tools/healthcheck: further fix the problem discussed in previous commit

    Unfortunately, the CAS solution in the last commit is not effective for healthcheck
    process cannot maintain the correct previous weight value. Thus resource version is
    used instead, which should be passed to the lb api when the healthcheck try to update
    rs, and the api would reject the request if the resouce version is outdated.
    
    use vs version to resync target status when notification fails
    
    Signed-off-by: ywc689 <[email protected]>
    ywc689 committed Jan 24, 2024
    Configuration menu
    Copy the full SHA
    1806cd0 View commit details
    Browse the repository at this point in the history

Commits on Jan 25, 2024

  1. Configuration menu
    Copy the full SHA
    cba78e9 View commit details
    Browse the repository at this point in the history
  2. more debug log

    you-looks-not-tasty authored and ywc689 committed Jan 25, 2024
    Configuration menu
    Copy the full SHA
    3bc93dd View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2024

  1. Configuration menu
    Copy the full SHA
    c2d2863 View commit details
    Browse the repository at this point in the history

Commits on Feb 21, 2024

  1. tools/healthcheck: fix serveral problems

    - update config version finally in updateConfig and only when the backend config is in healthy state
    - fix notification resync fail problem caused by insufficient channel size
    - fix array index problem in function NewTargetFromStr
    
    Signed-off-by: ywc689 <[email protected]>
    ywc689 committed Feb 21, 2024
    Configuration menu
    Copy the full SHA
    a392793 View commit details
    Browse the repository at this point in the history

Commits on Feb 22, 2024

  1. Configuration menu
    Copy the full SHA
    e18c333 View commit details
    Browse the repository at this point in the history

Commits on Feb 27, 2024

  1. dpvs-agent local cache builded depend on the user api invoke only. (e…

    …g. POST|PUT /vs/${ID}/rs, PUT /vs/${ID})
    
    Healthcheck api /vs/${ID}/rs/health would not update local cache.
    
    GET API /vs | /vs/${ID} response dpvs-agent local cache info default
    and response dpvs running service detail with query param `healthcheck=true`
    you-looks-not-tasty authored and ywc689 committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    e5d71d1 View commit details
    Browse the repository at this point in the history
  2. remove debug log

    you-looks-not-tasty authored and ywc689 committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    134a098 View commit details
    Browse the repository at this point in the history
  3. release service lock

    you-looks-not-tasty authored and ywc689 committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    3528ef2 View commit details
    Browse the repository at this point in the history

Commits on Mar 5, 2024

  1. Configuration menu
    Copy the full SHA
    6b153c0 View commit details
    Browse the repository at this point in the history
  2. tools/healthcheck: fix problems in config update

    Signed-off-by: ywc689 <[email protected]>
    ywc689 committed Mar 5, 2024
    Configuration menu
    Copy the full SHA
    30fb605 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    add443b View commit details
    Browse the repository at this point in the history

Commits on Mar 11, 2024

  1. tools/healthcheck: fix bad icmp checksum problem for udp and udpping …

    …checkers
    
    Signed-off-by: ywc689 <[email protected]>
    ywc689 committed Mar 11, 2024
    Configuration menu
    Copy the full SHA
    05eb6fa View commit details
    Browse the repository at this point in the history

Commits on Mar 15, 2024

  1. release v1.9.7

    Signed-off-by: ywc689 <[email protected]>
    ywc689 committed Mar 15, 2024
    Configuration menu
    Copy the full SHA
    fad525d View commit details
    Browse the repository at this point in the history