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

create component with git context and revision #326

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

christianvogt
Copy link
Contributor

@christianvogt christianvogt commented Jan 12, 2023

Fixes

https://issues.redhat.com/browse/HAC-2743

Description

Git reference (as revision) and context were not set for CDQ and component creation

Type of change

  • Feature
  • Bugfix
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

How to test or reproduce?

Create a monorepo such that a git context can be provided and also ensure there is a valid branch on the repo to test with.
When creating a component, set the git reference and context appropriately.
Create the component.

Note that due to a bug in HAS, the revision is not returned in the component stub response of CDQ. As such the Component is still missing the revision at creation time. This should get resolved when the HAS bug is addressed

Browser conformance:

  • Chrome
  • Firefox
  • Safari
  • Edge

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 12, 2023
@rohitkrai03
Copy link
Contributor

Hey, I've merged the PR that fixed the e2e tests - #305. So going forward it would be good to run the e2e tests locally and paste the screenshot here on the PR before merging. Can you do that?

@christianvogt christianvogt force-pushed the git-options branch 2 times, most recently from 0658be0 to d0973d7 Compare January 13, 2023 21:29
@codecov
Copy link

codecov bot commented Jan 13, 2023

Codecov Report

Merging #326 (dbeb294) into main (b8ad85e) will increase coverage by 0.02%.
The diff coverage is 70.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #326      +/-   ##
==========================================
+ Coverage   75.07%   75.09%   +0.02%     
==========================================
  Files         432      432              
  Lines        9170     9150      -20     
  Branches     2483     2470      -13     
==========================================
- Hits         6884     6871      -13     
+ Misses       2162     2156       -6     
+ Partials      124      123       -1     
Impacted Files Coverage Δ
src/components/ImportForm/ImportForm.tsx 100.00% <ø> (ø)
...ponents/ImportForm/ReviewSection/ReviewSection.tsx 24.00% <0.00%> (ø)
src/components/ImportForm/utils/submit-utils.ts 100.00% <ø> (ø)
src/components/ImportForm/utils/types.ts 100.00% <ø> (ø)
...ponents/ImportForm/SampleSection/SampleSection.tsx 94.00% <75.00%> (ø)
...components/ImportForm/SourceSection/GitOptions.tsx 83.33% <75.00%> (-16.67%) ⬇️
...nents/ImportForm/ReviewSection/RuntimeSelector.tsx 92.45% <100.00%> (ø)
...omponents/ImportForm/SourceSection/AuthOptions.tsx 96.15% <100.00%> (ø)
...ponents/ImportForm/SourceSection/SourceSection.tsx 98.92% <100.00%> (ø)
...rc/components/ImportForm/utils/validation-utils.ts 100.00% <100.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b8ad85e...dbeb294. Read the comment docs.


setFieldValue('source', sourceUrl);
setFieldValue('source.git.url', newSourceUrl);
Copy link
Contributor

Choose a reason for hiding this comment

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

Sample validation schema needs to be updated as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated

@christianvogt
Copy link
Contributor Author

Private git source is always failing right now
image

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 18, 2023

@christianvogt: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Copy link
Contributor

@rottencandy rottencandy left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 18, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 18, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: christianvogt, rottencandy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@christianvogt christianvogt merged commit 3ebf9ff into openshift:main Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants