Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: hashicorp/raft
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.7.1
Choose a base ref
...
head repository: hashicorp/raft
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.7.3
Choose a head ref
  • 13 commits
  • 25 files changed
  • 9 contributors

Commits on Jun 1, 2024

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

Commits on Sep 24, 2024

  1. Warn about usage of snapshots (#618)

    * Update fsm.go
    
    * Update fsm.go
    otoolep authored Sep 24, 2024
    Configuration menu
    Copy the full SHA
    dd1f3da View commit details
    Browse the repository at this point in the history

Commits on Oct 10, 2024

  1. Support retrieving current term (#620)

    Simple addition to make this internal variable accessible (without relying on `stats` object, which is a brittle approach).
    
    _Term_ is a fundamental concept in Raft consensus, so making it easily available to clients of this library seems like an obvious thing to do. There are also specific use cases supported by knowing the Term:
    
    - Allow clients of this library to retrieve and display diagnostic information about the state of the Raft system.
    - Support certain types of reads of a Raft-managed store. For example, if a client can check before and after a Read that an election has not taken place during the read, it allows those clients to make certain guarantees about the data read from the Raft-managed Store. One way to do this is to simply check that the Term has not changed (see [this discussion](https://groups.google.com/g/raft-dev/c/4QlyV0aptEQ/m/1JxcmSgRAwAJ) for more details).
    otoolep authored Oct 10, 2024
    Configuration menu
    Copy the full SHA
    2b0032e View commit details
    Browse the repository at this point in the history

Commits on Nov 18, 2024

  1. Configuration menu
    Copy the full SHA
    4b60222 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    fb360eb View commit details
    Browse the repository at this point in the history
  3. Merge pull request #595 from alexandear/docs-comments-fix-grammar

    Fix typos in docs, comments, and tests
    schmichael authored Nov 18, 2024
    Configuration menu
    Copy the full SHA
    ede1f8d View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    017ad34 View commit details
    Browse the repository at this point in the history

Commits on Nov 19, 2024

  1. Configuration menu
    Copy the full SHA
    8368671 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #629 from mukeshjc/main

    Add CODEOWNERS file in .github/CODEOWNERS
    mukeshjc authored Nov 19, 2024
    Configuration menu
    Copy the full SHA
    7e8e836 View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2024

  1. chore: Add flag to skip legacy duplicate telemetry (#630)

    Currently we have some legacy metrics with `peer_id` in the metrics suffix (in
    addition to same metrics with `peer_id`d as label)
    1. `raft_replication_appendEntries_rpc_peer0`
    2. `raft_replication_appendEntries_logs_peer0`
    3. `raft_replication_heartbeat_peer0`
    4. `raft_replication_installSnapshot_peer0`
    
    These metrics may have additional `_count` or `_sum` metrics. And each metrics
    are multiplicative. Meaning if I have 10 peers, these metrics will be 10x.
    
    This PR adds a flag `noLegacyTelemetry` (default: false) which by setting to
    `true` you can skip those duplicate metrics.
    
    ---------
    
    Signed-off-by: Kaviraj <kavirajkanagaraj@gmail.com>
    Co-authored-by: Piotr Kazmierczak <470696+pkazmierczak@users.noreply.github.com>
    kavirajk and pkazmierczak authored Dec 12, 2024
    Configuration menu
    Copy the full SHA
    a5bc06c View commit details
    Browse the repository at this point in the history

Commits on Jan 9, 2025

  1. Introduce hashicorp/go-metrics compatibility (#631)

    * Introduce hashicorp/go-metrics compatibility
    
    Compatability is attained with build tags
    
    Using tag armonmetrics or no tag at all will result in the library utilizing github.com/armon/go-metrics for metrics emission (like the library historically has done)
    
    Using tag hashicorpmetrics will result in the library utilizing the updated github.com/hashicorp/go-metrics for metrics emission.
    mkeeler authored Jan 9, 2025
    Configuration menu
    Copy the full SHA
    7650cd0 View commit details
    Browse the repository at this point in the history
  2. Add linter rules to ensure we don't accidentally emit metrics directl…

    …y instead of through the compat package (#633)
    mkeeler authored Jan 9, 2025
    Configuration menu
    Copy the full SHA
    8f99c15 View commit details
    Browse the repository at this point in the history

Commits on Mar 18, 2025

  1. Add logging on active node when sending a snapshot to a follower (#636)

    * Add a log line on active node when sending snapshot to a follower
    
    * Add snap ID and peer ID to relevant log lines in sendLatestSnapshot; fix - use peer.ID instead of peer as a value in log line
    
    * Add an INFO log line when opening a snapshot in sendLatestSnapshot
    kubawi authored Mar 18, 2025
    Configuration menu
    Copy the full SHA
    c0dc6a0 View commit details
    Browse the repository at this point in the history
Loading