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.

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

Improve test script #653

Merged
merged 3 commits into from
Jul 7, 2024
Merged

Improve test script #653

merged 3 commits into from
Jul 7, 2024

Conversation

kenhys
Copy link
Contributor

@kenhys kenhys commented Jul 1, 2024

  • set noninteractive not to block
  • do not set UID explicitly on ubuntu
  • switch to baseurl (only for testing with local container)

@kenhys kenhys force-pushed the improve-test-script branch 2 times, most recently from cd81df8 to a443699 Compare July 2, 2024 02:13
@kenhys
Copy link
Contributor Author

kenhys commented Jul 3, 2024

#654 should be merged in advance.

@daipom
Copy link
Contributor

daipom commented Jul 3, 2024

I think we need #655 to make this test succeed.

@kenhys kenhys force-pushed the improve-test-script branch from a443699 to 7991bbb Compare July 4, 2024 08:20
@kenhys
Copy link
Contributor Author

kenhys commented Jul 4, 2024

As #654 was merged, rebased it.

Copy link
Contributor

@daipom daipom left a comment

Choose a reason for hiding this comment

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

I have commented on a minor point, but it looks good to me!

fluent-package/test-install-in-docker.sh Outdated Show resolved Hide resolved
kenhys added 3 commits July 5, 2024 18:08
It is better not to be blocked with tzdata configuration.

Signed-off-by: Kentaro Hayashi <[email protected]>
Since ubuntu:noble, ID=1000 ubuntu user is created.
There is not a so many merit from specifying uid currently, so
just drop it.

Signed-off-by: Kentaro Hayashi <[email protected]>
As CentOS 7 reaches EOL, so it is expected that
mirrorlist will be broken gradually, so set baseurl explicitly to
improve network connectivity.

Signed-off-by: Kentaro Hayashi <[email protected]>
@kenhys kenhys force-pushed the improve-test-script branch from 7991bbb to b160a02 Compare July 5, 2024 09:09
@kenhys kenhys merged commit 9512351 into master Jul 7, 2024
56 checks passed
@kenhys kenhys deleted the improve-test-script branch July 7, 2024 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants