Skip to content

apiage: add commands to detect and report on APIs needing to be updated#759

Merged
mergify[bot] merged 4 commits intoceph:masterfrom
phlogistonjohn:jjm-stable-detector
Aug 30, 2022
Merged

apiage: add commands to detect and report on APIs needing to be updated#759
mergify[bot] merged 4 commits intoceph:masterfrom
phlogistonjohn:jjm-stable-detector

Conversation

@phlogistonjohn
Copy link
Collaborator

@phlogistonjohn phlogistonjohn commented Aug 16, 2022

The apiage part is a building block for eventual plans to try and automate this more fully.

Fixes: #591

Find preview apis that should become stable. Find deprecated apis that
should be removed. Have a mode to convert between the low level json
report and some markdown that could be pasted into an issue.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
Signed-off-by: John Mulligan <jmulligan@redhat.com>
Right now this workflow needs to be run manually and it will generate
some files that are archived. The plan is to build upon this as a base
eventually teaching the workflow how to automatically file an issue
reporting on what API updates are needed after each release.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
@phlogistonjohn phlogistonjohn marked this pull request as ready for review August 16, 2022 18:59
@phlogistonjohn phlogistonjohn added the no-API This PR does not include any changes to the public API of a go-ceph package label Aug 16, 2022
@phlogistonjohn
Copy link
Collaborator Author

FWIW I have been reading up on ways to more completely automate this process. I found an action that can help file issues: https://github.com/JasonEtco/create-an-issue which is looking promising. However, I'm starting small with a manual action that just archives the output of the new apiage.py commands.

This PR is mainly for the Makefile and aipage.py changes. I plan to use a later PR for the other actions enhancements so this should be reviewable as-is. Thanks!

Signed-off-by: John Mulligan <jmulligan@redhat.com>
Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm.

@mergify mergify bot merged commit c29eb9d into ceph:master Aug 30, 2022
@phlogistonjohn phlogistonjohn deleted the jjm-stable-detector branch August 31, 2022 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-API This PR does not include any changes to the public API of a go-ceph package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

apiage: detect APIs that should be stable now

2 participants