[5pt]Automatically aggregate TensorFlow symbols on new TensorFlow minor release

fridex commented 4 years ago

Is your feature request related to a problem? Please describe.

We recommend TensorFlow based on API symbols used from the library (match user's symbols used with the ones provided by TensorFlow). To support this idea long-term and reduce time spent on maintaining the symbols database, we could create an automatic symbols database update procedure. This procedure can be triggered automatically on new TensorFlow release.

Describe the solution you'd like

Each time there is a new minor TensorFlow release on PyPI, aggregate exposed TensorFlow symbols. A high level procedure description:

[ ] run https://github.com/thoth-station/tensorflow-symbols in an automated way within argo workflows (parametrized using TF version identifier)
[ ] open a PR to tensorflow-symbols repository updating the symbols database
[ ] use pre-aggregated tensorflow symbols from older releases and build a symbols database for adviser with symbols aggregated from the new release
[ ] open an automatic PR to the adviser repository
[ ] make sure bots automatically approve the symbols update PR and a new adviser release can be issued (can also be done automatically/semi-automatically)

Describe alternatives you've considered

Have this procedure as a manual process, which is time-consuming and mundane.

Additional context

See discussion in https://github.com/thoth-station/tensorflow-symbols/pull/6#issuecomment-734718946

sesheta commented 3 years ago

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

fridex commented 3 years ago

/remove-lifecycle stale

goern commented 3 years ago

/priority backlog

sesheta commented 3 years ago

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle rotten

goern commented 3 years ago

/remove-lifecycle rotten /lifecycle forzen

sesheta commented 3 years ago

Rotten issues close after 30d of inactivity. Reopen the issue with /reopen. Mark the issue as fresh with /remove-lifecycle rotten.

/close

sesheta commented 3 years ago

@sesheta: Closing this issue.

In response to [this](https://github.com/thoth-station/thoth-application/issues/682#issuecomment-913008809): >Rotten issues close after 30d of inactivity. >Reopen the issue with `/reopen`. >Mark the issue as fresh with `/remove-lifecycle rotten`. > >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

harshad16 commented 3 years ago

/lifecycle frozen

codificat commented 2 years ago

/triage accept

sesheta commented 2 years ago

@codificat: The label(s) triage/accept cannot be applied, because the repository doesn't have them.

In response to [this](https://github.com/thoth-station/thoth-application/issues/682#issuecomment-998848260): >/triage accept Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

codificat commented 2 years ago

/triage accepted

thoth-station / thoth-application

[5pt]Automatically aggregate TensorFlow symbols on new TensorFlow minor release #682