Skip to content

Commit

Permalink
backport of commit abb2203
Browse files Browse the repository at this point in the history
  • Loading branch information
jmurret committed Mar 18, 2023
1 parent 15a4645 commit c3e4b78
Show file tree
Hide file tree
Showing 5,020 changed files with 30,583 additions and 62,494 deletions.
The diff you're trying to view is too large. We only load the first 3000 changed files.
3 changes: 0 additions & 3 deletions .changelog/16224.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16251.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16259.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16262.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16266.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16292.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16298.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16412.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16428.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16519.txt

This file was deleted.

3 changes: 0 additions & 3 deletions .changelog/16577.txt

This file was deleted.

90 changes: 1 addition & 89 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,6 @@ references:
- "1.11.6"
- "1.10.9"
- "1.9.10"
consul-versions: &consul_versions
- "1.14"
- "1.15"
images:
# When updating the Go version, remember to also update the versions in the
# workflows section for go-test-lib jobs.
Expand Down Expand Up @@ -890,7 +887,7 @@ jobs:
-p=4 \
-timeout=30m \
-json \
`go list ./... | grep -v upgrade` \
./... \
--target-image consul \
--target-version local \
--latest-image consul \
Expand All @@ -909,84 +906,6 @@ jobs:
path: *TEST_RESULTS_DIR
- run: *notify-slack-failure

upgrade-integration-test:
machine:
image: *UBUNTU_CI_IMAGE
docker_layer_caching: true
parallelism: 3
resource_class: large
parameters:
consul-version:
type: enum
enum: *consul_versions
environment:
CONSUL_VERSION: << parameters.consul-version >>
steps:
- checkout
# Get go binary from workspace
- attach_workspace:
at: .
# Build the consul:local image from the already built binary
- run:
command: |
sudo rm -rf /usr/local/go
wget https://dl.google.com/go/go${GO_VERSION}.linux-amd64.tar.gz
sudo tar -C /usr/local -xzvf go${GO_VERSION}.linux-amd64.tar.gz
environment:
<<: *ENVIRONMENT
- run: *install-gotestsum
- run: docker build -t consul:local -f ./build-support/docker/Consul-Dev.dockerfile .
- run:
name: Upgrade Integration Tests
command: |
mkdir -p /tmp/test-results/
cd ./test/integration/consul-container
docker run --rm consul:local consul version
gotestsum \
--raw-command \
--format=short-verbose \
--debug \
--rerun-fails=3 \
--packages="./..." \
-- \
go test \
-p=4 \
-tags "${GOTAGS}" \
-timeout=30m \
-json \
./.../upgrade/ \
--target-image consul \
--target-version local \
--latest-image consul \
--latest-version $CONSUL_VERSION
ls -lrt
environment:
# this is needed because of incompatibility between RYUK container and circleci
GOTESTSUM_JUNITFILE: /tmp/test-results/results.xml
GOTESTSUM_FORMAT: standard-verbose
COMPOSE_INTERACTIVE_NO_CLI: 1
# tput complains if this isn't set to something.
TERM: ansi
- store_test_results:
path: *TEST_RESULTS_DIR
- store_artifacts:
path: *TEST_RESULTS_DIR
- run: *notify-slack-failure

# Lints all *.dockerfile but don't fail at this time
dockerfile-lint:
docker:
- image: docker.mirror.hashicorp.services/hadolint/hadolint:latest-debian
steps:
- run: apt-get -qq update; apt-get -y install git # the hadolint container doesn't have git
- checkout
- run:
name: Dockefile lint
command: |
for file in $(find . -type f -name *.dockerfile); do
hadolint $file || true
done
envoy-integration-test: &ENVOY_TESTS
machine:
image: *UBUNTU_CI_IMAGE
Expand Down Expand Up @@ -1291,13 +1210,6 @@ workflows:
- compatibility-integration-test:
requires:
- dev-build
- upgrade-integration-test:
requires:
- dev-build
matrix:
parameters:
consul-version: *consul_versions

- noop
frontend:
unless: << pipeline.parameters.trigger-load-test >>
Expand Down
16 changes: 0 additions & 16 deletions .copywrite.hcl

This file was deleted.

3 changes: 2 additions & 1 deletion .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,8 @@ When you're ready to submit a pull request:
| --- | --- |
| `pr/no-changelog` | This PR does not have an intended changelog entry |
| `pr/no-metrics-test` | This PR does not require any testing for metrics |
| `backport/1.12.x` | Backport the changes in this PR to the targeted release branch. Consult the [Consul Release Notes](https://www.consul.io/docs/release-notes) page to view active releases. Website documentation merged to the latest release branch is deployed immediately |
| `backport/stable-website` | This PR contains documentation changes that are ready to be deployed immediately. Changes will also automatically get backported to the latest release branch |
| `backport/1.12.x` | Backport the changes in this PR to the targeted release branch. Consult the [Consul Release Notes](https://www.consul.io/docs/release-notes) page to view active releases. |
Other labels may automatically be added by the Github Action CI.
7. After you submit, the Consul maintainers team needs time to carefully review your
contribution and ensure it is production-ready, considering factors such as: security,
Expand Down
30 changes: 6 additions & 24 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,41 +4,27 @@ about: You're experiencing an issue with Consul that is different than the docum

---

<!-- When filing a bug, please include the following headings if possible. Any example text in this template can be deleted.
-->
When filing a bug, please include the following headings if possible. Any example text in this template can be deleted.

#### Overview of the Issue

<!-- Please provide a paragraph or two about the issue you're experiencing. -->

---
A paragraph or two about the issue you're experiencing.

#### Reproduction Steps

<!-- Please provide steps to reproduce the bug, without any details it would be hard to troubleshoot:
Steps to reproduce this issue, eg:

1. Create a cluster with n client nodes n and n server nodes
1. Run `curl ...`
1. View error

-->

### Consul info for both Client and Server


<!--- Please provide both `consul info` and agent HCL config for both client and servers to help us better diagnose the issue. Take careful steps to remove any sensitive information from config files that include secrets such as Gossip keys. --->

<details>
<summary>Client info</summary>

```
Output from client 'consul info' command here
```

```
Client agent HCL config
output from client 'consul info' command here
```

</details>
Expand All @@ -47,19 +33,15 @@ Client agent HCL config
<summary>Server info</summary>

```
Output from server 'consul info' command here
```

```
Server agent HCL config
output from server 'consul info' command here
```

</details>

### Operating system and Environment details

<!-- OS, Architecture, and any other information you can provide about the environment. -->
OS, Architecture, and any other information you can provide about the environment.

### Log Fragments

<!-- Include appropriate Client or Server log fragments. If the log is longer than a few dozen lines, please include the URL to the [gist](https://gist.github.com/) of the log instead of posting it in the issue. Use `-log-level=TRACE` on the client and server to capture the maximum log detail. -->
Include appropriate Client or Server log fragments. If the log is longer than a few dozen lines, please include the URL to the [gist](https://gist.github.com/) of the log instead of posting it in the issue. Use `-log-level=TRACE` on the client and server to capture the maximum log detail.
6 changes: 3 additions & 3 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@ about: If you have something you think Consul could improve or add support for.

---

<!--- Please search the existing issues for relevant feature requests, and use the reaction feature (https://blog.github.com/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) to add upvotes to pre-existing requests. --->
Please search the existing issues for relevant feature requests, and use the reaction feature (https://blog.github.com/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/) to add upvotes to pre-existing requests.

#### Feature Description

<!--- Please provide a written overview of the feature and why this feature solves challenges that you are facing today. --->
A written overview of the feature.

#### Use Case(s)

<!--- Please describe the use case for this feature (i.e. Service Mesh, Service Discovery, KV, API Gateway) and also deployment environments you are looking to see this addressed in (K8s, VMs, Nomad, ECS, Lambda). -->
Any relevant use-cases that you see.
25 changes: 12 additions & 13 deletions .github/ISSUE_TEMPLATE/ui_issues.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,42 +4,41 @@ about: You have usage feedback for the browser based UI

---

<!--- When filing a bug, please include the following headings if possible. Any example text in this template can be deleted. --->
When filing a bug, please include the following headings if possible. Any example text in this template can be deleted.

### Overview of the Issue

<!--- Please provide a paragraph or two about the issue you're experiencing / suggestion for
improvement. --->
A paragraph or two about the issue you're experiencing / suggestion for
improvement.

### Reproduction Steps

<!--- Steps to reproduce this issue/view the area for improvement, eg:
Steps to reproduce this issue/view the area for improvement, eg:

1. Visit the UI page at `/ui/services`
1. Click .... then click...etc.
1. View error/area.

--->

### Describe the solution you'd like

<!--- If this is an improvement rather than a bug, a clear and concise description
If this is an improvement rather than a bug, a clear and concise description
of what you want to happen. How have you seen this problem solved in other
UIs? --->
UIs?

### Consul Version

<!--- This can be found either in the footer of the UI (Consul versions pre 1.10) or
at the top of the help menu that is in the top right side of the UI. --->
This can be found either in the footer of the UI (Consul versions pre 1.10) or
at the top of the help menu that is in the top right side of the UI.

### Browser and Operating system details

<!--- Browser, Browser Version, OS, and any other information you can provide about the environment that may be relevant. --->
Browser, Browser Version, OS, and any other information you can provide about the environment that may be relevant.

### Screengrabs / Web Inspector logs

<!--- If you think it's worthwhile, include appropriate screengrabs showing the
If you think it's worthwhile, include appropriate screengrabs showing the
error/area for improvement. Try to include the URL bar of the browser so we
can see the current URL where the error manifests. Please be careful to
obfuscate any sensitive information either in the URL or in the browser page
itself. --->
itself.

13 changes: 1 addition & 12 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,27 +1,16 @@
### Description

<!-- Please describe why you're making this change, in plain English. -->
Describe why you're making this change, in plain English.

### Testing & Reproduction steps

<!--
* In the case of bugs, describe how to replicate
* If any manual tests were done, document the steps and the conditions to replicate
* Call out any important/ relevant unit tests, e2e tests or integration tests you have added or are adding

-->

### Links

<!--
Include any links here that might be helpful for people reviewing your PR (Tickets, GH issues, API docs, external benchmarks, tools docs, etc). If there are none, feel free to delete this section.

Please be mindful not to leak any customer or confidential information. HashiCorp employees may want to use our internal URL shortener to obfuscate links.

-->

### PR Checklist

* [ ] updated test coverage
Expand Down
27 changes: 27 additions & 0 deletions .github/workflows/backport-assistant.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,33 @@ jobs:
runs-on: ubuntu-latest
container: hashicorpdev/backport-assistant:0.3.0
steps:
- name: Run Backport Assistant for stable-website
run: |
backport-assistant backport -merge-method=squash -gh-automerge
env:
BACKPORT_LABEL_REGEXP: "type/docs-(?P<target>cherrypick)"
BACKPORT_TARGET_TEMPLATE: "stable-website"
BACKPORT_MERGE_COMMIT: true
GITHUB_TOKEN: ${{ secrets.ELEVATED_GITHUB_TOKEN }}
- name: Backport changes to latest release branch
run: |
# Use standard token here
resp=$(curl -f -s -H 'authorization: Bearer ${{ secrets.GITHUB_TOKEN }}' "https://api.github.com/repos/$GITHUB_REPOSITORY/labels?per_page=100")
ret="$?"
if [[ "$ret" -ne 0 ]]; then
echo "The GitHub API returned $ret"
exit $ret
fi
# get the latest backport label excluding any website labels, ex: `backport/0.3.x` and not `backport/website`
latest_backport_label=$(echo "$resp" | jq -r '.[] | select(.name | (startswith("backport/") and (contains("website") | not))) | .name' | sort -rV | head -n1)
echo "Latest backport label: $latest_backport_label"
# set BACKPORT_TARGET_TEMPLATE for backport-assistant
# trims backport/ from the beginning with parameter substitution
export BACKPORT_TARGET_TEMPLATE="release/${latest_backport_label#backport/}.x"
backport-assistant backport -merge-method=squash -gh-automerge
env:
BACKPORT_LABEL_REGEXP: "type/docs-(?P<target>cherrypick)"
GITHUB_TOKEN: ${{ secrets.ELEVATED_GITHUB_TOKEN }}
- name: Run Backport Assistant for release branches
run: |
backport-assistant backport -merge-method=squash -gh-automerge
Expand Down
Loading

0 comments on commit c3e4b78

Please sign in to comment.