Skip to content

Comments

Remove migrated deep storage standard ITs #16933

Merged
kfaraz merged 20 commits intoapache:masterfrom
findingrish:azure-gcs-it-group
Sep 5, 2024
Merged

Remove migrated deep storage standard ITs #16933
kfaraz merged 20 commits intoapache:masterfrom
findingrish:azure-gcs-it-group

Conversation

@findingrish
Copy link
Contributor

@findingrish findingrish commented Aug 21, 2024

@findingrish findingrish changed the title Add new standard IT groups Remove migrated deep storage standard ITs Aug 27, 2024
Comment on lines 22 to 26
# CentOS is EOL, have to use vault
RUN sed -i s/mirror.centos.org/vault.centos.org/g /etc/yum.repos.d/*.repo && \
sed -i s/^#.*baseurl=http/baseurl=http/g /etc/yum.repos.d/*.repo && \
sed -i s/^mirrorlist=http/#mirrorlist=http/g /etc/yum.repos.d/*.repo

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this warrants a separate PR with a clear explanation of what is being done here.
The rest of the changes look good.

Copy link
Contributor Author

@findingrish findingrish Sep 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted changes in this file.

Copy link
Contributor Author

@findingrish findingrish Sep 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Raised separate PR here #16999.

@findingrish findingrish requested a review from kfaraz September 4, 2024 10:14
@kfaraz kfaraz merged commit 40f38f0 into apache:master Sep 5, 2024
edgar2020 pushed a commit to edgar2020/druid that referenced this pull request Sep 5, 2024
@kfaraz kfaraz added this to the 31.0.0 milestone Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants