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

Add output for warnings during apply phase #191

Open
cberge908 opened this issue May 23, 2022 · 4 comments
Open

Add output for warnings during apply phase #191

cberge908 opened this issue May 23, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@cberge908
Copy link

Suggestion

Hi,

I just cam a across a warning for a deprecated field, e.g:

Warning: "use_microsoft_graph": [DEPRECATED] This field now defaults to `true` and will be removed in v1.3 of Terraform Core due to the deprecation of ADAL by Microsoft.

I guess this will happen from time to time, as providers are changing.
Would it be possible to add this output from the apply action also to the PR comment output? This might save some time, as not many people will actively dig through the action log (if the action completed successfully).

Other idea would be a variable which can be used as output for another action (e.g. a send-mail action) to send these warnings.

Cheers,
cberge908

@cberge908 cberge908 added the enhancement New feature or request label May 23, 2022
@dflook
Copy link
Owner

dflook commented May 23, 2022

Hi @cberge908, warnings should be included at the bottom of the plan in the PR comment. Can you show the complete workflow log so we can try and see why that isn't working?

Could you also try and see if the warning is reported by dflook/terraform-validate?

@cberge908
Copy link
Author

Hi @dflook, the warning was only included in the log file of the apply action. During the tf plan action no warning was reported. Also the PR comment did not contain the warning at the bottom. Also after the apply was completed there was no update to the content of the PR comment (besides the change from applying plan -> plan applied in ...).

Regarding validate I can't really judge, I'm not using this action. Only plan, fmt and apply.

Do you need the plan or the apply job log? I'm asking because I need to mask some output (as we're using an own GHES instance).

@dflook
Copy link
Owner

dflook commented May 23, 2022

The apply log would be most helpful

@cberge908
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants