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

Jump to bottom

[systemtest] remove (and replace with bazel) #12446

Closed
2 tasks done
Tracked by #12449
timothytrippel opened this issue May 3, 2022 · 0 comments · Fixed by #12929
Closed
2 tasks done
Tracked by #12449

[systemtest] remove (and replace with bazel) #12446

timothytrippel opened this issue May 3, 2022 · 0 comments · Fixed by #12929
Assignees
@milesdai
Labels
Component:Tooling Issues related to tooling, e.g. tools/scripts for doc, code generation (docgen, reggen), CSR Priority:P1 Priority: high SW:Build System

Comments

@timothytrippel
Copy link
Contributor

timothytrippel commented May 3, 2022 •
edited
Loading

After completing #12445, the system test python test framework can be removed and permanently replaced by bazel:

  • remove system test CI jobs
  • remove system test completely from the repository
@timothytrippel timothytrippel added Component:Tooling Issues related to tooling, e.g. tools/scripts for doc, code generation (docgen, reggen), CSR SW:Bazel Adoption Requirement SW:Build System labels May 3, 2022
@timothytrippel timothytrippel mentioned this issue May 3, 2022
Closed
18 tasks
@arunthomas arunthomas added the Priority:P0 Priority: critical label May 3, 2022
@drewmacrae drewmacrae assigned drewmacrae and unassigned drewmacrae May 4, 2022
@timothytrippel timothytrippel added Priority:P1 Priority: high and removed Priority:P0 Priority: critical labels May 6, 2022
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 27, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
73d0fe2 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
@timothytrippel timothytrippel mentioned this issue May 27, 2022
Merged
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue May 27, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
2da6941 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 2, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
1f95458 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 3, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
97e4b8d 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 3, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
fc7eda7 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 3, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
9edada4 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
@drewmacrae drewmacrae removed their assignment Jun 6, 2022
timothytrippel added a commit to timothytrippel/opentitan that referenced this issue Jun 6, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
4e3db2a 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes lowRISC#12446.

Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel added a commit that referenced this issue Jun 6, 2022
@timothytrippel
[systemtest] remove systemtest in favor of bazel + opentitantool
3b0ac11 
Now that bazel/opentitantool are running all CI chip-level tests,
systemtest may be removed. This will enable us to remove meson (which
systemtest relies on).

This fixes #12446.

Signed-off-by: Timothy Trippel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@milesdai milesdai

Labels
Component:Tooling Issues related to tooling, e.g. tools/scripts for doc, code generation (docgen, reggen), CSR Priority:P1 Priority: high SW:Build System
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[systemtest] remove systemtest in favor of bazel + opentitantool timothytrippel/opentitan
4 participants
@arunthomas @drewmacrae @timothytrippel @milesdai