chore(ci): use new version determination

Signed-off-by: Justin Kolberg <[email protected]>

.github/workflows/dev_builds.yml

@@ -22,12 +22,14 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-tags: true
+
       - name: Get version
         id: get-version
         run: |
-          TAG=$(git describe --tags --abbrev=5 --match "v[0-9]*")
-          echo "version=${TAG/v}" > $GITHUB_OUTPUT
+          version=$(./ci/get_version.sh full | xargs)-$(git rev-parse HEAD | xargs)
+          echo "version=${version}" > $GITHUB_OUTPUT
+
       - name: Print version
         run: echo ::notice title=Version::${{ steps.get-version.outputs.version }}
 
@@ -151,7 +153,7 @@ jobs:
       runs-on: ${{ matrix.runs-on }}
       fips: ${{ matrix.fips == true }}
       save-cache: true
-      sumo_component_gomod_version: "v${{ needs.get-version.outputs.version }}"
+      version: ${{ needs.get-version.outputs.version }}
     secrets:
       apple_developer_certificate_p12_base64: ${{ secrets.APPLE_DEVELOPER_CERTIFICATE_P12_BASE64 }}
       apple_developer_certificate_password: ${{ secrets.APPLE_DEVELOPER_CERTIFICATE_PASSWORD }}

.github/workflows/pull_requests.yml

@@ -33,6 +33,25 @@ jobs:
             **/Makefile.common
             **/Dockerfile*
 
+  get-version:
+    name: Get application version for this revision
+    runs-on: ubuntu-latest
+    outputs:
+      version: ${{ steps.get-version.outputs.version }}
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-tags: true
+
+      - name: Get version
+        id: get-version
+        run: |
+          version=$(./ci/get_version.sh full | xargs)-$(git rev-parse HEAD | xargs)
+          echo "version=${version}" > $GITHUB_OUTPUT
+
+      - name: Print version
+        run: echo ::notice title=Version::${{ steps.get-version.outputs.version }}
+
   markdownlint:
     runs-on: ubuntu-20.04
     steps:
@@ -249,6 +268,7 @@ jobs:
       arch_os: ${{ matrix.arch_os }}
       runs-on: ${{ matrix.runs-on }}
       fips: ${{ matrix.fips == true }}
+      version: ${{ needs.get-version.outputs.version }}
     secrets:
       apple_developer_certificate_p12_base64: ${{ secrets.APPLE_DEVELOPER_CERTIFICATE_P12_BASE64 }}
       apple_developer_certificate_password: ${{ secrets.APPLE_DEVELOPER_CERTIFICATE_PASSWORD }}

.github/workflows/releases.yml

@@ -315,9 +315,6 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - name: Fetch current branch
-        run: ./ci/fetch_current_branch.sh
-
       - name: Archive Chef cookbook
         run: tar -cvf ./examples/chef/chef-cookbook.tar.gz -C ./examples/chef sumologic-otel-collector/
 

.github/workflows/workflow-build-otelcol-config.yml

@@ -46,9 +46,6 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - name: Fetch current branch
-        run: ./ci/fetch_current_branch.sh
-
       - name: Setup go
         uses: actions/setup-go@v5
         with:

.github/workflows/workflow-build.yml

@@ -22,6 +22,10 @@ on:
         description: Save the module and build caches.
         default: false
         type: boolean
+      version:
+        description: Version to use when building the binary
+        required: true
+        type: string
     secrets:
       apple_developer_certificate_p12_base64:
         required: false
@@ -46,6 +50,7 @@ defaults:
 
 env:
   GO_VERSION: "1.22.6"
+  VERSION: ${{ inputs.version }}
 
 jobs:
   build:
@@ -57,9 +62,6 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-      - name: Fetch current branch
-        run: ./ci/fetch_current_branch.sh
-
       - name: Determine if MacOS package should be signed
         if: runner.os == 'macOS'
         env:
@@ -186,6 +188,14 @@ jobs:
           go tool nm ${{ steps.set-binary-name.outputs.binary_name }} | \
           grep "vendor/github.com/microsoft/go-crypto-winnative/internal/bcrypt.GetFipsAlgorithmMode"
 
+      - name: Verify binary version
+        if: inputs.arch_os == 'linux_amd64' || inputs.arch_os == 'darwin_amd64'
+        run: |
+          binary=${{ steps.set-binary-name.outputs.binary_name }}
+          binary_path=otelcolbuilder/cmd/${binary}
+          ./ci/get_version_from_binary.sh otc "${binary_path}"
+          ./ci/get_version_from_binary.sh sumo "${binary_path}"
+
       - uses: apple-actions/import-codesign-certs@v3
         if: ${{ runner.os == 'macOS' && env.MACOS_SIGNING_ENABLED == 'true' }}
         with:

.github/workflows/workflow-trigger-packaging.yml

@@ -20,12 +20,6 @@ jobs:
     name: Trigger Packaging
     runs-on: ubuntu-latest
     steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-          filter: 'tree:0'
-
       # TODO: output the workflow url
       - name: Trigger packaging workflow
         id: trigger-packaging-workflow

ci/get_version.sh

@@ -7,7 +7,7 @@ declare -i sumo_version
 
 usage() {
   cat <<EOF
-Usage: $(basename "${BASH_SOURCE[0]}") [-h] [core|sumo|productversion]
+Usage: $(basename "${BASH_SOURCE[0]}") [-h] [core|sumo|full|productversion]
 
 Detects the version to use for building otelcol-sumo using a combination of the
 otelcol-builder config and Git tags. It can output version information in

ci/get_version_from_binary.sh

@@ -7,7 +7,7 @@ declare -i sumo_version
 
 usage() {
   cat <<EOF
-Usage: $(basename "${BASH_SOURCE[0]}") [-h] [otc|sumo] [path/to/otelcol-sumo]
+Usage: $(basename "${BASH_SOURCE[0]}") [-h] [core|sumo|productversion] [path/to/otelcol-sumo]
 
 Retrieves the version of an otelcol-sumo binary by parsing the output of
 otelcol-sumo's version flag.
@@ -53,7 +53,7 @@ parse_version() {
 
     output="$(${binary_path} --version)"
 
-    regex=".* v([0-9]+)\.([0-9]+)\.([0-9]+)\-sumo\-([0-9]+).*"
+    regex=".* ([0-9]+)\.([0-9]+)\.([0-9]+)\-sumo\-([0-9]+).*"
 
     if [[ $output =~ $regex ]]; then
         major_version="${BASH_REMATCH[1]}"

otelcolbuilder/Makefile

@@ -86,7 +86,7 @@ install-builder:
 
 .PHONY: _builder
 _builder:
-	$(eval VERSION ?= $(shell ./ci/get_version.sh full | xargs)-$(shell git rev-parse HEAD | xargs))
+	$(eval VERSION ?= $(shell ../ci/get_version.sh full | xargs)-$(shell git rev-parse HEAD | xargs))
 # Need to specify go path because otherwise opentelemetry-collector-builder
 # uses /usr/bin/go which on Github Actions is using preinstalled 1.15.12 by default.
 	CGO_ENABLED=$(CGO_ENABLED) $(BUILDER_BIN_NAME) \