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.

Sign up for GitHub

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

To fix the issue: show_techsupport & saidump errors during testbed testing by replacing redis-rdb-tool with rdb-cli #19268

Merged
merged 3 commits into from
Nov 7, 2024
Merged

To fix the issue: show_techsupport & saidump errors during testbed testing by replacing redis-rdb-tool with rdb-cli #19268

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f042307
To fix the issue: show_techsupport & saidump errors during testbed te…
JunhongMao Jun 11, 2024
6b5dbc5
According to the review comments, update the rdb-cli installation loc…
JunhongMao Aug 2, 2024
5dc10e6
According to reivew comments, build librdb just like any other regul…
JunhongMao Oct 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 0 additions & 12 deletions dockers/docker-base-bookworm/Dockerfile.j2
100644 → 100755
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,18 +81,6 @@ RUN pip3 install supervisord-dependent-startup==1.4.0

RUN mkdir -p /var/log/supervisor /etc/supervisor/conf.d

# Install gcc, libc6-dev and python3-dev for compiling python-lzf
RUN apt-get -y install build-essential libc6-dev python3-dev

# Install python-lzf
RUN pip3 install 'python-lzf==0.2.4'

# Install rdbtools
RUN pip3 install 'rdbtools==0.1.15'

# Uninstall gcc, libc6-dev and python3-dev for compiling python-lzf
RUN apt-get -y purge build-essential libc6-dev python3-dev

# Uninstall unused dependencies
RUN apt autoremove -y --purge

Expand Down
12 changes: 0 additions & 12 deletions dockers/docker-base-bullseye/Dockerfile.j2
100644 → 100755
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,18 +83,6 @@ RUN pip3 install supervisord-dependent-startup==1.4.0

RUN mkdir -p /var/log/supervisor /etc/supervisor/conf.d

# Install gcc, libc6-dev and python3-dev for compiling python-lzf
RUN apt-get -y install build-essential libc6-dev python3-dev

# Install python-lzf
RUN pip3 install 'python-lzf==0.2.4'

# Install rdbtools
RUN pip3 install 'rdbtools==0.1.15'

# Uninstall gcc, libc6-dev and python3-dev for compiling python-lzf
RUN apt-get -y purge build-essential libc6-dev python3-dev

# Uninstall unused dependencies
RUN apt autoremove -y --purge

Expand Down
18 changes: 18 additions & 0 deletions platform/broadcom/docker-syncd-brcm/Dockerfile.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,24 @@ COPY ["supervisord.conf", "/etc/supervisor/conf.d/"]
COPY ["files/supervisor-proc-exit-listener", "/usr/bin"]
COPY ["critical_processes", "/etc/supervisor/"]

# Install make and git for compiling librdb
RUN apt-get -y install build-essential git

# Obtain librdb source codes
RUN git clone https://github.com/redis/librdb.git

# Build librdb
RUN cd ./librdb/;git checkout b401df1e7025501c24f0c852aa32f6900fd888f7;git submodule update --init --recursive;make;cd ..
JunhongMao marked this conversation as resolved.
Show resolved Hide resolved

# Install rdb-cli
RUN install -p -D -m 0775 ./librdb/bin/rdb-cli /usr/bin/rdb-cli

# Clear librdb source codes
RUN rm -rf ./librdb

# Uninstall make and git for compiling librdb
RUN apt-get -y purge build-essential git

## Clean up
RUN apt-get clean -y; apt-get autoclean -y; apt-get autoremove -y
RUN rm -rf /debs
Expand Down
Loading