Skip to content
Snippets Groups Projects

Add Consul Upgrade Documentation

Merged Robert Marshall requested to merge 2906-update-consul-documentation into master
All threads resolved!
  • Adds documentation for what to do during Consul Upgrades.

Related omnibus-gitlab#2906 (closed)

Signed-off-by: Robert Marshall rmarshall@gitlab.com

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ian Baum
  • Ian Baum
  • Ian Baum assigned to @rmarshall and unassigned @ibaum

    assigned to @rmarshall and unassigned @ibaum

  • added 1 commit

    Compare with previous version

  • Ready next pass.

  • Robert Marshall assigned to @ibaum and unassigned @rmarshall

    assigned to @ibaum and unassigned @rmarshall

  • @ibaum doing a -amend and force push to fix the bot warning.

  • added 1 commit

    • ff862b65 - Review various fixups requested in documentation

    Compare with previous version

  • @ibaum amended, but can't run a new pipe to clear the bot message.

  • Ian Baum
  • Ian Baum
  • Ian Baum
  • Ian Baum
  • Ian Baum
  • Ian Baum
  • Ian Baum assigned to @rmarshall and unassigned @ibaum

    assigned to @rmarshall and unassigned @ibaum

  • Robert Marshall resolved all threads

    resolved all threads

  • Robert Marshall added 2727 commits

    added 2727 commits

    Compare with previous version

  • Robert Marshall assigned to @ibaum and unassigned @rmarshall

    assigned to @ibaum and unassigned @rmarshall

  • Robert Marshall resolved all threads

    resolved all threads

  • @ibaum @rmarshall I left some suggestions! There's also one that fixes the broken test.

  • Robert Marshall resolved all threads

    resolved all threads

  • added 1 commit

    • fea1ad0f - Fix issues pointed out in Review

    Compare with previous version

  • fix for failing tests over here: !18882 (closed)

  • @ibaum - not sure if there's more to do here content wise. My patch for master, once applied and I rebase this branch, will fix the current linter failures.

    The failures appeared because of a known issue in the doc linter.

    If you are all set on content @ibaum, we can re-assign to @axil for final style recommendations and the merge.

  • Ian Baum
  • Ian Baum
  • Ian Baum assigned to @rmarshall and unassigned @ibaum

    assigned to @rmarshall and unassigned @ibaum

  • added 1 commit

    • 7915ff4d - Apply suggestion to doc/administration/high_availability/consul.md

    Compare with previous version

  • Robert Marshall resolved all threads

    resolved all threads

  • added 1 commit

    • ce6e9c0b - Apply suggestion to doc/administration/high_availability/consul.md

    Compare with previous version

  • Robert Marshall assigned to @ibaum and unassigned @rmarshall

    assigned to @ibaum and unassigned @rmarshall

  • Maintainer

    I think this looks good to me. Passing onto @axil to get the docs team point of view

  • Ian Baum assigned to @axil and unassigned @ibaum

    assigned to @axil and unassigned @ibaum

  • Achilleas Pipinellis changed milestone to %12.5

    changed milestone to %12.5

  • Achilleas Pipinellis approved this merge request

    approved this merge request

  • mentioned in commit 8bc1fc53

  • mentioned in merge request omnibus-gitlab!3617 (closed)

  • mentioned in merge request omnibus-gitlab!3659 (closed)

  • Please register or sign in to reply
    Loading