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

Adapt integration tests to reuse the packed charm #13

Merged
merged 14 commits into from
Aug 25, 2023

Conversation

arturo-seijas
Copy link
Collaborator

@arturo-seijas arturo-seijas commented Jul 21, 2023

Adapt the integration tests to reuse the exisintg charm artifact as per canonical/operator-workflows#162 and fix the docker build:

  • Install node using the nvm package manager to avoid installing the lsb_release package, which is failing to install due to issues with the docker storage driver
  • Remove the python2-saml package, which shouldn't be needed, which is failing to install due to issues with the docker storage driver

@arturo-seijas arturo-seijas requested a review from a team as a code owner July 21, 2023 10:16
jdkandersson
jdkandersson previously approved these changes Jul 24, 2023
weiiwang01
weiiwang01 previously approved these changes Aug 24, 2023
* Fix image build

* Fix image build

* Ignore CVE
weiiwang01
weiiwang01 previously approved these changes Aug 24, 2023
yanksyoon
yanksyoon previously approved these changes Aug 24, 2023
Copy link

@yanksyoon yanksyoon left a comment

Choose a reason for hiding this comment

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

LGTM!

weiiwang01
weiiwang01 previously approved these changes Aug 24, 2023
yanksyoon
yanksyoon previously approved these changes Aug 24, 2023
yanksyoon
yanksyoon previously approved these changes Aug 25, 2023
weiiwang01
weiiwang01 previously approved these changes Aug 25, 2023
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has totally checked 41 files.

Valid Invalid Ignored Fixed
12 10 19 0
Click to see the invalid file list
  • COPYRIGHT
  • actions.yaml
  • charmcraft.yaml
  • config.yaml
  • files/canonical_flavour/https-proxy-agent.patch
  • files/canonical_flavour/themes.patch
  • mattermost.Dockerfile
  • metadata.yaml
  • tests/conftest.py
  • tests/integration/localstack-installation.sh

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has totally checked 42 files.

Valid Invalid Ignored Fixed
19 2 21 0
Click to see the invalid file list
  • COPYRIGHT
  • tox.ini

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has totally checked 42 files.

Valid Invalid Ignored Fixed
19 2 21 0
Click to see the invalid file list
  • COPYRIGHT
  • tox.ini

weiiwang01
weiiwang01 previously approved these changes Aug 25, 2023
merkata
merkata previously approved these changes Aug 25, 2023
Copy link
Contributor

@merkata merkata left a comment

Choose a reason for hiding this comment

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

LGTM

tox.ini Outdated Show resolved Hide resolved
yanksyoon
yanksyoon previously approved these changes Aug 25, 2023
Copy link

@yanksyoon yanksyoon left a comment

Choose a reason for hiding this comment

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

LGTM!

.woke.yml Show resolved Hide resolved
merkata
merkata previously approved these changes Aug 25, 2023
@arturo-seijas arturo-seijas merged commit 24e2dd6 into main Aug 25, 2023
19 checks passed
@arturo-seijas arturo-seijas deleted the reuse-charm-in-tests branch August 25, 2023 15:48
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.

6 participants