Marvin Bredal Lillehaug
Marvin Bredal Lillehaug
@mbfrahry When I update the conflict in CHANGELOG.md, should the change be amended and force pushed, or is a merge commit OK?
Ok, I have rebased and pushed with not changes to CHANGELOG.md.
Yeah, looks like it. Added one now.
Yeah, no idea what that means. Changing the expected version string fails in the expected way. So it seems that this is something else.
No. Someone have to find out what `After applying this test step, the non-refresh plan was not empty.` means and why it appears on the new test for java 21,...
Thank you! Will take a look in a few days. On Thu, Nov 28, 2024, 14:05 jackofallops ***@***.***> wrote: > ***@***.**** requested changes on this pull request. > > Hi...
``` marlil@marlil:~/src/terraform-provider-azurerm$ make acctests SERVICE='appservice' TESTARGS='-run=TestAccLinuxWebApp_withJre21Java' TESTTIMEOUT='60m' ==> Checking that code complies with gofmt requirements... ==> Checking that Custom Timeouts are used... ==> Checking that acceptance test packages are used......
I can confirm that 1.20.3 fixes the problem. A relief after having spent a few hours trying to stepping through the code looking for the cause of ``` 9:33:14 PM...
The status is that I have a [PR](https://github.com/hashicorp/terraform-provider-azurerm/pull/26304) that has a failing test, that I have no idea how to resolve.