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.

Sign up for GitHub

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

Improve building process with cache goes dependencies #689

Merged
merged 15 commits into from
Oct 29, 2021
Merged

Improve building process with cache goes dependencies #689

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
09434d0
first commit to merge and request
khanhntd Oct 26, 2021
c9e9853
add cache dependencies for ubuntu linux based on this example: https:…
khanhntd Oct 27, 2021
641285b
download all dependencies from go.mod and cache
khanhntd Oct 27, 2021
2db63cf
fix ci yaml file format
khanhntd Oct 27, 2021
5e738d8
add comment for cache dependencies
khanhntd Oct 28, 2021
e2c24a5
testing commit
khanhntd Oct 28, 2021
3988504
final test before doing a pr
khanhntd Oct 28, 2021
53a910a
delete branch fix-issue-688 from CI when pushing
khanhntd Oct 28, 2021
7054e19
delete go mod download when not having mod packages, add go build cac…
khanhntd Oct 28, 2021
39c08c3
testing CI once more to know the cache affects on new commits
khanhntd Oct 28, 2021
7872523
avoid merge branch when cache
khanhntd Oct 28, 2021
a64aea2
testing final time before sending a pull request
khanhntd Oct 28, 2021
edb6da0
combine two cache into one for go build and go dependencies
khanhntd Oct 28, 2021
785c85c
delete CI for fix-issue-688 branch
khanhntd Oct 28, 2021
a434923
delete restore key
khanhntd Oct 28, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion .github/workflows/CI.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
# permissions and limitations under the License.

name: C/I

on:
push:
branches:
Expand Down Expand Up @@ -100,6 +99,21 @@ jobs:
cat go.mod
ls pkg

#Cache go build and dependencies before making unit testing and build
#Samples codes for different OS: https://github.com/actions/cache/blob/main/examples.md#go---modules
#Since we are using Linux, the go packages are in /go/pkg/mod and build are in /.cache/go-build
#Also speed up unit testing since go test uses the go build cache and also speed up the go build.
- name: Cache go
id: cached_go
uses: actions/cache@v2
env:
cache-name: cached_go_modules
with:
path: |
~/go/pkg/mod
~/.cache/go-build
key: go-pkg-mod-${{ runner.os }}-${{ hashFiles('**/go.sum') }}

- name: Cache binaries
id: cached_binaries
uses: actions/cache@v2
Expand Down
1 change: 0 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
![CD](https://github.com/aws-observability/aws-otel-collector/workflows/CD/badge.svg)
![GitHub release (latest by date)](https://img.shields.io/github/v/release/aws-observability/aws-otel-collector)


### Overview

AWS Distro for OpenTelemetry Collector(AWS OTel Collector) is a AWS supported version of the upstream OpenTelemetry Collector and is distributed by Amazon. It supports the selected components from the OpenTelemetry community. It is fully compatible with AWS computing platforms including EC2, ECS and EKS. It enables users to send telemetry data to AWS CloudWatch Metrics, Traces and Logs backends as well as the other supported backends.
Expand Down