cyberark / conjur-oss-suite-release

Under development - Latest stable releases of the Conjur OSS suite
Apache License 2.0
7 stars 2 forks source link

Each suite component repo has a tagged release that meets suite requirements #43

Closed izgeri closed 4 years ago

izgeri commented 4 years ago
hughsaunders commented 4 years ago

Reading the description, it looks like this issue is about making sure each component has a release that meets the requirements for inclusion in the suite.

The title could be read as suggesting each component should be tagged with a suite release verison. Maybe the title could be changed to "Components latest release meets suite release requirements"

izgeri commented 4 years ago

Note from @sgnn7:

izgeri commented 4 years ago

Note: all repos have valid tags, except for the cyberark/conjur-api-dotnet project, which has no tags. We did not include it in the first release.

It would be useful if all repos included instructions on how to get artifacts in their GH release pages - it still seems tricky for people to know where to get all the pieces, which is not ideal.

We still need to verify:

doodlesbykumbi commented 4 years ago

Re:

Does each release look correct in the unreleased changes page?

@izgeri Could you please elaborate on what correct means here?


Re:

Does each most recent tag for each component actually correspond to an artifact with the same version?

Managed to get answers to the second question via some automation, see the results below.

cat << EOL | while read LINE; do;
cyberark/conjur
cyberark/conjur-oss-helm-chart
cyberark/conjur-aws
cyberark/conjur-cli
cyberark/conjur-api-dotnet
cyberark/conjur-api-go
cyberark/conjur-api-java
cyberark/conjur-api-python3
cyberark/conjur-api-ruby
cyberark/conjur-authn-k8s-client
cyberark/secrets-provider-for-k8s
cyberark/conjur-service-broker
cyberark/cloudfoundry-conjur-buildpack
cyberark/ansible-conjur-host-identity
cyberark/conjur-credentials-plugin
cyberark/conjur-puppet
cyberark/terraform-provider-conjur
cyberark/secretless-broker
cyberark/summon
cyberark/summon-aws-secrets
cyberark/summon-chefapi
cyberark/summon-conjur
cyberark/summon-keyring
cyberark/summon-s3
EOL
  echo "---"
  echo "${LINE}"
  latest_tag=$(hub api "/repos/${LINE}/tags" | jq -r 'sort_by(.name) | reverse | .[0].name')
  echo "latest tag is ${latest_tag}"
  hub api "/repos/${LINE}/releases" | jq 'map(select(.tag_name == "'${latest_tag}'")) | map("name: \(.name), tag_name: \(.tag_name)")'
done

Results below. Note: 1.latest tag is null means there are no tags

  1. [] after the latest tag, means there are no releases using that tag
---
cyberark/conjur
latest tag is v1.5.1
[
  "name: v1.5.1, tag_name: v1.5.1"
]
---
cyberark/conjur-oss-helm-chart
latest tag is v1.3.8
[
  "name: v1.3.8, tag_name: v1.3.8"
]
---
cyberark/conjur-aws
latest tag is null
[]
---
cyberark/conjur-cli
latest tag is v6.2.1
[
  "name: v6.2.1, tag_name: v6.2.1"
]
---
cyberark/conjur-api-dotnet
latest tag is v1.4.0
[
  "name: v1.4.0, tag_name: v1.4.0"
]
---
cyberark/conjur-api-go
latest tag is v0.6.0
[
  "name: v0.6.0, tag_name: v0.6.0"
]
---
cyberark/conjur-api-java
latest tag is v2.0.0
[
  "name: v2.0.0, tag_name: v2.0.0"
]
---
cyberark/conjur-api-python3
latest tag is v0.1.0
[]
---
cyberark/conjur-api-ruby
latest tag is v5.3.1
[
  "name: v5.3.1, tag_name: v5.3.1"
]
---
cyberark/conjur-authn-k8s-client
latest tag is v0.16.1
[
  "name: v0.16.1, tag_name: v0.16.1"
]
---
cyberark/secrets-provider-for-k8s
latest tag is v0.4.0
[
  "name: v0.4.0, tag_name: v0.4.0"
]
---
cyberark/conjur-service-broker
latest tag is v1.1.1
[
  "name: v1.1.1, tag_name: v1.1.1"
]
---
cyberark/cloudfoundry-conjur-buildpack
latest tag is v2.1.3
[
  "name: v2.1.3, tag_name: v2.1.3"
]
---
cyberark/ansible-conjur-host-identity
latest tag is v0.3.1
[
  "name: v0.3.1, tag_name: v0.3.1"
]
---
cyberark/conjur-credentials-plugin
latest tag is v0.8.0
[
  "name: v0.8.0, tag_name: v0.8.0"
]
---
cyberark/conjur-puppet
latest tag is v2.0.2
[
  "name: v2.0.2, tag_name: v2.0.2"
]
---
cyberark/terraform-provider-conjur
latest tag is v0.2.0
[
  "name: v0.2.0, tag_name: v0.2.0"
]
---
cyberark/secretless-broker
latest tag is v1.5.2
[
  "name: v1.5.2, tag_name: v1.5.2"
]
---
cyberark/summon
latest tag is v0.8.1
[
  "name: v0.8.1, tag_name: v0.8.1"
]
---
cyberark/summon-aws-secrets
latest tag is v0.3.0
[
  "name: v0.3.0, tag_name: v0.3.0"
]
---
cyberark/summon-chefapi
latest tag is v0.1.1
[
  "name: v0.1.1, tag_name: v0.1.1"
]
---
cyberark/summon-conjur
latest tag is v0.5.3
[
  "name: v0.5.3, tag_name: v0.5.3"
]
---
cyberark/summon-keyring
latest tag is v0.2.1
[
  "name: v0.2.1, tag_name: v0.2.1"
]
---
cyberark/summon-s3
latest tag is v0.2.0
[
  "name: v0.2.0, tag_name: v0.2.0"
]
doodlesbykumbi commented 4 years ago

So it looks like

  1. cyberark/conjur-aws doesn't have any tags
  2. cyberark/conjur-api-python3's latest tag doesn't have a corresponding release
izgeri commented 4 years ago

nice!! we'll have to get the python3 maintainer to add a release to that latest tag ... (@sgnn7)

sgnn7 commented 4 years ago

The 0.1.0 tag was only to push the docker python3-based cli container to dockerhub but that got stalled in mgmt. There's no user-facing change in that release.

izgeri commented 4 years ago

@sgnn7 how do you want to handle including that project in the suite? based on what's above, it looks like it doesn't meet the reqs yet

sgnn7 commented 4 years ago

I don't think this is a very uncommon situation where we have a tag but no release - it's desired but it shouldn't be a requirement. I can maybe create a draft release for now?

izgeri commented 4 years ago

Maybe I misunderstood, but I thought the suite was querying releases and wouldn't see tags with no releases - I can wait to read more in your draft guidelines once you've got them up.

sgnn7 commented 4 years ago

@izgeri : You are correct - @doodlesbykumbi ran a custom script to check the tags. Tags without releases won't be seen by the suite. More directly to the repo in question, 0.1.0 tag doesn't warrant a release right now so it should be invisible to the suite (it should pick up 0.0.5).

sgnn7 commented 4 years ago

Resolution: tag deleted for now in python3 api repo

izgeri commented 4 years ago

@doodlesbykumbi @sgnn7 are we satisfied that this card is complete?

we might be technically missing release instructions in each repo, but we've already ensured each repo links to the general CONTRIBUTING.md and that includes the general tag/release instructions now. so I think this part is done too.

izgeri commented 4 years ago

I'm going to mark it as complete, feel free to reopen if there's anything additional to do.