Isaac
Isaac
Looks like https://github.com/terraform-providers/terraform-provider-local/pull/5 will close this
@jamengual could you please review when you get a chance? This probably isn't very elegant and I'd like some input on: https://github.com/runatlantis/atlantis/pull/2495/files#diff-22d1bafb2e6e476aabe2f33484daa3f74932c6e0266f0830d7d948b8b9f60183R236-R243
Anything I can do to help move things along?
@michelmzs no this would be mering branchB into the mainline branch and I can say it's still happening with latest version of Atlantis and Gitlab version 15.2.2 To elaborate on...
Interesting, I hadn't realized that was a new layout in 15.x. I'll also look at whether this is unintended behavior that could be fixed in later 15.x versions or a...
@michelmzs I've confirmed this again locally and against the publicly hosted gitlab.com here: https://gitlab.com/Jukie/tf-demo/-/merge_requests/3#note_1130657254 This is with the latest stable release version of Atlantis and pre-release v0.20.1
@michelmzs any updates here? It's a confirmed issue with 15.x but I'm unclear whether it's something that should be resolved here or in Gitlab
Issue is in `mr.HeadPipeline.SHA` at: https://github.com/runatlantis/atlantis/blob/71233ef3b6ed8081dc49dc33c45aceeceaaea753/server/events/vcs/gitlab_client.go#L209
@jamengual now I'm wondering if this is a valid issue for Atlantis or if I should open another one at https://github.com/xanzy/go-gitlab? https://pkg.go.dev/github.com/xanzy/[email protected]?utm_source=gopls#MergeRequest.HeadPipeline
After digging into it a bit more the issue is here and how Atlantis is using the client. In my example mr.HeadPipeline is nil so that just needs to be...