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

feat/handle-webhook-notifications #33

Merged
merged 14 commits into from
Feb 9, 2023
Merged

feat/handle-webhook-notifications #33

merged 14 commits into from
Feb 9, 2023

Conversation

spoukke
Copy link
Contributor

@spoukke spoukke commented Feb 3, 2023

No description provided.

@spoukke spoukke added this to the v0.1.0 milestone Feb 3, 2023
@spoukke spoukke linked an issue Feb 3, 2023 that may be closed by this pull request
@spoukke spoukke force-pushed the feat/github-webhook branch from f4372aa to 89607be Compare February 7, 2023 15:16
@spoukke spoukke linked an issue Feb 7, 2023 that may be closed by this pull request
@spoukke spoukke force-pushed the feat/github-webhook branch from 357bf22 to dc1bc9c Compare February 7, 2023 16:50
Copy link
Contributor Author

@spoukke spoukke left a comment

Choose a reason for hiding this comment

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

Need to add proper logging inside the server

fmt.Println("Received push event repo: %s, revision: %s, touchedHead: %v", webURL, revision, touchedHead)
}
// The next 2 lines probably dont work, waiting for chat GPT to be up \o/
repositories := &configv1alpha1.TerraformRepositoryList{}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to find how to get all TerraformRepository resources inside a namespace

}
// The next 2 lines probably dont work, waiting for chat GPT to be up \o/
// Should we link lkayer and repositories by label to make this list easier?
layers := &configv1alpha1.TerraformLayerList{}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to find how to get all TerraformLayer resources linked to a a TerraformRepository

internal/server/server.go Outdated Show resolved Hide resolved
@spoukke spoukke linked an issue Feb 7, 2023 that may be closed by this pull request
@spoukke spoukke changed the title add webhook to receive push notifications from github feat/handle-webhook-notifications Feb 7, 2023
internal/burrito/burrito.go Outdated Show resolved Hide resolved
internal/server/server.go Outdated Show resolved Hide resolved
@spoukke spoukke force-pushed the feat/github-webhook branch from 7c788ac to 11e8663 Compare February 9, 2023 10:50
@spoukke spoukke merged commit a206236 into main Feb 9, 2023
@spoukke spoukke deleted the feat/github-webhook branch February 22, 2023 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants