[PR #2682] [MERGED] Changed some INFO level log messages to DBG level #2706

Closed
opened 2026-03-04 02:06:54 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/2682
Author: @ggtakec
Created: 6/6/2025
Status: Merged
Merged: 6/6/2025
Merged by: @gaul

Base: masterHead: update/loglevel


📝 Commits (1)

  • a9ad25e Changed some INFO level log messages to DBG level

📊 Changes

3 files changed (+3 additions, -7 deletions)

View changed files

📝 src/curl.cpp (+1 -1)
📝 src/curl_util.cpp (+2 -2)
📝 src/s3fs.cpp (+0 -4)

📄 Description

Relevant Issue (if applicable)

n/a

Details

Some message output levels have been changed from INFO to DBG.

The changed messages previously required the INFO level, but I have now determined that it is no longer necessary to have it at that level.
Also, by changing these message output levels, the execution time of Github Actions(CI) can be slightly reduced.

  • s3fs_getxattr
    On MacOS, there are a large number of calls, so it is already at the DBG level.
    On Linux systems, there are also a large number of calls, so we can change it to the DBG level.

  • prepare_url
    This function rewrites the request URL, but this function is stable and does not need to be INFO, so we can change it.

  • S3fsCurl::insertV4Headers
    Although this function makes a signature, we decided that there is always no need to check the contents when it is operating normally, so we can change it.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/2682 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 6/6/2025 **Status:** ✅ Merged **Merged:** 6/6/2025 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `update/loglevel` --- ### 📝 Commits (1) - [`a9ad25e`](https://github.com/s3fs-fuse/s3fs-fuse/commit/a9ad25ec2ae01e738caa931839584ee16cb7a887) Changed some INFO level log messages to DBG level ### 📊 Changes **3 files changed** (+3 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+1 -1) 📝 `src/curl_util.cpp` (+2 -2) 📝 `src/s3fs.cpp` (+0 -4) </details> ### 📄 Description ### Relevant Issue (if applicable) n/a ### Details Some message output levels have been changed from `INFO` to `DBG`. The changed messages previously required the `INFO` level, but I have now determined that it is no longer necessary to have it at that level. Also, by changing these message output levels, the execution time of Github Actions(`CI`) can be slightly reduced. - s3fs_getxattr On MacOS, there are a large number of calls, so it is already at the `DBG` level. On Linux systems, there are also a large number of calls, so we can change it to the `DBG` level. - prepare_url This function rewrites the request URL, but this function is stable and does not need to be `INFO`, so we can change it. - S3fsCurl::insertV4Headers Although this function makes a signature, we decided that there is always no need to check the contents when it is operating normally, so we can change it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-04 02:06:54 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/s3fs-fuse#2706
No description provided.