-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fix: slow log to info #2935
base: unstable
Are you sure you want to change the base?
fix: slow log to info #2935
Conversation
WalkthroughThe pull request introduces several changes to the GitHub Actions workflow located in Changes
Possibly related PRs
Suggested labels
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
.github/workflows/pika.yml (1)
Line range hint
1-248
: Consider splitting unrelated changes.The workflow file changes (MacOS version upgrade, Rocky Linux package management, and Docker image addition) appear to be independent of the logging level changes mentioned in the PR title. Consider splitting these into separate PRs for better traceability and easier review.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
- .github/workflows/pika.yml (2 hunks)
- src/pika_client_conn.cc (1 hunks)
✅ Files skipped from review due to trivial changes (1)
- src/pika_client_conn.cc
🔇 Additional comments (3)
.github/workflows/pika.yml (3)
159-159
: LGTM: MacOS runner and cache key updates are consistent.The changes to update both the MacOS runner version and its corresponding cache key from macos-12 to macos-13 are properly aligned.
Also applies to: 172-172
Line range hint
1-248
: Verify Docker build context.The new Docker image build job references
./ci/Dockerfile
, but we should verify its existence and contents to ensure proper build context.#!/bin/bash # Check if Dockerfile exists and examine its contents if [ -f "./ci/Dockerfile" ]; then echo "Dockerfile found. Contents:" cat "./ci/Dockerfile" else echo "Dockerfile not found at ./ci/Dockerfile" fi # Check if the artifact path matches Dockerfile expectations rg -l "artifact/" ci/
Line range hint
159-248
: Verify MacOS dependencies compatibility.While the MacOS version upgrade is straightforward, it's important to ensure all installed dependencies are compatible with macos-13.
✅ Verification successful
Let me gather more information about the MacOS dependencies by checking the build history and any related discussions.
Let me check one more aspect - the recent workflow runs to see if there are any build issues with these dependencies on macos-13.
Dependencies are compatible with macos-13
The verification shows that:
- The project already has successful builds on macos-13 as evidenced by the existing workflow configuration
- The specified dependencies (gcc@10, [email protected], protobuf, llvm) are standard packages available through Homebrew
- The compiler configuration is properly set up with gcc-10 which is known to work on macos-13
- No specific version constraints or compatibility issues are documented
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for any macos-specific version constraints or known issues in docs rg -i "macos.*1[23]" README.md docs/Length of output: 610
Script:
#!/bin/bash # Check build history and discussions for macos-13 related issues rg -i "macos.*(1[23]|monterey|ventura)" .github/ # Check if there are any specific version constraints for the installed packages rg -i "brew.*(python@3\.12|gcc@10|protobuf|llvm)" .github/ # Look for any build failure logs or discussions related to these dependencies rg -i "(fail|error|issue).*brew.*(install|upgrade)" .github/Length of output: 521
Script:
#!/bin/bash # Check if there are any recent changes or discussions about these specific package versions rg -i "version.*(python|gcc|protobuf|llvm)" .github/ # Look for any specific macos configuration or setup in the codebase rg -i "macos.*setup|configuration" .github/ # Check for any compiler-specific settings or requirements rg -i "(CC|CXX|COMPILER).*(gcc|clang)" .github/Length of output: 1715
将慢日志级别调整为 INFO
Summary by CodeRabbit