- Supporting multiple Go versions
- Updating Go version
The Omnibus GitLab project creates a single, monolithic operating system package containing all the binaries, while the Cloud-Native GitLab (CNG) project publishes a set of Docker images deployed and configured by Helm Charts or the GitLab Operator.
Testing matrices for all projects using Go must include the version shipped by Distribution:
- Check the Go version shipping with Omnibus GitLab.
- Check the Go version shipping with Cloud-Native GitLab (CNG).
Individual Golang projects need to support multiple Go versions because:
- When a new version of Go is released, we should start integrating it into the CI pipelines to verify compatibility with the new compiler.
- We must support the official Omnibus GitLab Go version, which may be behind the latest minor release.
- When Omnibus switches Go version, we still may need to support the old one for security backports.
These 3 requirements may easily be satisfied by keeping support for the 3 latest minor versions of Go.
It is ok to drop support for the oldest Go version and support only the 2 latest releases, if this is enough to support backports to the last 3 minor GitLab releases.
For example, if we want to drop support for
go 1.11 in GitLab
12.10, we need
to verify which Go versions we are using in
12.7. We do not
consider the active milestone,
12.10, because a backport for
12.7 is required
in case of a critical security release.
- If both Omnibus GitLab and Cloud-Native GitLab (CNG) were using Go
12.7and later, then we can safely drop support for
- If Omnibus GitLab or Cloud-Native GitLab (CNG) were using
12.7, then we still need to keep support for Go
1.11for easier backporting of security fixes.
We should always:
- Use the same Go version for Omnibus GitLab and Cloud Native GitLab.
- Use a supported version.
- Use the most recent patch-level for that version to keep up with security fixes.
Changing the version affects every project being compiled, so it’s important to ensure that all projects have been updated to test against the new Go version before changing the package builders to use it. Despite Go’s compatibility promise, changes between minor versions can expose bugs or cause problems in our projects.
The upgrade process involves several key steps:
- Track component updates and validation.
- Track component integration for release.
- Communication with stakeholders.
Use the product categories page if you need help finding the correct person or labels:
- Create the epic in
- Title the epic
Update Go version to <VERSION_NUMBER>.
- Ping the engineering managers responsible for the projects listed below.
- Title the epic
Create an upgrade issue for each dependency in the location indicated below titled
Support building with Go <VERSION_NUMBER>. Add the proper label to each issue for easier triage.
- Schedule an update with the GitLab Development Kit:
- Title the issue
Support using Go version <VERSION_NUMBER>.
- Set the issue as related to every issue created in the previous step.
- Title the issue
- Schedule one issue per Sec Section team that maintains Go based Security Analyzers and add the
section::seclabel to each:Updates to these Security analyzers should not block upgrades to Charts or Omnibus since the analyzers are built independently as separate container images.
- Schedule builder updates with Distribution projects:
- Dependency and GitLab Development Kit issues created in previous steps should be set as blockers.
- Each issue should have the title
Support building with Go <VERSION_NUMBER>and description as noted:
|Component Name||Where to track work|
|Docker Distribution Pruner||Issue Tracker|
|GitLab Compose Kit||Issuer Tracker|
|GitLab Container Registry||Issue Tracker|
|GitLab Elasticsearch Indexer||Issue Tracker|
|GitLab Kubernetes Agent (KAS)||Issue Tracker|
|GitLab Pages||Issue Tracker|
|GitLab Quality Images||Issue Tracker|
|GitLab Shell||Issue Tracker|
|GitLab Workhorse||Issue Tracker|
|Node Exporter||Issue Tracker|
|PgBouncer Exporter||Issue Tracker|
|Postgres Exporter||Issue Tracker|
|Redis Exporter||Issue Tracker|
Communication is required at several key points throughout the process and should be included in the relevant issues as part of the definition of done:
- Immediately after creating the epic, it should be posted to Slack. Community members must ask the pinged engineering managers for assistance with this step. The responsible GitLab team member should share a link to the epic in the following Slack channels:
- Immediately after merging the GitLab Development Kit Update, the same maintainer should add an entry to the engineering week-in-review sync and
announce the change in the following Slack channels:
- Immediately upon merge of the updated Go versions in
Cloud-Native GitLab and
Omnibus GitLab add the
change to the engineering-week-in-review sync and announce in the following
Upstream component maintainers must validate their Go-based projects using:
- Established unit tests in the codebase.
- Procedures established in Merge Request Performance Guidelines.
- Procedures established in Performance, Reliability, and Availability guidelines.
Upstream component maintainers should consider validating their Go-based projects with:
Isolated component operation performance tests.
Integration tests are costly and should be testing inter-component operational issues. Isolated component testing reduces mean time to feedback on updates and decreases resource burn across the organization.
- Components should have end-to-end test coverage in the GitLab Performance Test tool.
- Integration validation through installation of fresh packages and upgrade from previous versions for:
- Single GitLab Node
- Reference Architecture Deployment
- Geo Deployment