Skip to content

Step 4: Process logs and generate inventory.txt#1

Open
BGoulem wants to merge 10 commits intomainfrom
assignment
Open

Step 4: Process logs and generate inventory.txt#1
BGoulem wants to merge 10 commits intomainfrom
assignment

Conversation

@BGoulem
Copy link
Owner

@BGoulem BGoulem commented Mar 3, 2026

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

I was trying to organize create files on git and organize log files.

What did you learn from the changes you have made?

I have learned how to copy and organize files, and transfer files to a txt file.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I was thinking of using a manual copy strategy (cp) for the log files as I was struggling with copying the server_log.log files

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

I encountered many challenges, namely with the assignment.sh file being in a different location on my computer. I had to copy this file to newproject. I relied heavily on lecture slides, videos, and internet resources that helped troubleshoot. I think I took a very roundabout way to get to the end result, and I am eager to learn how to undo things. I had to delete the file manually many times, and re-upload the git script. I eventually felt comfortable restarting from the already-downloaded forked file.

How were these changes tested?

the changes were tested by running bash assignment.sh, I also checked in the file on my computer to ensure that the log files were organized, that the txt file was in the right place and showing what I wanted it to show.

A reference to a related issue in your repository (if applicable)

Checklist

  • [x ] I can confirm that my changes are working as intended (I think!)

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.

Autograder results

question status comment
Part 1 - Q1 data directory does not exist
Part 1 - Q2 rawdata not moved to data/raw
Part 1 - Q3 ls command not run
Part 1 - Q4 Missing directories: data/processed/server_logs, data/processed/user_logs, data/processed/event_logs
Part 1 - Q5 No server log files in data/raw
Part 1 - Q6 No user log files in data/raw; No event log files in data/raw
Part 1 - Q7
Part 1 - Q8 data/inventory.txt does not exist
Part 2 coworker-changes branch not found in commit history

Please address the issues listed above.

@BGoulem BGoulem marked this pull request as draft March 3, 2026 17:42
Brigid Goulem added 5 commits March 3, 2026 20:39
…e data directory has been made, but no log files have yet been added.

:wq

 Please enter the commit message for your changes. Lines starting
…ta. I have not yet moved the raw data into the files.
… is the script up to step 8.

Please enter the commit message for your changes. Lines starting
…e files have been organized and there is a txt file with a list of all log files.
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.

Autograder results

question status comment
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2 coworker-changes branch not found in commit history

Please address the issues listed above.

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.

Autograder results

question status
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2

@BGoulem BGoulem marked this pull request as ready for review March 4, 2026 02:08
Copy link

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi left a comment

Choose a reason for hiding this comment

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

Approved

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.

3 participants