-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
chore(ci): upload coredump #11401
chore(ci): upload coredump #11401
Conversation
c09ec6f
to
54d9ea6
Compare
0d70e60
to
459dea5
Compare
fd3e1a1
to
61a57c5
Compare
61a57c5
to
92eb548
Compare
f8c09dc
to
f3a3f17
Compare
sudo mkdir -p ${{ env.BUILD_ROOT }}/kong-dev | ||
sudo tar -C ${{ env.BUILD_ROOT }}/kong-dev -xzvf artifacts/build.tar.gz | ||
|
||
- name: Install Kong dev |
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.
are those Install Kong dev
steps needed?
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.
@fffonion It will install some dev-rocks and some binary like grpcurl
.
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.
iirc we already installed the dev rocks on build step for ce (and cached) 🤔
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.
- name: Build Dev Kong dependencies
if: steps.cache-deps.outputs.cache-hit != 'true'
run: |
make install-dev-rocks
This line has been removed in this PR, to keep consistency with EE.
f3a3f17
to
51e29a8
Compare
51e29a8
to
1e4a11c
Compare
Is this still needed? |
Summary
Upload core dump if it exists, it is good for troubleshooting in the local env,
Checklist
Full changelog
N/A
Issue reference
N/A