* feat: when using order group, abort plan/apply if any fail
* feat: add 'abort_on_execution_order_fail' flag on repo level
* feat: use runProjectCmdsParallelGroups in version_command_runner
* chore: add plan tests
---------
Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
* fix: Previous implementation tried to do comment parsing inside github, and only worked for 'atlantis' command.
This commit improves on that to make it possible to implement reactions in other VCS providers and also reuse the existing comment parsing.
* test: Add negative test to confirm reactions are not added for invalid comments
* test: Add a negative test to confirm invalid commands do not get a reaction
* Basic case insensitive for pr comments.
* Add test for uppercase/lowercase case.
* Merge test functions that check similar behaviors.
---------
Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
Co-authored-by: PePe Amengual <jose.amengual@gmail.com>
* feat: Basic implementation of github reactions on PRs
Adds eyes whenever it detects an `atlantis` command.
* feat: Make the emoji reaction configurable
* tests: Add a mocked test for EmojiReaction being called in github
* ci: Disable revive linter for stubs
Reverts "fix(github): branch protection not supported (#3276)"
Reverts "fix: allow Require Linear History when selecting merge method (#3211)"
Closes#3320
* Modify the automere commit message to include the PR number [DEVOPS-674]
* Update server/events/vcs/common/common.go
Co-authored-by: Ken Kaizu <k.kaizu38@gmail.com>
* added test for PR message func [DEVOPS-674]
---------
Co-authored-by: Ken Kaizu <k.kaizu38@gmail.com>
* merge again if base branch has been updated
The current "merge" checkout strategy is unsafe. It merges the PR and the base
branch without holding the directory lock(s), so there is a potentially
very long window where another PR can be applied and be unexpectedly reverted
later.
This happens occasionally if a PR causes plans in multiple directories, but
is almost _guaranteed_ to happen if the initial plan has to wait until a
lock is freed up and a manual "atlantis plan" command is given.
Instead of printing a warning when this happens, we now merge again
if necessary while holding the lock. Plans are then guaranteed to only
be made when merged with the base branch for each directory being planned,
and applying later should be safe even if the base branch sees further
updates.
This fixes/affects #804, #867, #979
* Remove diverged test that no longer applies
* Reinstate TestClone_MasterHasDiverged test and make it pass
* Extend TestClone_MasterHasDiverged to test new merging functionality
We now verify that the first Clone with CheckoutMerge=true with
a diverged base branch atually clones and merges again.
---------
Co-authored-by: PePe Amengual <jose.amengual@gmail.com>
The tests would mutate $HOME/.gitconfig and $HOME/.git-credentials.
Set the HOME environment variable to a suitable temporary
directory before running git commands that mutate the global config.
Fixes#2604
* fix: account for Require Linear History when selecting merge method
* fix logic and add tests
* add nolint hints for w.Write
---------
Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
* Bugfix: TestProjectCommandOutputHandler is unstable
Use buffered channels in TestProjectCommandOutputHandler
The output handler registered with
ProjectCommandOutputHandler.Register() needs the channel that
is given to not block writes. If the channel
blocks writes, the channel is removed from the list
of active receivers on that JobID.
The tests are set up with unbuffered channels, and there
is a race condition where the channel reader does
not hit `for msg := range ch` before the message that
is sent is handled by AsyncProjectCommandOutputHandler.writeLogLine
By making the channel buffered it does not matter
that the reader isn't reading yet.
* Add missing wg.Wait() in TestProjectCommandOutputHandler
There is a rare race condition where messages come in
while the asserts run in the
"clean up all jobs when PR is closed" test.
Added wg.Wait() to make sure all messages are processed
before running the final asserts.
---------
Co-authored-by: Finn Arne Gangstad <finnag@gmail.com>
* Adding check for Azure DevOps Test Webhook URL so they aren't processed like real events
* feat: customize vcs comment command executable name
* Altering case on isAzureDevOpsTestRepoURL function
* Altering case on URL parameter
* Altering checks to ensure all Azure DevOps Ignore tests pass
* Extending setup method to support Azure DevOps
* Fixing issue from merge conflict error
* gofmt
* Adding null checks to Test URL checks
* moving azuredevopsTestUrl to const
* Changing azuredevopsTestUrl to azuredevopsTestURL
* Cleaning up return statement
Co-authored-by: Ken Kaizu <k.kaizu38@gmail.com>
---------
Co-authored-by: Ken Kaizu <k.kaizu38@gmail.com>
Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
* feat: state rm
* review feedback
* fix conflict for pegomock generation code
* adopt state command into allow-commands
* fix conflicts
* fix: state rm works on workspace
* notify import/state rm discard plan file
* fix lint
* use repeat instead warning for re-plan
* perl -pi -e 's!\* 🔁 plan file was discarded. to!🚮 A plan file was discarded. Re-plan would be required before applying.\n\n\* 🔁 To!g' server/**/*
* follow main branch
* add web_templates render tests
* add ansic strip tests
* move fixtures into testdata dir which is golang specific test dir name
* add server/metrics tests
* add recovery test
* add runtime stats test
* feat: Update plan GitHub Status summary
* feat: revert result changes for Plan and Apply functions
* docs: Add CommandResult description
* feat: change plan status without ctx.CommandResult
* ensure DefaultCommitStatusUpdater implements runtime.statusUpdater
Co-authored-by: Alberto Rojas <alberto.rojas@n26.com>
