Skip to content

chore: Update server/events/working_dir Logging Configuration#3636

Merged
GenPage merged 5 commits intorunatlantis:mainfrom
X-Guardian:ServerEventWorkingDir-Log-Update
Aug 1, 2023
Merged

chore: Update server/events/working_dir Logging Configuration#3636
GenPage merged 5 commits intorunatlantis:mainfrom
X-Guardian:ServerEventWorkingDir-Log-Update

Conversation

@X-Guardian
Copy link
Contributor

what

Update the server/events/working_dir file logging configuration:

  • Add logging messages to state changing functions (Delete and DeleteForWorkspace).
  • Update functions in working_dir (Clone, HasDiverged, forceClone, recheckDiverged), to use the parent object logger rather than requiring a logger passed to each function. This simplifies the code in the calling functions and unit tests.

@X-Guardian X-Guardian requested a review from a team as a code owner July 29, 2023 11:55
@github-actions github-actions bot added go Pull requests that update Go code provider/github labels Jul 29, 2023
Copy link
Member

@GenPage GenPage left a comment

Choose a reason for hiding this comment

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

:shipit:

@GenPage GenPage merged commit b024cef into runatlantis:main Aug 1, 2023
@X-Guardian X-Guardian deleted the ServerEventWorkingDir-Log-Update branch October 27, 2023 08:05
ijames-gc pushed a commit to gocardless/atlantis that referenced this pull request Feb 13, 2024
…antis#3636)

* Update server/events/working_dir logging

* Add e2e FileWorkspace logger

* Fix github app working dir test logger

* Update working_dir_test

---------

Co-authored-by: Dylan Page <dylan.page@autodesk.com>
ijames-gc pushed a commit to gocardless/atlantis that referenced this pull request Feb 13, 2024
…antis#3636)

* Update server/events/working_dir logging

* Add e2e FileWorkspace logger

* Fix github app working dir test logger

* Update working_dir_test

---------

Co-authored-by: Dylan Page <dylan.page@autodesk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code provider/github

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants