Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade the pipeline trigger logic to run on updates to previously processed release branches #82

Open
chrisammon3000 opened this issue Apr 10, 2023 · 1 comment · May be fixed by #102
Assignees

Comments

@chrisammon3000
Copy link
Contributor

chrisammon3000 commented Apr 10, 2023

Description

The gfe-db update pipeline (AWS Lambda) checks the IMGT repository once a day and triggers a build if a new release has been added. However if additional changes are made to that release they are not picked up. The solution would be to upgrade the logic so that the pipeline also runs when a release branch is updated.

Proposed Solution

In addition to checking for new release branches, the Lambda function should also check for recent commits that include changes to relevant files on the current release branch and rebuild the data so that gfe-db reflects the most current changes. The logic would look something like this:

  • State variables:
    • List of releases that have been processed (these are ignored, any release found not on this list is processed and added)
    • List of file assets used for processing (this list never changes)
  • Poll the IMGT repository and check for new releases
    • → if a new release is found then trigger a build
    • → or, if a commit has been made to one of the files used for processing on the current or a previously processed release branch then trigger a build
@chrisammon3000 chrisammon3000 self-assigned this Apr 10, 2023
@chrisammon3000 chrisammon3000 linked a pull request May 9, 2024 that will close this issue
2 tasks
@chrisammon3000
Copy link
Contributor Author

This feature branch needs to be merged with new commits in the main branch before the pull request can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant