fix(publish): inverted logic of noninteractive log on failure (#23154)

The logic condition was inverted for only logging failures from
noninteractive publish processes. It was only logging on success if set
to log failures. 😃

Author: jgranstrom

docs/notes/2.32.x.md

@@ -52,6 +52,8 @@ For `generate-lockfiles`, typos in the name of a resolve now give "Did you mean?
 
 `PublishFieldSet` now has a `check_skip_request` hook to enable preemptive skips of packaging requests for targets where the publishing will be skipped (such as when a `skip_push=True` field exists).
 
+Fixed inverted logic when setting noninteractive_process_output to only log failures.
+
 ### Backends
 
 #### Docker

src/python/pants/core/goals/publish.py

@@ -525,7 +525,7 @@ async def run_publish(
 
         if publish.noninteractive_process_output == ShowOutput.ALL or (
             publish.noninteractive_process_output == ShowOutput.FAILED
-            and background_res.exit_code == 0
+            and background_res.exit_code != 0
         ):
             console.print_stdout(output_msg)