openshift / openshift/cloud-credential-operator

Add Version to ccoctl binary arguments?

Open
#1,091 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go
Stars
75
Forks
196
Avg merge
8d 13h
Merged PRs (30d)
5

Description

Collaborating on an OCP Tooling project, most all of the tools have version outputs that are used to populate an index (similar to the Web Terminal Operator Tools container). However, ccoctl doesn't have any version output.

The lifecycle is tied to OCP release channels it seems, and there is a package at pkg/version that seems to be unused?

I can make a PR to pipe that into ccoctl - the test extension already has an ldflag to pass the pkg/version/version.versionFromGit, similar inputs could/should be used to expand the versioning output.

Thoughts?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the unused pkg/version package and the ccoctl argument handling. Compare its version inputs with the test extension's existing ldflag for pkg/version/version.versionFromGit. Done means ccoctl exposes a version output suitable for the OCP tooling index and the related behavior is covered by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.